On 02/18/2010 09:53 AM, Bram Moolenaar wrote:
>
> James Vega wrote:
>>> diff -Naur vim72.360.rubypatch/src/if_ruby.c vim72.361/src/if_ruby.c
>>> --- vim72.360.rubypatch/src/if_ruby.c 2010-02-18
06:48:21.000000000 -0500
>>> +++ vim72.361/src/if_ruby.c 2010-02-18 06:44:32.000000000 -0500
>>> @@ -864,7 +869,10 @@
>>> long n = NUM2LONG(num);
>>> aco_save_T aco;
>>>
>>> - if (n >= 0 && n <= buf->b_ml.ml_line_count && line != NULL)
>>> + if (line != NULL) {
>>
>> This condition should have been changed to "line == NULL" when it was
>> made its own, standalone check.
>
> Ah, good catch. I'll send out a patch.
>
> Would be nice to have a test for the basic Ruby stuff, so that things
> like this are caught early.
Thanks - that fixed it. I now get the same behavior as with
Masaki's patch. On 32-bit Fedora 11 with ruby 1.8.6, things are
fine. On 64-bit Arch Linux with ruby 1.9.1p378, there odd error
message is still there for the LustyExplorer plugin. I suspect
that's a problem with the plugin and ruby 1.9, but I'll look
into that separately and submit a bug report to the plugin
author.
Thanks for the prompt patches :-)
Michael Henry
--
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
