GNU bug report logs - #64232
28.2; Incorrect(?) byte compilation warning about ‘ansi-color-apply-on-region’

Previous Next

Full log

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: michael_heerdegen <at> web.de, zack <at> owlfolio.org, monnier <at> iro.umontreal.ca
Cc: 64232 <at> debbugs.gnu.org
Subject: Re: bug#64232: 28.2; Incorrect(?) byte compilation warning about
 ‘ansi-color-apply-on-region’
Date: Thu, 06 Jul 2023 09:43:44 +0300

Ping!  Any reasons this was not yet installed on master?

> Cc: zack <at> owlfolio.org, 64232 <at> debbugs.gnu.org, monnier <at> iro.umontreal.ca
> Date: Fri, 30 Jun 2023 09:00:33 +0300
> From: Eli Zaretskii <eliz <at> gnu.org>
> 
> > From: Michael Heerdegen <michael_heerdegen <at> web.de>
> > Cc: 64232 <at> debbugs.gnu.org,  eliz <at> gnu.org,  zack <at> owlfolio.org
> > Date: Fri, 30 Jun 2023 06:02:36 +0200
> > 
> > Stefan Monnier <monnier <at> iro.umontreal.ca> writes:
> > 
> > > Someone™ should sit down and figure out how to make `bytecomp.el` keep
> > > track of those things more reliably/predictably.
> > > In the mean time, maybe we can simply tweak `byte-compile-eval` so it
> > > never adds to `byte-compile-noruntime-functions` functions which have
> > > already been seen/declared during the current compilation?
> > >
> > >
> > > diff --git a/lisp/emacs-lisp/bytecomp.el b/lisp/emacs-lisp/bytecomp.el
> > > index 99202185d8d..cfec4b3be06 100644
> > > --- a/lisp/emacs-lisp/bytecomp.el
> > > +++ b/lisp/emacs-lisp/bytecomp.el
> > > @@ -1128,7 +1128,8 @@ byte-compile-eval
> > >  		     ;; we arguably should add it to b-c-noruntime-functions,
> > >                       ;; but it's not clear it's worth the trouble
> > >  		     ;; trying to recognize that case.
> > > -		     (unless (get f 'function-history)
> > > +		     (unless (or (get f 'function-history)
> > > +                                 (assq f byte-compile-function-environment))
> > >                         (push f byte-compile-noruntime-functions)))))))))))))
> > 
> > Seems appropriate (and fixes this bug).
> > 
> > Actually with that patch the situation looks already quite ok'ish to me.
> 
> Please install this on master, and thanks.
> 
> 
> 
> 

Previous Next

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