GNU bug report logs - #68812
30.0.50; vhdl-speedbar-insert-hierarchy in progmodes/vhdl-mode.el

Previous Next

Full log

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

From: Reto Zimmermann <reto <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>, rms <at> gnu.org
Cc: 68812 <at> debbugs.gnu.org
Subject: Re: bug#68812: 30.0.50; vhdl-speedbar-insert-hierarchy in
 progmodes/vhdl-mode.el
Date: Tue, 30 Jan 2024 14:20:05 +0100

On 2024-01-30 13:58, Eli Zaretskii wrote:
>> From: Richard Stallman <rms <at> gnu.org>
>> Date: Mon, 29 Jan 2024 22:56:20 -0500
>>
>>
>> Byte compilation gave me this warning from lisp/progmodes/vhdl-mode.el:
>>
>> Compiling file /home/rms/emacs-git/copy-jan-1/lisp/progmodes/vhdl-mode.el at Mon Jan 29 08:29:07 2024
>>
>> In vhdl-speedbar-insert-hierarchy:
>> vhdl-mode.el:14981:41: Warning: Lexical argument shadows the dynamic variable
>>      package-alist
> 
> AFAIU, this only happens if you compile with package.el loaded.
> Normal byte compilation ("touch vhdl-mode.el && make") doesn't emit
> this warning
> 
>> The latest master has the same problem.
>>
>> I don't know that code, so I can't tell whether renaming the argument
>> would fix the problem.
> 
> I think it can be safely renamed.  Reto, do you agree?

Yes, package-alist can be renamed to pack-alist.

Reto

Previous Next

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