The attached patch appears to do the right thing for me (but I haven't
tested it extensively).

Amos Bird <amosbird@gmail.com> schrieb am Do., 15. Juni 2017 um 11:04 Uhr:

>
> hmm, can you try doing it at a prog mode? say cc-mode or java-mode.
>
> Philipp Stephani <p.stephani2@gmail.com> writes:
>
> > Yuri Khan <yuri.v.khan@gmail.com> schrieb am Mi., 14. Juni 2017 um
> > 17:03 Uhr:
> >
> >> On Wed, Jun 14, 2017 at 5:11 PM, Amos Bird <amosbird@gmail.com> wrote:
> >>
> >> > When doing  (goto-address-at-point) at parenthesis surrounded urls
> like
> >> this
> >> > (https://github.com/nikomatsakis/rayon/issues/231) in any prog mode
> (c,
> >> java, python ...)
> >> >
> >> > it will grab the trailing ')' thus ends up with opening a wrong url.
> >>
> >> > Could you tell me a workaround for this?
> >>
> >> Don’t do that, then?
> >>
> >> A closing parenthesis is a valid URL character. Any heuristic that
> >> fixes your problem will break legal, standard-conforming URLs, such as
> >> this:
> >>
> >>     https://en.wikipedia.org/wiki/Dunnet_(video_game)
> >>
> >
> > The URL heuristic could match only paired parentheses. At least that's
> what
> > my mail app appears to be doing, as it correctly matches both URLs.
>
>