rb_time_new prototype doesn't match definition
----------------------------------------------

                 Key: JRUBY-5468
                 URL: http://jira.codehaus.org/browse/JRUBY-5468
             Project: JRuby
          Issue Type: Bug
          Components: C Extensions
            Reporter: Jeremy Evans
            Assignee: Thomas E Enebo
         Attachments: patch-cext_src_include_ruby_ruby_h

The prototype for rb_time_new doesn't match the definition.  The prototype is:

  RUBY_DLLSPEC VALUE rb_time_new(long sec, long usec);

https://github.com/jruby/jruby/blob/master/cext/src/include/ruby/ruby.h#L1003

the definition is:

  extern "C" VALUE
  rb_time_new(time_t sec, long usec)

https://github.com/jruby/jruby/blob/master/cext/src/time.cpp#L23

On OpenBSD i386, the compiler errors out because time_t is int and and not 
long.  Changing the prototype to time_t fixes the issue.  Patch attached.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to