in reply to Re: Risks in the oblivious use of qr//
in thread Risks in the oblivious use of qr//

Agreed. I think this is a clear bug and just indicates that when a qr// object is used as an entire expression with different flags, the local flags should override. Similarly, when it is interpolated into a larger expression it should also be subject to local rules.

  • Comment on Re: Re: Risks in the oblivious use of qr//

Replies are listed 'Best First'.
Re: Re: Re: Risks in the oblivious use of qr//
by waswas-fng (Curate) on Aug 11, 2003 at 04:41 UTC
    hmm, are you saying that: 
    $myregex = qr/test/i;
if (/$myregex = yes/) {...}

    [download]
    should or should not be case insensitive for the /test/ part of the regex? to me the lack of options at the end of a regex means dont use them, but it makes sence to me that the precompiled regex would force the flags given to it. I dont see the bug there...

    -Waswas
[reply]
[d/l]

      My thinking is that affirmatively setting a flag should override. Since there is no way to positively unset a flag I would expect this to be a case insensitive match. Similarly, if you swapped /i between the expressions I would still expect this to be case insensitive.

      Added: I'm going to moderate my opinion here. I thought a bit further and on consideration, the flags /s, /m and /x are likely not safe to have a locally specified flag override. The only qr// flag that could sanely be overridden is /i and in this case I still call the current behaviour a bug.

[reply]

        The only qr// flag that could sanely be overridden is /i and in this case I still call the current behaviour a bug.

        Nope. I disagree. If thats a bug then there would be no way to disable /i from inside the regex when the i modifier is used. Which would totally break quite a few regexes that I have. And conversly, would putting a regex compiled with /i into a regex without /i cause the i to cease functioning? Definately not a bug.


        ---
        demerphq

        <Elian> And I do take a kind of perverse pleasure in having an OO assembly language...
[reply]
[d/l]
        I guess my expectations would be precompiled regex are static and immutable in both shape and flag options so the behavior makes sense to me. how would you reap any speed benefit if the interpreter had to reevaluate all locations where qr//'ed regex were used to recompile and apply flags?

        -Waswas
[reply]

In Section Meditations