Hi Al,
Al Haji-Ali <[email protected]> writes:
> (I hope it is okay to send this patch while my two other patches are
> still in review, since it modifies a different file).
Yes, of course.
> The attached patch bundles a few small, related improvements to the
> TeX/LaTeX compilation sentinels and TeX-error-overview-mode (I also
> modified ConTeXt's sentinel for consistency).
I didn't have the time to install and test your proposal locally, but
some comments:
> - TeX-error-overview-open-after-TeX-run' can have the value errors', in
> which case TeX-TeX-sentinel, TeX-LaTeX-sentinel and
> TeX-ConTeXt-sentinel will only call TeX-error-overview if there are
> actual errors, rather than for warnings or bad boxes alone.
>
> - To make sure that the user always gets a summary of warnings and
> bad-boxes (even when TeX-error-overview is not shown), the "success"
> message shown by TeX-LaTeX-sentinel is now one of:
>
> XXX: successfully formatted {29} pages A -- B
>
> or
>
> XXX: faced problems formatting {29} pages A -- B
>
> where 'A' is something like " (with 4 warnings and 5 bad boxes)" and
> 'B' is something like "run Biber" or "unresolved citations" (both
> optional, shown only when relevant). TeX-TeX-sentinel's message is
> just "XXX: formatted N pages A".
>
> The number of warnings and bad boxes in 'A' is only shown when
> `TeX-debug-warnings'/`TeX-debug-bad-boxes' are non-nil, respectively.
I wonder if that's too verbose for the echo area. Personally, I prefer
to see single line messages in the echo area. WDYT?
> - TeX-error-overview-mode's File and Type columns are now sortable.
Sounds cool.
> I have been using these changes for some time now, and I find it much
> more pleasant to deal with TeX-error-overview window, so I hope at least
> some of these suggestions can be incorporated in AUCTeX.
I hope too, thanks for sharing.
> From 3127d265379480822767e1a4c13857f9e56c3389 Mon Sep 17 00:00:00 2001
> From: Al Haji-Ali <[email protected]>
> Date: Fri, 24 Jul 2026 11:35:25 +0100
> Subject: [PATCH] Change TeX/LaTeX compilation message and error overview
> [...]
>
> (defcustom TeX-error-overview-open-after-TeX-run nil
> - "Whether to open automatically the error overview after running TeX."
> + "Whether to open automatically the error overview after running TeX.
> +If it has the value \\='errors, then only open the overview if
> +there are errors (rather than warnings)."
> :group 'TeX-output
> - :type 'boolean)
> + :type (or 'boolean 'symbol))
What do you think if we change this to:
(defcustom TeX-error-overview-open-after-TeX-run nil
"Whether to open automatically the error overview after running TeX.
If it has the value \\='errors, then only open the overview if
there are errors (rather than warnings)."
:group 'TeX-output
:type '(choice (const :tag "Don't open error/warning overview" nil)
(const :tag "Open error/warning overview" nil)
(const :tag "Open overview only with errors" errors)))
I think it is easier for the users to deal with. And I'm not sure if
your (or ...) works with `defcustom', but I'm not an expert in that
area.
Best, Arash
_______________________________________________
bug-auctex mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/bug-auctex
