-
Notifications
You must be signed in to change notification settings - Fork 556
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
Delete does not work on % slices #15982
Comments
From altreus@altre.usThis is a bug report for perl from altreus@altre.us, The `delete` operator deletes one or many keys from a hash and returns the It seems logical that if one were to run delete on a % section of a hash, it my %sanitised_values = delete %input_values{@allowed_keys}; Flags: Site configuration information for perl 5.20.1: Configured by alastair.mcgowan at Thu Jan 29 12:23:29 GMT 2015. Summary of my perl5 (revision 5 version 20 subversion 1) configuration: Locally applied patches: @INC for perl 5.20.1: Environment for perl 5.20.1: |
From @jkeenanOn Fri, 19 May 2017 10:40:45 GMT, altreus@altre.us wrote:
Can you explain your concept of the '$', '@' and '%' "sections" of a hash? I have been programming in Perl for some time and have never heard of "sections" of a hash. Nor can I find any mention of them in the documentation. Thank you very much.
-- |
The RT System itself - Status changed from 'new' to 'open' |
From @maukeAm 19.05.2017 um 13:20 schrieb James E Keenan via RT:
http://perldoc.perl.org/perldata.html#Slices -- |
From @jkeenanOn Fri, 19 May 2017 11:23:41 GMT, plokinom@gmail.com wrote:
I'm familiar with slices. It's the OP's use of the term "sections" that puzzles me. Thank you very much. -- |
From @ilmari(via RT) <perlbug-followup@perl.org> writes:
As is my wont, I thought "how hard could it possibly be" and had a go. https://perl5.git.perl.org/perl.git/commitdiff/smoke-me/ilmari/delete-kvslice -- |
From @xsawyerxOn 05/20/2017 03:22 PM, Dagfinn Ilmari Mannsåker wrote:
Also possible to merge to blead-next, which will be merged when 5.27 opens. |
From @ilmariilmari@ilmari.org (Dagfinn Ilmari Mannsåker) writes:
This is now in blead as commit cc0776d. -- |
From @demerphqOn 2 June 2017 at 12:05, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> wrote:
I'll just note that the new syntax was an error in previous perl's, in Which is real cool. Thanks for making it happen! cheers, -- |
Migrated from rt.perl.org#131328 (status was 'open')
Searchable as RT131328$
The text was updated successfully, but these errors were encountered: