GNU bug report logs - #72329
Avoid recursion in gnus-message-citation-mode

Previous Next

Full log

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: eric <at> ericabrahamsen.net
Cc: 72329 <at> debbugs.gnu.org, thuna.cing <at> gmail.com
Subject: Re: bug#72329: Avoid recursion in gnus-message-citation-mode
Date: Sat, 14 Sep 2024 10:33:07 +0300

Ping! Ping! Ping! Eric, are you there?

> Cc: thuna.cing <at> gmail.com, 72329 <at> debbugs.gnu.org
> Date: Sat, 31 Aug 2024 10:56:58 +0300
> From: Eli Zaretskii <eliz <at> gnu.org>
> 
> Ping! Ping!  Eric, any comments?
> 
> > Cc: 72329 <at> debbugs.gnu.org, thuna.cing <at> gmail.com
> > Date: Sat, 17 Aug 2024 10:29:31 +0300
> > From: Eli Zaretskii <eliz <at> gnu.org>
> > 
> > Ping! Eric, any comments?
> > 
> > > Cc: 72329 <at> debbugs.gnu.org
> > > Date: Fri, 02 Aug 2024 10:27:54 +0300
> > > From: Eli Zaretskii <eliz <at> gnu.org>
> > > 
> > > > From: Thuna <thuna.cing <at> gmail.com>
> > > > Date: Sun, 28 Jul 2024 03:30:40 +0200
> > > > 
> > > > In `gnus-message-citation-mode' the
> > > > 
> > > >   (if (not font-lock-mode)
> > > >       (gnus-message-citation-mode (font-lock-mode 1))
> > > >     ...)
> > > > 
> > > > can lead to excessive nesting if font-lock-mode fails to be set (quick
> > > > enough or altogether).  Changing this with
> > > > 
> > > >   (while (not font-lock-mode) (font-lock-mode 1))
> > > >   ...
> > > > 
> > > > should be better, assuming it is necessary for `font-lock-mode' to be
> > > > called repeatedly, otherwise
> > > > 
> > > >   (font-lock-mode 1)
> > > >   (while (not font-lock-mode))
> > > >   ...
> > > > 
> > > > might also work, or simply
> > > > 
> > > >   (font-lock-mode 1)
> > > >   ...
> > > > 
> > > > if it is not necessary to wait.
> > > > 
> > > > One possible problem with these arrangements is that it may be desirable
> > > > to error in case there is an issue which prevents font-lock-mode from
> > > > being set perpetually, in which case something like
> > > > 
> > > >   (font-lock-mode 1)
> > > >   (with-timeout (<however-long-before-giving-up>
> > > >                  (error "Failed to enable `font-lock-mode'."))
> > > >     (while (not font-lock-mode)))
> > > >   ...
> > > > 
> > > > might be preferable.
> > > 
> > > Thanks.
> > > 
> > > Eric, any comments or suggestions?
> > > 
> > > 
> > > 
> > > 
> > 
> > 
> > 
> > 
> 
> 
> 
> 

Previous Next

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