Hi, 2016/6/13 Mon 23:38:50 UTC+9 Ken Takata wrote: > Hi Bram, > > 2016/6/12 Sun 5:27:17 UTC+9 Bram Moolenaar wrote: > > Patch 7.4.1922 > > Problem: Ruby 2.4.0 unifies Fixnum and Bignum into Integer. > > Solution: Use rb_cInteger. (Weiong Mao) > > Files: src/if_ruby.c > > This patch has some indentation problems. Attached patch fixes them. > > > > + # if defined(DYNAMIC_RUBY_VER) && DYNAMIC_RUBY_VER >= 24 > > + # define USE_RUBY_Integer > > + #endif > > Is this definition name okay? > I think USE_RUBY_INTEGER is prefer. (all upper case)
Sorry, missing attachment. Regards, Ken Takata -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
# HG changeset patch # Parent ba37207d9d55e53678ee668e0b4f3b267407b3b5 diff --git a/src/if_ruby.c b/src/if_ruby.c --- a/src/if_ruby.c +++ b/src/if_ruby.c @@ -31,8 +31,8 @@ # define RUBYEXTERN extern #endif -# if defined(DYNAMIC_RUBY_VER) && DYNAMIC_RUBY_VER >= 24 - # define USE_RUBY_Integer +#if defined(DYNAMIC_RUBY_VER) && DYNAMIC_RUBY_VER >= 24 +# define USE_RUBY_Integer #endif #ifdef DYNAMIC_RUBY @@ -44,7 +44,7 @@ # define rb_cFalseClass (*dll_rb_cFalseClass) # define rb_cFixnum (*dll_rb_cFixnum) # if defined(USE_RUBY_Integer) -# define rb_cInteger (*dll_rb_cInteger) +# define rb_cInteger (*dll_rb_cInteger) # endif # if defined(DYNAMIC_RUBY_VER) && DYNAMIC_RUBY_VER >= 20 # define rb_cFloat (*dll_rb_cFloat)
