Front page | perl.perl5.porters |
Postings from July 2011
[oops, meant to send this to p5p, not perl5-changes]
In article
<craigberry-5E8443.16544210072011@cpc2-bmly6-0-0-cust974.2-3.cable.virgi
nmedia.com>,
craigberry@mac.com ("Craig A. Berry") wrote:
> In article <E1Qfjck-0005rH-A1@camel.ams6.corp.booking.com>,
> sprout@cpan.org ("Father Chrysostomos") wrote:
>
> > In perl.git, the branch blead has been updated
> >
> > <http://perl5.git.perl.org/perl.git/commitdiff/777d9014444bb88a98ccd6c09ebb5
> > 20
> > cdc4c0d8b?hp=fe57f3b7598666107c5e6e9c9ffd844da47ea527>
> >
> > - Log -----------------------------------------------------------------
> > commit 777d9014444bb88a98ccd6c09ebb520cdc4c0d8b
> > Author: Father Chrysostomos <sprout@cpan.org>
> > Date: Sat Jul 9 19:18:45 2011 -0700
> >
> > Propagate (non-)lvalue context through nested calls
>
> In case you're not following the smoke reports, this broke the build on
> all the stricter platforms and configurations because PUSH_MULTICALL is
> used by List::Util, but PUSH_MULTICALL is defined in terms of PUSHSUB
> and PUSHSUB now calls a new private function that is not exported.
>
>
> > --- a/embed.fnc
> > +++ b/embed.fnc
> > @@ -545,6 +545,8 @@ p |bool |io_close |NN IO* io|bool not_implicit
> > : Used in perly.y
> > pR |OP* |invert |NULLOK OP* cmd
> > ApR |I32 |is_lvalue_sub
> > +: Used in cop.h
> > +pR |I32 |was_lvalue_sub
>
> It looks to me like PUSH_MULTICALL is part of the public API, or at
> least it's documented in perlcall.pod. So I think you're going to want
> an X in there to make Perl_was_lvalue_sub visible outside of perllib.
Thread Next