On Thu Jul 11 17:28:56 2013, tonyc wrote: > On Sun Jun 23 18:04:00 2013, tonyc wrote: > > So should we be removing (or deprecating) the SvREFCNT_*() modifiers > > from Devel::Peek, but keep SvREFCNT() itself? > > > > Or nuke them all, since they're available (if deliberately undocumented) > > in Internals? > > Here's an alternative patch that: > > - removes the modifier functions > > - modifies the SvREFCNT() XS function to accept any type of object > > - adds a very basic test > > Note that the original patch leaked the SV SvREFCNT() returned. > > For debate, I'll apply it in a few days unless there's some > overwhelmingly negative response. There was no response at all, everyone loves it! Applied as da1929e7566d4399b20541f8609bd48b40663034. Tony --- via perlbug: queue: perl5 status: open https://rt.perl.org:443/rt3/Ticket/Display.html?id=117793Thread Previous | Thread Next