GNU bug report logs - #74132
31.0.50; thing-at-pt, ffap and Github markdown

Previous Next

Package: emacs;

Reported by: Madhu <enometh <at> meer.net>

Date: Thu, 31 Oct 2024 10:43:02 UTC

Severity: wishlist

Tags: patch

Found in version 31.0.50

Full log


View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: enometh <at> meer.net, 74132 <at> debbugs.gnu.org
Subject: bug#74132: 31.0.50; thing-at-pt, ffap and Github markdown
Date: Sat, 09 Nov 2024 18:33:05 +0200
> From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Cc: Madhu <enometh <at> meer.net>,  74132 <at> debbugs.gnu.org
> Date: Sat, 09 Nov 2024 10:52:06 -0500
> 
> > What will this do to URLs such as
> >
> >   http://web.archive.org/web/20240221082647/https://www.imdb.com/
> 
> Depends where point is: if it's after the `https`, then you get the
> "sub-URL" and if it's on or before the `https` then you get the whole URL.

Exactly.  This could be considered a bug, because the actual URL is
the entire thing.

> > I'm not sure I see how to resolve this dilemma.  Stefan, any ideas?
> 
> "url at point" is inherently heuristic, so I'm not too worried.
> But I do very much agree with Stefan that we need tests, because it's
> all too easy to run around in circles otherwise, fixing the heuristic to
> handle case A but breaking case B along the way.

I'm okay with adding tests, of course, but I'm not sure which of the
two behaviors leave you "not too worried": the current or the new one
after the proposed change.  And why.




This bug report was last modified 117 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.