Small patch for the URL perl matcher

[ubitux]

On Thu, Mar 11, 2010 at 09:39:45PM +0100, Marc Lehmann wrote:
> On Wed, Mar 10, 2010 at 11:37:47AM +0100, ubitux <ubitux at gmail.com> wrote:
> > I expect some "issues" with the URL perl matcher: for example, urls like
> > http://translate.google.com/#en|fr|nail are not well recognized due to the
> 
> The problem is that the url in your example is
> "http://translate.google.com/#en", it ends at the |, which cannot be part
> of the URL.
> 
> > '|' character. There is also some URL with '[', ']', '{' and '}' which are
> > not converted by browsers when copying them.
> 
> Sounds like the well-known firefox bug that corrupts URLs. This is a known
> bug in firefox and is supposed to be fixed in some future version.
> 

If it's an issue, I've the same one with Chromium, and I suppose it's the
same for every other browsers…

> > So I attached a small patch to this mail in order to fix it. I'm also
> > wondering if other characters like '^' or '`' should be appened, but
> > since I never saw them in an URL...
> 
> The problem is that adding all these URL-delimueters as valid characters will
> make it harder to select valid urls - it's easier to extend then to shorten,
> so I guess we should be more strict rather than less.
> 
> Note that you can provide your own url if you feel the need to work around
> firefox bugs (or similar bugs in other programs).

Well, as you want, it's too bad :(

Anyway, I'll just note that the URL is well recognized under Mailman :)

> 
> -- 
>                 The choice of a       Deliantra, the free code+content MORPG
>       -----==-     _GNU_              http://www.deliantra.net
>       ----==-- _       generation
>       ---==---(_)__  __ ____  __      Marc Lehmann
>       --==---/ / _ \/ // /\ \/ /      schmorp at schmorp.de
>       -=====/_/_//_/\_,_/ /_/\_\

-- 
ubitux