Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Javadoc nanosecond-related methods in org.jruby.RubyTime #5130

Merged
merged 1 commit into from Apr 6, 2018

Conversation

dmikurube
Copy link
Contributor

Hi!

As I was misunderstanding nsec in RubyTime, and trapped in the pitfall when manipulating RubyTime from Java, I just wondered if I could leave that in Javadoc explicitly. Nothing is described in http://jruby.org/apidocs/org/jruby/RubyTime.html for the time being.

I'm totally fine to get them edited, but wanted at least to describe they're not the entire fraction part. What do you think?

@kares
Copy link
Member

kares commented Apr 6, 2018

Thanks Dai, this is good stuff.
I know I got confused as well. The reason for the API is how JODA's DateTime works it handles only millis so the nanos part is tracked manually. Which I guess you figured, planning some API updates for the Java side e.g. publicize getTimeInMillis etc. so we might add a method to calculate nanos properly.

@kares kares merged commit bc2a772 into jruby:master Apr 6, 2018
@kares kares added this to the Non-Release milestone Apr 6, 2018
@dmikurube dmikurube deleted the javadoc-for-RubyTime-nsec branch April 9, 2018 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants