Re^3: Precompiling qr/.../o question

[hardburn]

Sure, it technically works (probably because /o gets dragged along into qr// like any other regex option list), but it's still redundant and confusing.

The problem with /o is when you start with something harmless like this:

for (1 .. 5) {
    print "Match!\n" if $some_string =~ /very long regex/o;
}

Then later you realize you need to add a loop variable to the regex:

for (1 .. 5) {
    print "Match!\n"
        if $some_string =~ /very long regex $_ more regex/o;
}

Which results in the regex checking against '1' instead of the current loop var. This is almost never what you want, and has caused new and experienced programmers alike to waste many hours of debugging. That's why qr// was invented to Do the Right Thing.

[mr_mischief]

Your wording read to me like you meant that the /o option was semantically redundant when used on the qr// itself.

Now I realize you meant something else entirely. I think you're saying that as a feature it's been made redundant because there are better ways to accomplish that it was originally meant to do. I wouldn't disagree with that.