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
lc() or uc() inside sort affect the return value. #8927
Comments
From @shlomifCreated by @shlomifIf you run: <<<<<<<<<<<<<<<<<< You'll get the output of: {{{{{ Which indicates that the output result of the string "$h" that was given as It doesn't seem to affect a simple $h, without the quotes. Perl Info
|
From @rgsOn 13/06/07, via RT Shlomi Fish <perlbug-followup@perl.org> wrote:
I don't understand your report, but it looks like you want a stable |
The RT System itself - Status changed from 'new' to 'open' |
From @JuerdRafael Garcia-Suarez skribis 2007-06-13 15:52 (+0200):
"$h" should be Hello, but hello is printed. Apparently lc($a) mutated juerd waalboer: perl hacker <juerd@juerd.nl> <http://juerd.nl/sig> |
From @rgsOn 13/06/07, Juerd Waalboer <juerd@convolution.nl> wrote:
aah, yes. Very odd. |
From a.r.ferreira@gmail.comOn 6/13/07, Rafael Garcia-Suarez <rgarciasuarez@gmail.com> wrote:
Nope. The issue was lc() changing its argument. The example with sort Here's another: $ perl -e ' $a = "Hello"; for ("$a") { lc $_; print }' When the "$a" gets aliased, it is being changed in place. It does not happen without quotes or with constants: $ perl -e ' $a = "Hello"; for ($a) { lc $_; print }' $ perl -e ' for ("Hello") { lc $_; print }' |
From @rgsOn 13/06/07, Adriano Ferreira <a.r.ferreira@gmail.com> wrote:
Yes, it seems that the logic in pp_lc is flawed: if (SvPADTMP(source) && !SvREADONLY(source) && !SvAMAGIC(source) Maybe we should add an SvTEMP check here too... |
@rgs - Status changed from 'open' to 'resolved' |
From @rgsOn 13/06/07, Rafael Garcia-Suarez <rgarciasuarez@gmail.com> wrote:
Now added (and to uc/lcfirst/ucfirst too) as change #31377. |
From @cpansproutOn Thu Jun 14 04:26:54 2007, rafael wrote:
(aka 17fa077) But PADTMPs are never SvTEMP, so now this optimisation never happens. I This bug is actually related to #78194. If we make for() and ()x... -- Father Chrysostomos |
From @cpansproutOn Sun Jul 29 11:04:32 2012, sprout wrote:
I have reënabled in-place uc/lc in commit 5cd5e2d. -- Father Chrysostomos |
Migrated from rt.perl.org#43207 (status was 'resolved')
Searchable as RT43207$
The text was updated successfully, but these errors were encountered: