Hi Damien,

2016/5/2 Mon 8:55:40 UTC+9 Damien wrote:
> Le Sun, May 01, 2016 at 09:23:04PM +0300, LCD 47 a écrit :
> >     This seems to be a recent change in Perl rather than Vim:
> > 
> > clang -c -I. -Iproto -DHAVE_CONFIG_H     -pipe -O2 -U_FORTIFY_SOURCE 
> > -D_FORTIFY_SOURCE=1       -I/usr/include/python3.5m 
> > -DPYTHON3_HOME='L"/usr"' -pthread 
> > -DDYNAMIC_PYTHON3_DLL=\"libpython3.5m.so.1.0\"  -o objects/if_python3.o 
> > if_python3.c
> > In file included from if_perl.xs:610:
> > /usr/lib/perl5/CORE/inline.h:20:5: error: use of undeclared identifier 
> > 'PERL_ARGS_ASSERT_AV_TOP_INDEX'
> >     PERL_ARGS_ASSERT_AV_TOP_INDEX;
> >     ^
> > /usr/lib/perl5/CORE/inline.h:242:5: error: use of undeclared identifier 
> > 'PERL_ARGS_ASSERT_APPEND_UTF8_FROM_NATIVE_BYTE'
> >     PERL_ARGS_ASSERT_APPEND_UTF8_FROM_NATIVE_BYTE;
> >     ^
> > /usr/lib/perl5/CORE/inline.h:274:5: error: use of undeclared identifier 
> > 'PERL_ARGS_ASSERT__IS_UTF8_CHAR_SLOW'
> >     PERL_ARGS_ASSERT__IS_UTF8_CHAR_SLOW;
> >     ^
> > /usr/lib/perl5/CORE/inline.h:305:5: error: use of undeclared identifier 
> > 'PERL_ARGS_ASSERT_IS_SAFE_SYSCALL'
> >     PERL_ARGS_ASSERT_IS_SAFE_SYSCALL;
> >     ^
> > /usr/lib/perl5/CORE/inline.h:390:5: error: use of undeclared identifier 
> > 'PERL_ARGS_ASSERT_SV_ONLY_TAINT_GMAGIC'
> >     PERL_ARGS_ASSERT_SV_ONLY_TAINT_GMAGIC;
> >     ^
> > 
> > $ perl --version | head -2
> > 
> > This is perl 5, version 22, subversion 2 (v5.22.2) built for 
> > i586-linux-thread-multi
> > 
> >     /lcd
> 
> Hi LCD and list,
> 
> Please check this patch 
> https://framagit.org/Damien43/patches/raw/master/vim/perl-5.22.2.patch 

I haven't try your patch, because ActivePerl/StrawberryPerl 5.22.2 is not
released yet. BTW, personally I don't like this part:

> @@ -777,7 +784,7 @@ newWINrv(SV *rv, win_T *ptr)
>       sv_setiv(ptr->w_perl_private, PTR2IV(ptr));
>      }
>      else
> -     SvREFCNT_inc(ptr->w_perl_private);
> +     SvREFCNT((SV *)ptr->w_perl_private)++;
>      SvRV(rv) = ptr->w_perl_private;
>      SvROK_on(rv);
>      return sv_bless(rv, gv_stashpv("VIWIN", TRUE));

Isn't it better to (re)defining SvREFCNT_inc instead of using SvREFCNT()++?

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.

Raspunde prin e-mail lui