[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#68812: 30.0.50; vhdl-speedbar-insert-hierarchy in progmodes/vhdl-mod
From: |
Eli Zaretskii |
Subject: |
bug#68812: 30.0.50; vhdl-speedbar-insert-hierarchy in progmodes/vhdl-mode.el |
Date: |
Tue, 30 Jan 2024 16:18:04 +0200 |
> Date: Tue, 30 Jan 2024 14:20:05 +0100
> Cc: 68812@debbugs.gnu.org
> From: Reto Zimmermann <reto@gnu.org>
>
> On 2024-01-30 13:58, Eli Zaretskii wrote:
> >> From: Richard Stallman <rms@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.
Thanks, done on the master branch. I renamed to pkg-alist, since
pack-alist is already used elsewhere in vhdl-mode.el, and caused the
same warning.
With that, I close this bug.