Hrm. You might want to add \s and probably ? (at least, I think amearse doesn't want the query string included...) to your character class, since both of these would also delimit the end of the URL.

In fact, you might just be better off using an affirmative class instead of a negative one, since the list of allowable characters is only [\w/:$-_.+!*'(),%@] (though I've often seen ~ unescaped, and there might be others that are commonly not escaped properly...) (and, again, this is only for the non-query-string part of the URL).

Then again, there's a module on CPAN to do all of this (and more) already...

bbfu
Seasons don't fear The Reaper.
Nor do the wind, the sun, and the rain.
We can be like they are.