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
Unclear note about \Q, etc., in perlretut #11215
Comments
From @cpansproutAt <http://www.nntp.perl.org/group/perl.perl5.porters/2011/03/msg170208.html>, Tom Christiansen pointed out that this paragraph I added to perlretut is missing \l and \u: +C<\Q>, C<\L>, C<\U> and C<\E> are actually part of the syntax of regular Then Karl Williamson pointed out that ‘regular expression literal’ is not defined anywhere. So I would like to change this to: C<\Q>, C<\L>, C<\l>, C<\U>, C<\u> and C<\E> are actually part of Flags: Site configuration information for perl 5.13.11: Configured by sprout at Thu Mar 24 14:02:46 PDT 2011. Summary of my perl5 (revision 5 version 13 subversion 11) configuration: Locally applied patches: @INC for perl 5.13.11: Environment for perl 5.13.11: |
From tchrist@perl.com
And \N{}. --tom |
The RT System itself - Status changed from 'new' to 'open' |
From @khwilliamsonOn 03/27/2011 01:47 PM, Father Chrysostomos (via RT) wrote:
+1 |
From @cpansproutOn Mar 27, 2011, at 12:57 PM, tchrist1 via RT wrote:
$ perl5.13.11 -le '$p = q"\N{U+66}"; print "ofo" =~ /o $p o/x' What am I missing? |
From tchrist@perl.com
Gr. It's the charnames issue. --tom |
From @khwilliamsonOn 03/27/2011 02:24 PM, Tom Christiansen wrote:
By which he means that in something like \N{COLON}, the name is |
From @demerphqOn 27 March 2011 22:44, Karl Williamson <public@khwilliamson.com> wrote:
Im kinda aware of the issues here and I didn't understand that -- |
From @khwilliamsonOn 03/28/2011 06:45 AM, demerphq wrote:
http://groups.google.com/group/perl.perl5.porters/msg/0716102cfbcde32e |
From @khwilliamsonOn 03/28/2011 09:43 AM, Karl Williamson wrote:
To add some to that. The solution to #56444 was to move the parsing of The problem is that regex compilation is not done at the time of the Since the time I fixed #56444, it has come to light that the \N{} But a possibility for future work is to revisit this fix, and put things But in 5.14, \N should be listed in the pods as not being part of the |
From @demerphqOn 28 March 2011 18:14, Karl Williamson <public@khwilliamson.com> wrote:
Ok. I get it now. The lexer only sees patterns compiled at compile time, not those Historically: Long ago BOTH the regex engine and the lexer handled \N{ .... }. Then we moved it into the regex engine only. (because we thought that Then we moved it back to the lexer as a translator. But I guess when So I dont see this as a problem of moving the logic back to the lexer, To repeat: *any* construct the lexer handles for the regex engine (and We should put that in a comment somewhere. cheers, -- |
From @demerphqOn 28 March 2011 18:14, Karl Williamson <public@khwilliamson.com> wrote:
I dont get you here. I don't see how charnames *can* be fixed and not The problem here is purely that the translation logic has to be if ($foo=~/\N{bar}/) { and via a string in something like: if ($foo=~/$bar/) { So when $bar contains "\N{bar}" we will find it in the regex engine. Yves -- |
From @cpansproutOn Sun Mar 27 12:47:01 2011, sprout wrote:
<http://www.nntp.perl.org/group/perl.perl5.porters/2011/03/msg170208.html>,
I have just committed this change as 8e71069. |
From [Unknown Contact. See original ticket]On Sun Mar 27 12:47:01 2011, sprout wrote:
<http://www.nntp.perl.org/group/perl.perl5.porters/2011/03/msg170208.html>,
I have just committed this change as 8e71069. |
@cpansprout - Status changed from 'open' to 'resolved' |
From @epaFather Chrysostomos <perlbug-followup <at> perl.org> writes:
Unless that regular expression uses '' as delimiters? -- |
From @cpansproutOn Mar 30, 2011, at 6:04 AM, Ed Avis via RT wrote:
Yes, but I did say ‘double-quotish syntax’. Since this is tutorial, I think we can leave that implicit. |
Migrated from rt.perl.org#87128 (status was 'resolved')
Searchable as RT87128$
The text was updated successfully, but these errors were encountered: