develooper Front page | perl.perl5.porters | Postings from September 2017

Re: [perl #132142] Bleadperl v5.27.3-34-gf6107ca24b breaksMLEHMANN/AnyEvent-HTTP-2.23.tar.gz

Thread Previous | Thread Next
From:
Tony Cook
Date:
September 25, 2017 22:23
Subject:
Re: [perl #132142] Bleadperl v5.27.3-34-gf6107ca24b breaksMLEHMANN/AnyEvent-HTTP-2.23.tar.gz
Message ID:
20170925222310.GI1651@mars.tony.develop-help.com
On Fri, Sep 22, 2017 at 03:48:18AM +0100, Zefram wrote:
> Dagfinn Ilmari Mannsaker wrote:
> >With the old behaviour, getting the strengthening effect was trivial:
> 
> With the new behaviour, one can fairly trivially get the
> weakness-preserving effect by wrapping the possibly-weak references in
> an extra layer of reference.
> 
> >This makes me think that despite all the work that's gone into making
> >the in-place sort optimisation invisible, the weak ref special case
> >needs to be kept, i.e. the above commit reverted.
> 
> -1.  The optimisation should be a pure optimisation.  If the useful
> behaviour of the old faulty optimisation were somehow not easily
> reproduced, it should be made available by explicit means, not kept as
> a funny special case of an operator that normally means something else.

I agree.

Tony

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About