emacs-devel
[Top][All Lists]
Advanced

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

Re: native-comp-async-report-warnings-errors default value


From: Eli Zaretskii
Subject: Re: native-comp-async-report-warnings-errors default value
Date: Tue, 07 Dec 2021 19:16:16 +0200

> From: Robert Pluim <rpluim@gmail.com>
> Cc: akrl@sdf.org,  larsi@gnus.org,  stefankangas@gmail.com,
>   emacs-devel@gnu.org
> Date: Tue, 07 Dec 2021 17:54:49 +0100
> 
> Let the wordsmithing begin!

Thanks, this is fine, with two gotchas:

> +is available.  Note that because of this use of a subprocess, native
> +compilation may produce warning and errors which byte-compilation does
> +not, and lisp code may thus need to be modified to work correctly. See
                                                                    ^^
Two spaces there, please.

> +@code{native-comp-async-report-warnings-errors} in @xref{Native-Compilation

@xref is inappropriate in the middle of a sentence, since it produces
a capitalized "See".  You want @pxref instead.

> Regarding discoverability, the
> *Warnings* buffer has entries like this:
> 
>     Warning (comp): tex-site.el:168:31: Warning: the function
>     ‘BibTeX-auto-store’ is not known to be defined. Disable showing
>     Disable logging
> 
> I guess we could add a link to
> `native-comp-async-report-warnings-errors' when the warning type == (comp)

We could, yes.  Assuming the code to add that is simple enough to
install on the release branch.

Thanks.



reply via email to

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