emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: A lot of warnings building Emacs master


From: Andrea Corallo
Subject: Re: A lot of warnings building Emacs master
Date: Sun, 27 Oct 2024 04:17:20 -0400
User-agent: Gnus/5.13 (Gnus v5.13)

Andy Moreton <andrewjmoreton@gmail.com> writes:

> On Sun 27 Oct 2024, Angelo Graziosi wrote:
>
>> Building last master (commit 523aade3ea11c188e30e3889f031d1848129cf82) I get
>> about 10615 (grep -c ...) "Warning" like this:
>>
>> org-src.el: Warning: `if-let' is an obsolete macro (as of 31.1); use 
>> `if-let*'
>> instead.
>
> I see this also. The change to mark this as obsolete is premature.
>
> Marking as obsolete to avoid new uses being introduced should only be
> done after reworking the existing  codebase  to remove the majority of
> existing calls.
>
> The current situation of leaving hundreds of lines of warning noise in
> the build output makes it harder to see other problems, and is not good.
>
>     AndyM

Agree, we should strive for pushing only warning free commits.

I see Sean is putting effort in fixing this, but I often even see
commits introducing new warnings in the hope that other contributors fix
them in the future, this is not okay.

That said I don't understand as well why was necessary to obsolete
if-let in favor of if-let*, but bug#73853 has some reference to previous
discussions (which I had no time to go through) and the change was
agreed by a maintainer.

  Andrea



reply via email to

[Prev in Thread] Current Thread [Next in Thread]