New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bleadperl v5.25.3-232-g10f9b9b breaks LEONT/Const-Fast-0.014.tar.gz #15522
Comments
From @andkbisect commit 10f9b9b move Internals::hv_clear_placeholders() to Hash::Util::_clear_placeholders() diagnostics http://www.cpantesters.org/cpan/report/40ee7106-62a0-11e6-b236-c19558b9f28c perl -V Summary of my perl5 (revision 5 version 25 subversion 4) configuration: Characteristics of this binary (from libperl): |
From @eserteAlso affected: BMORROW/Readonly-Tiny-3.tar.gz |
The RT System itself - Status changed from 'new' to 'open' |
From @LeontOn Tue, Aug 16, 2016 at 9:44 PM, Andreas J. Koenig via RT <
I think the removal from Internals:: was a bit premature. This function is Or actually, what would be even better is having a sensible split between Leon |
From @andkalso affected: TOBYINK/Sub-Trigger-Lock-0.002.tar.gz -- |
From @demerphqOn 24 August 2016 at 16:41, Leon Timmermans <fawaka@gmail.com> wrote:
TL;DR: It was readded some time ago. But if you are going to use
I have a few issues with this. First off, Internals::hv_clear_placehold() is only needed to get rid So, if your code is meant to mark a data structure readonly then it is Second, as far as I can tell the code in Const::Fast overlaps with and However what you have done in this module is use undocumented Third, I think that functionality like Const::Fast should be coded in Having said all that: I re-added hv_clear_placehold() back to
I think we should ditch restricted hashes, and just have what you Yves -- |
From @jkeenanOn Fri, 26 Aug 2016 06:15:37 GMT, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
Appears to have been fixed at least as of: This is perl 5, version 25, subversion 9 (v5.25.9 (v5.25.8-76-gb1b8fb6)) built for x86_64-linux-thread-multi I was able to use 'cpanm' to install the distro at this point in blead. Ticket not yet closable until the Const-Fast discussion is resolved. -- |
From @iabynOn Sun, Dec 25, 2016 at 06:51:40PM -0800, James E Keenan via RT wrote:
But unless I've misunderstood this thread, this ticket is no longer a -- |
From @jkeenanOn Mon, 20 Mar 2017 12:59:21 GMT, davem wrote:
Agreed; removing from 5.26.0 blockers list. -- |
I think it's closable, though at some point we may need to have a policy around |
Migrated from rt.perl.org#128966 (status was 'open')
Searchable as RT128966$
The text was updated successfully, but these errors were encountered: