Tony Mechelynck wrote:
> On 17/03/10 18:15, Bram Moolenaar wrote:
> >
> > Patch 7.2.400 (after 7.2.387)
> > Problem: Dynamic Ruby is not initialised properly for version 1.9.1.
> > Ruby cannot create strings from NULL.
> > Solution: Cleanup #ifdefs. Handle NULL like an empty string. Add
> > ruby_init_stack. (Sergey Khorev)
> > Files: src/if_ruby.c
>
> After this patch, I get a warning (probably harmless) as follows:
>
> if_ruby.c: In function =91vim_to_ruby=92:
> if_ruby.c:726: warning: pointer type mismatch in conditional expression
>
> The relevant hunk is the following:
>
> > ***************
> > *** 720,775 ****
> >
> > if (tv->v_type =3D=3D VAR_STRING)
> > {
> > ! result =3D rb_str_new2((char *)tv->vval.v_string);
> > }
> > else if (tv->v_type =3D=3D VAR_NUMBER)
> [...]
> > --- 722,778 ----
> >
> > if (tv->v_type =3D=3D VAR_STRING)
> > {
> > ! result =3D rb_str_new2((char *)(tv->vval.v_string =3D=3D NULL
> > ! ? "" : tv->vval.v_string));
> > }
> > else if (tv->v_type =3D=3D VAR_NUMBER)
> [...]
I'll fix it. Strange that I missed the warning previously.
--
My girlfriend told me I should be more affectionate.
So I got TWO girlfriends.
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ download, build and distribute -- http://www.A-A-P.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
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
To unsubscribe from this group, send email to
vim_dev+unsubscribegooglegroups.com or reply to this email with the words
"REMOVE ME" as the subject.
