bug#21389: closed (bogus diagnostic about unused lexical variable in eshell/time)

[GNU bug Tracking System]

Your message dated Tue, 1 Sep 2020 18:08:24 -0700
with message-id 
<CADwFkmmpR4zw=FGNXio780+UDRenKWStL0CR_J+a256NoqGaPA@mail.gmail.com>
and subject line Re: bug#21389: bogus diagnostic about unused lexical variable 
in eshell/time
has caused the debbugs.gnu.org bug report #21389,
regarding bogus diagnostic about unused lexical variable in eshell/time
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
21389: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=21389
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> bogus diagnostic about unused lexical variable in eshell/time
>
>
>
>
> Date: 
>
> Mon, 31 Aug 2015 14:36:33 -0700
>
>
>
>
> User-agent: 
>
> Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0
>
>
>
>
> To reproduce the problem in a fresh build from Emacs master, do this:
>
> touch lisp/eshell/em-unix.el
> make
>
> The output will include the following lines:
>
> eshell/em-unix.el:933:1:Warning: Unused lexical variable ‘args’
> eshell/em-unix.el:963:1:Warning: Unused lexical argument ‘args’
>
> The first diagnostic is bogus.  The second one is correct.  Perhaps the 
> byte compiler is getting confused by the second one and the confusion is 
> affecting the first one.
>
> The first diagnostic is generated for code that starts off like this:
>
> (defun eshell/time (&rest args)
>   "Implementation of \"time\" in Lisp."
>   (let ((time-args (copy-alist args))
>      ...
>
> so the lexical variable ‘args’ is being used.
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#21389: bogus diagnostic about unused lexical variable in eshell/time
>
>
>
>
> Date: 
>
> Tue, 1 Sep 2020 18:08:24 -0700
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)
>
>
>
>
> Paul Eggert <eggert@cs.ucla.edu> writes:
>
> > To reproduce the problem in a fresh build from Emacs master, do this:
> >
> > touch lisp/eshell/em-unix.el
> > make
> >
> > The output will include the following lines:
> >
> > eshell/em-unix.el:933:1:Warning: Unused lexical variable ‘args’
> > eshell/em-unix.el:963:1:Warning: Unused lexical argument ‘args’
> >
> > The first diagnostic is bogus.  The second one is correct.  Perhaps the byte
> > compiler is getting confused by the second one and the confusion is affecting
> > the first one.
> >
> > The first diagnostic is generated for code that starts off like this:
> >
> > (defun eshell/time (&rest args)
> >   "Implementation of \"time\" in Lisp."
> >   (let ((time-args (copy-alist args))
> >      ...
> >
> > so the lexical variable ‘args’ is being used.
>
> This seems to have been fixed on current master, so I'm closing this bug
> report.
>
>
> --- End Message ---