The attached patch appears to do the right thing for me (but I haven't tested it extensively).
Amos Bird <
address@hidden> 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 <address@hidden> writes:
> Yuri Khan <address@hidden> schrieb am Mi., 14. Juni 2017 um
> 17:03 Uhr:
>
>> On Wed, Jun 14, 2017 at 5:11 PM, Amos Bird <address@hidden> 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.