Dominique Pelle wrote:

> 2008/11/29 Bram Moolenaar wrote:
> 
> > Tony Mechelynck wrote:
> ...[snip...]
> >> if_perl.xs:463: null argument where non-null required
> >>       a line specific to Perl ver. 5.10 (included) to 6.0 (excluded).
> >
> > I actually think NULL is OK here.  At least, it has always worked.  I
> > don't know how to pass something else, one can't simply pass an empty
> > environment.
> 
> 
> I see this compilation warning as well. In my Perl the header file
> /usr/lib/perl/5.10.0/CORE/proto.h on Ubuntu-8.10, I see:
> 
> 
> PERL_CALLCONV void      Perl_sys_init(int* argc, char*** argv)
>                         __attribute__nonnull__(1)
>                         __attribute__nonnull__(2);
> 
> PERL_CALLCONV void      Perl_sys_init3(int* argc, char*** argv, char*** env)
>                         __attribute__nonnull__(1)
>                         __attribute__nonnull__(2)
>                         __attribute__nonnull__(3);
> 
> 
> The "__attribute__nonnull(3)" explains why we get the warning.
> 
> We can probably call the other function Perl_sys_init(...) instead
> of Perl_sys_init3(...).  The patch below gets rid of the compilation
> warning but I prefer to let the author of the Perl interface (in CC)
> verify whether it's correct.

You also need to adjust the other places where "init3" appears.
Otherwise it looks like a good change to avoid the warning.  Question is
if it also works with all versions of Perl.

-- 
`When any government, or any church for that matter, undertakes to say to
 its subjects, "This you may not read, this you must not see, this you are
 forbidden to know," the end result is tyranny and oppression no matter how
 holy the motives' -- Robert A Heinlein, "If this goes on --"

 /// 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.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Raspunde prin e-mail lui