Re: (no subject)

[Kevin Rodgers]

Werner LEMBERG wrote:
> [CVS 2006-10-28]
>
> I've just seen the following link in an email:
>
>   
> http://www.google.com/codesearch?hl=en&q=+pango_layout_(iter_)%3Fget_(lines%3F%7Crun)%5B%5E_%5D&start=50&sa=N
>
> As you can probably can see immediately, thingatpt only colorizes (and
> uses) the string up to `..._layout_', stopping at the opening
> parenthesis.
>
> Is this intentional or is this a bug?

It is intentional, as parentheses are explicitly excluded from URLs by
thing-at-point-url-path-regexp.  It could be a bug, as parentheses are
defined as reserved characters in URIs, which are only sometimes percent-
encoded, depending on the context; but I think they are allowed within
both the path and query components of generic URIs (without being
percent-encoded).

So just as the slashes in the URI above should not need to be %-encoded
(since they follow the ?, they are clearly within the query component),
neither should the parentheses need to be %-encoded -- but perhaps they
ought to be.

--
Kevin Rodgers
Denver, Colorado, USA