[perl #122070] [PATCH] Change newSVpvn("…", …) tonewSVpvs("…")

On Thu Jun 12 00:50:41 2014, ilmari wrote:
> "Tony Cook via RT" <perlbug-followup@perl.org> writes:
> 
> > The change to Data::Dumper did fix a bug, the following:
> >
> > ./perl -Ilib -MData::Dumper -le 'sub x {} print Data::Dumper-
> > >new([\&x, \\&x], ["*b", "c"])->Dump'
> >
> > would produce the incorrect result you predicted.  The pure perl code
> > produced the correct result.
> >
> > I added a test for the correct behaviour in
> > 4662a05932175fe185d382b9b1b730e2b5cfad25.
> 
> Shouldn't that test be un-TODOed now that it's fixed?

Removed the $TODO in b65827489f432c6b8ea39e1ae6abef4e8d8b0723.

This didn't actually make the test TODO, since dumper.t uses its own test framework which ignores $TODO.

Tony


---
via perlbug:  queue: perl5 status: resolved
https://rt.perl.org/Ticket/Display.html?id=122070

• [perl #122070] [PATCH] Change newSVpvn("…", …) tonewSVpvs("…") by Tony Cook via RT