GNU bug report logs - #63288
30.0.50; Emacs 30 packages fail to build with native comp on some machines

Previous Next

Full log

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

From: Thierry Volpiatto <thievol <at> posteo.net>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: Pip Cet <pipcet <at> protonmail.com>, Stefan Kangas <stefankangas <at> gmail.com>,
 63288 <at> debbugs.gnu.org, Thierry Volpiatto <thievol <at> posteo.net>,
 Eli Zaretskii <eliz <at> gnu.org>, acorallo <at> gnu.org
Subject: Re: bug#63288: [PATCH] Mark cl-proclaim, cl-declaim, and cl-declare
 as obsolete
Date: Thu, 20 Feb 2025 14:38:14 +0000

[Message part 1 (text/plain, inline)]

Stefan Monnier <monnier <at> iro.umontreal.ca> writes:

>>>> (defmacro my-with-progress (&rest body)
>>>>   `(let ((reporter (make-progress-reporter "Updating...")))
>>>>      (progn
>>>>        ,@body)))
>>>>
>>>> (defun my-updating ()
>>>>   (cl-declare (special reporter))
>>>>   (my-with-progress
>>>>    (cl-loop for elm in '(a b c d e f g)
>>>>             do (progn
>>>>                  (sit-for 1)
>>>>                  (progress-reporter-update reporter))
>>>>             collect elm)))
>>>
>>> I don't see what is the use of (cl-declare (special reporter)) here.
>>> What do you think it changes to the behavior of this code?
>>
>> It declare reporter as a variable and prevent byte compile warnings,
>
> Really?  AFAICT the code should work just fine, and without warnings, if
> you just remove the `cl-declare` and leave the variable be lexically scoped.

Hu yes indeed, I haven't tried this beeing sure it would produce
warnings but it doesn't!

Thanks.


-- 
Thierry

[signature.asc (application/pgp-signature, inline)]

Previous Next

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