GNU bug report logs - #39980
[PATCH] gnus-shorten-url: Improve and avoid args-out-of-range error

Previous Next

Full log

Message #32 received at 39980 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Štěpán Němec <stepnem <at> gmail.com>
Cc: larsi <at> gnus.org, 39980 <at> debbugs.gnu.org
Subject: Re: bug#39980: [PATCH] gnus-shorten-url: Improve and avoid
 args-out-of-range error
Date: Sun, 12 Apr 2020 13:33:37 +0300

> From: Štěpán Němec <stepnem <at> gmail.com>
> Cc: 39980 <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>
> Date: Sun, 12 Apr 2020 11:53:47 +0200
> 
> On Thu, 02 Apr 2020 13:01:35 +0200
> Lars Ingebrigtsen wrote:
> 
> > Štěpán Němec <stepnem <at> gmail.com> writes:
> >
> >> Considering the opinions expressed here and in
> >> https://lists.gnu.org/archive/html/emacs-devel/2020-03/msg00541.html I
> >> have reverted to defun and autoload the function instead to prevent
> >> having to load subr-x at runtime whenever gnus-sum is loaded, as it is
> >> currently only used there by `gnus-summary-browse-url'.
> >>
> >> Revised patch attached.
> >
> > Looks good to me.
> 
> This can go to emacs-27, right? `gnus-summary-browse-url' is new in
> Emacs 27, it would seem a shame to ship it with this bug.

OK for the change in gnus-summary-browse-url, but let's please leave
out of emacs-27 changes that aren't necessarily needed to fix the
original problem; cleanups are for master.  On emacs-27, let's just
fix the original problem locally.

Thanks.

Previous Next

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