Bug #128074 for Sub-Auto: Sub::Auto::inject_scope is using hash hint to store references

Thu Dec 20 15:42:17 2018 me [...] eboxr.com - Ticket created

Subject:

Sub::Auto::inject_scope is using hash hint to store references

this is not a good idea to store references to hash hint like performed by inject_scope # $^H{DD_METHODHANDLERS} = Scope::Guard->new(..); View the grep here: https://grep.metacpan.org/search?qci=&q=%5C%24%5C%5EH.*%3D.*%7B&qft=&qd=Sub-Auto&f=lib%2FSub%2FAuto.pm The hints hash (%^H) cannot be used to store hashrefs as values, so any implementation that relies on putting hashrefs in %^H is intrinsically broken. See "Implementation details" in the perlpragma manpage. https://perldoc.perl.org/perlpragma.html#Implementation-details

Thu Dec 20 18:09:17 2018 osfameron [...] cpan.org - Correspondence added

Thanks for the reply - would you like maint? On Thu Dec 20 15:42:17 2018, atoomic wrote: Show quoted text

> this is not a good idea to store references to hash hint like > performed by inject_scope > # $^H{DD_METHODHANDLERS} = Scope::Guard->new(..); > > View the grep here: > https://grep.metacpan.org/search?qci=&q=%5C%24%5C%5EH.*%3D.*%7B&qft=&qd=Sub- > Auto&f=lib%2FSub%2FAuto.pm > > The hints hash (%^H) cannot be used to store hashrefs as values, > so any implementation that relies on putting hashrefs in %^H is > intrinsically broken. > > See "Implementation details" in the perlpragma manpage. > https://perldoc.perl.org/perlpragma.html#Implementation-details

Thu Dec 20 18:09:17 2018 The RT System itself - Status changed from 'new' to 'open'

Thu Dec 20 18:52:27 2018 me [...] eboxr.com - Correspondence added

Would be glad to help and provide a release for the module and probably move it at the same time to GitHub and set travis running. My PAUSE nickname is atoomic. You would need to give me co-maint permission on the distro. thanks nicolas On Thu Dec 20 18:09:17 2018, OSFAMERON wrote: Show quoted text

> Thanks for the reply - would you like maint? > > On Thu Dec 20 15:42:17 2018, atoomic wrote:

> > this is not a good idea to store references to hash hint like > > performed by inject_scope > > # $^H{DD_METHODHANDLERS} = Scope::Guard->new(..); > > > > View the grep here: > > https://grep.metacpan.org/search?qci=&q=%5C%24%5C%5EH.*%3D.*%7B&qft=&qd=Sub- > > Auto&f=lib%2FSub%2FAuto.pm > > > > The hints hash (%^H) cannot be used to store hashrefs as values, > > so any implementation that relies on putting hashrefs in %^H is > > intrinsically broken. > > > > See "Implementation details" in the perlpragma manpage. > > https://perldoc.perl.org/perlpragma.html#Implementation-details

Fri Dec 21 03:54:37 2018 Hakim.Cassimally [...] bbc.co.uk - Correspondence added

Subject:	Re: [rt.cpan.org #128074] Sub::Auto::inject_scope is using hash hint to store references
Date:	Fri, 21 Dec 2018 08:42:27 +0000
To:	"bug-Sub-Auto [...] rt.cpan.org" <bug-Sub-Auto [...] rt.cpan.org>
From:	Hakim Cassimally <Hakim.Cassimally [...] bbc.co.uk>

Done! Please feel free to do anything you feel appropriate with the module. I'm happy to transfer ownership entirely if you need any more privs than co-maint. I don't much time to spend on it, but happy to have a quick chat on Skype/irc at some point if you have any questions on anything. (And of course, if you lose interest/time yourself, don't worry about it at all, but thanks for the interest :-) -- Hakim Cassimally, Technology Training From: Atoomic via RT <bug-Sub-Auto@rt.cpan.org<mailto:bug-Sub-Auto@rt.cpan.org>> Reply-To: "bug-Sub-Auto@rt.cpan.org<mailto:bug-Sub-Auto@rt.cpan.org>" <bug-Sub-Auto@rt.cpan.org<mailto:bug-Sub-Auto@rt.cpan.org>> Date: Thursday, 20 December 2018 at 23:52 Subject: [rt.cpan.org #128074] Sub::Auto::inject_scope is using hash hint to store references Queue: Sub-Auto Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=128074 > Would be glad to help and provide a release for the module and probably move it at the same time to GitHub and set travis running. My PAUSE nickname is atoomic. You would need to give me co-maint permission on the distro. thanks nicolas On Thu Dec 20 18:09:17 2018, OSFAMERON wrote: Thanks for the reply - would you like maint? On Thu Dec 20 15:42:17 2018, atoomic wrote: Show quoted text

> this is not a good idea to store references to hash hint like > performed by inject_scope > # $^H{DD_METHODHANDLERS} = Scope::Guard->new(..); > > View the grep here: > https://grep.metacpan.org/search?qci=&q=%5C%24%5C%5EH.*%3D.*%7B&qft=&qd=Sub- > Auto&f=lib%2FSub%2FAuto.pm > > The hints hash (%^H) cannot be used to store hashrefs as values, > so any implementation that relies on putting hashrefs in %^H is > intrinsically broken. > > See "Implementation details" in the perlpragma manpage. > https://perldoc.perl.org/perlpragma.html#Implementation-details

---------------------------- http://www.bbc.co.uk This e-mail (and any attachments) is confidential and may contain personal views which are not the views of the BBC unless specifically stated. If you have received it in error, please delete it from your system. Do not use, copy or disclose the information in any way nor act in reliance on it and notify the sender immediately. Please note that the BBC monitors e-mails sent or received. Further communication will signify your consent to this. ---------------------

Fri Dec 21 11:18:05 2018 me [...] eboxr.com - Correspondence added

Ticket migrated to github as https://github.com/atoomic/Sub-Auto/issues/1

Fri Dec 21 11:18:05 2018 me [...] eboxr.com - Status changed from 'open' to 'resolved'