in reply to Re^6: Parser Performance Question (updated)
in thread Parser Performance Question

>
" .. \\" .. " Eily found " .. \\" .. " LanX found " .. \\"

Well, you proved that my version is correct and yours doesn't work.

:p

Cheers Rolf
(addicted to the Perl Programming Language and ☆☆☆☆ :)
Je suis Charlie!

Replies are listed 'Best First'.
Re^8: Parser Performance Question (updated)
by Eily (Monsignor) on Oct 06, 2017 at 08:45 UTC

    Hu, how did I ever reach sainthood in our book? Eily => qr/ " (?: [^"\\] | \\. )* " /x); <= added the missing " in the character class

    use Data::Dump qw( pp ); my $str = <<STR; " .. \\" .. STR print $str; my %re = (LanX => qr/ " (?: \\\\ | \\" | [^"] )* " /x, Eily => qr/ " (?: [^"\\] | \\. )* " /x); $str =~ /$re{$_}/ and say "$_ found $&" or say "$_ found nothing" for +keys %re;
    " .. \" .. Eily found nothing LanX found " .. \"

      Atomic grouping (?>...) seems to fix the problem
      use strict; use warnings; use feature 'say'; use Data::Dump qw( pp ); my @strs = qw( "..\\".. "abc" "a\"bc" "a\\\\bc" "a\" ); my %re = ( LanX => qr/ " (?> \\\\ | \\" | [^"] )* " /x, Eily => qr/ " (?: [^"\\] | \\. )* " /x ); for my $str (@strs) { say "\nTesting: <$str> = ", pp ($str); $str =~ /$re{$_}/ and say "$_ found $&" or say "$_ found nothing" for keys %re; }

      Testing: <"..\"..> = "\"..\\\".." LanX found nothing Eily found nothing Testing: <"abc"> = "\"abc\"" LanX found "abc" Eily found "abc" Testing: <"a\"bc"> = "\"a\\\"bc\"" LanX found "a\"bc" Eily found "a\"bc" Testing: <"a\\bc"> = "\"a\\\\bc\"" LanX found "a\\bc" Eily found "a\\bc" Testing: <"a\"> = "\"a\\\"" LanX found nothing Eily found nothing

      Cheers Rolf
      (addicted to the Perl Programming Language and ☆☆☆☆ :)
      Je suis Charlie!

        I had completly forgotten about the atomic match :). I reread the documentation (because I wasn't sure what it does exactly), and it shows that it is equivalent to the possessive quantifiers. So in the spirit of TIMTOWTDI:

        use strict; use warnings; use feature 'say'; use Data::Dump qw( pp ); my @strs = qw( "..\\".. "abc" "a\"bc" "a\\\\bc" "a\" ); my %re = ( LanX => qr/ " (?> \\\\ | \\" | [^"] )* " /x, Eily => qr/ " (?: [^"\\] | \\. )* " /x, Poss => qr/ " (?: \\\\ | \\" | [^"] )*+ " /x, ); for my $str (@strs) { say "\nTesting: <$str> = ", pp ($str); $str =~ /$re{$_}/ and say "$_ found $&" or say "$_ found nothing" for keys %re; }