"Adam D. Barratt" <[email protected]> writes:

> Assuming the above is correct, is there any reason we shouldn't change
> the test from the current
>
>         $LEADIN . qr'source\s+[\"\']?(?:\.\/|\/|\$)[^\s]+'
>
> to something more along the lines of
>
>         $LEADIN . qr'source\s+[\"\']?(?:\.\/|\/|\$|[\w.-])[^\s]+'
>         
> ?

I can't think of any reason not to flag that.  Seems like a good idea to
me.

-- 
Russ Allbery ([email protected])               <http://www.eyrie.org/~eagle/>


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to