GNU bug report logs - #24877
Feature request: let TeX-error-overview show warnings, but ignore them in TeX-next-error

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 24877 in the body.
You can then email your comments to 24877 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Clément Pit--Claudel <clement.pitclaudel <at> live.com>
To: bug-auctex <at> gnu.org
Subject: Feature request: let TeX-error-overview show warnings, but ignore
 them in TeX-next-error
Date: Fri, 4 Nov 2016 12:00:14 -0400

[Message part 1 (text/plain, inline)]

Hi all,

I just found about TeX-error-overview, but when I first tried to use it it just complained that there were "no errors or warning to show".  Reading through the implementation, I realized that it filters warnings using TeX-error-list-skip-warning-p, which itself excludes all warnings by default (maybe the message should be changed? It could say "no errors or warning to show; try customizing `…'"? Also, the error list has a menu that allows me to change what kind of errors are displayed but it's useless here: AucTeX errors out instead of showing an empty list when there's nothing to display)

I found this default surprising, until I realized that it's also used for TeX-next-error (and it makes a lot of sense to jump to errors first, instead of warnings).  So now I have my own wrapper that binds TeX-debug-warnings and TeX-debug-bad-boxes to t around a call to TeX-error-overview; that's not very user-friendly.

Maybe we need separate parameters for the error list and for the next-error function? I don't usually want to browse through warnings if there are errors, but I seldom want to see a list of only errors.  Another option would be to show all warnings in the error list when there are no errors.

In any case, I think it would make sense to have a different default for the errors and warnings list and for the next-error function.

Cheers and thanks!
Clément.

Emacs  : GNU Emacs 26.0.50.14 (x86_64-pc-linux-gnu, GTK+ Version 3.18.9)
 of 2016-10-29
Package: 11.89.6

current state:
==============
(setq
 AUCTeX-date "2016-10-14"
 window-system 'x
 LaTeX-version "2e"
 TeX-style-path '("~/.emacs.d/auctex" "/home/clement/.emacs.d/.cask/26.0/elpa/auctex-11.89.6/style"
                  "/home/clement/.emacs.d/auctex/auto" "/home/clement/.emacs.d/auctex/style" "auto" "style")
 TeX-auto-save t
 TeX-parse-self t
 TeX-master t
 TeX-command-list '(("Make" "make" TeX-run-compile nil t)
                    ("TeX" "%(PDF)%(tex) %(file-line-error) %(extraopts) %`%S%(PDFout)%(mode)%' %t" TeX-run-TeX nil
                     (plain-tex-mode ams-tex-mode texinfo-mode) :help "Run plain TeX")
                    ("LaTeX" "%`%l%(mode)%' %t" TeX-run-TeX nil (latex-mode doctex-mode) :help "Run LaTeX")
                    ("Makeinfo" "makeinfo %(extraopts) %t" TeX-run-compile nil (texinfo-mode) :help
                     "Run Makeinfo with Info output")
                    ("Makeinfo HTML" "makeinfo %(extraopts) --html %t" TeX-run-compile nil (texinfo-mode) :help
                     "Run Makeinfo with HTML output")
                    ("AmSTeX" "amstex %(PDFout) %(extraopts) %`%S%(mode)%' %t" TeX-run-TeX nil (ams-tex-mode) :help
                     "Run AMSTeX")
                    ("ConTeXt" "%(cntxcom) --once --texutil %(extraopts) %(execopts)%t" TeX-run-TeX nil
                     (context-mode) :help "Run ConTeXt once")
                    ("ConTeXt Full" "%(cntxcom) %(extraopts) %(execopts)%t" TeX-run-TeX nil (context-mode) :help
                     "Run ConTeXt until completion")
                    ("BibTeX" "bibtex %s" TeX-run-BibTeX nil t :help "Run BibTeX")
                    ("Biber" "biber %s" TeX-run-Biber nil t :help "Run Biber")
                    ("View" "%V" TeX-run-discard-or-function t t :help "Run Viewer")
                    ("Print" "%p" TeX-run-command t t :help "Print the file")
                    ("Queue" "%q" TeX-run-background nil t :help "View the printer queue" :visible TeX-queue-command)
                    ("File" "%(o?)dvips %d -o %f " TeX-run-dvips t t :help "Generate PostScript file")
                    ("Dvips" "%(o?)dvips %d -o %f " TeX-run-dvips nil t :help "Convert DVI file to PostScript")
                    ("Dvipdfmx" "dvipdfmx %d" TeX-run-dvipdfmx nil t :help "Convert DVI file to PDF with dvipdfmx")
                    ("Ps2pdf" "ps2pdf %f" TeX-run-ps2pdf nil t :help "Convert PostScript file to PDF")
                    ("Index" "makeindex %s" TeX-run-index nil t :help "Run makeindex to create index file")
                    ("Xindy" "texindy %s" TeX-run-command nil t :help "Run xindy to create index file")
                    ("Check" "lacheck %s" TeX-run-compile nil (latex-mode) :help "Check LaTeX file for correctness")
                    ("ChkTeX" "chktex -v6 %s" TeX-run-compile nil (latex-mode) :help
                     "Check LaTeX file for common mistakes")
                    ("Spell" "(TeX-ispell-document \"\")" TeX-run-function nil t :help "Spell-check the document")
                    ("Clean" "TeX-clean" TeX-run-function nil t :help "Delete generated intermediate files")
                    ("Clean All" "(TeX-clean t)" TeX-run-function nil t :help
                     "Delete generated intermediate and output files")
                    ("Other" "" TeX-run-command t t :help "Run an arbitrary command"))
 )

[signature.asc (application/pgp-signature, attachment)]

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

From: Mosè Giordano <mose <at> gnu.org>
To: Clément Pit--Claudel <clement.pitclaudel <at> live.com>
Cc: 24877 <at> debbugs.gnu.org
Subject: Re: bug#24877: Feature request: let TeX-error-overview show warnings, 
 but ignore them in TeX-next-error
Date: Mon, 7 Nov 2016 00:20:01 +0100

Hi Clément,

2016-11-04 17:00 GMT+01:00 Clément Pit--Claudel <clement.pitclaudel <at> live.com>:
> Hi all,
>
> I just found about TeX-error-overview, but when I first tried to use it it just complained that there were "no errors or warning to show".  Reading through the implementation, I realized that it filters warnings using TeX-error-list-skip-warning-p, which itself excludes all warnings by default (maybe the message should be changed? It could say "no errors or warning to show; try customizing `…'"?

That's a sensible suggestion, thanks.  Maybe we can conditionally add
that suggestion when there are warnings/bad boxes.  I can do this, but
not right now, probably tomorrow or in the next few days.  Feel free
to ping me here if I miss to do it this week.

> Also, the error list has a menu that allows me to change what kind of errors are displayed but it's useless here: AucTeX errors out instead of showing an empty list when there's nothing to display)

Sorry, I don't get what you mean here.

> I found this default surprising, until I realized that it's also used for TeX-next-error (and it makes a lot of sense to jump to errors first, instead of warnings).  So now I have my own wrapper that binds TeX-debug-warnings and TeX-debug-bad-boxes to t around a call to TeX-error-overview; that's not very user-friendly.
>
> Maybe we need separate parameters for the error list and for the next-error function? I don't usually want to browse through warnings if there are errors, but I seldom want to see a list of only errors.  Another option would be to show all warnings in the error list when there are no errors.
>
> In any case, I think it would make sense to have a different default for the errors and warnings list and for the next-error function.

I don't quite agree on this.  I think there are already many options
available in this regard, I fear that adding even more complexity
(different behavior for `TeX-next-error` and `TeX-erorr-overview`) is
going to confuse users, that tend no to read documentation at all (and
it seems you confirmed this suspect ;-).  I could agree on changing
the defaults of `TeX-debug-warnings' and `TeX-debug-bad-boxes' (I do
it in my init file), but I'm personally against duplicating options
for the two error reporting tools.  If instead you truly believe this is a
good idea, please start a discussion at auctex-devel <at> gnu.org so that
other people can participate, bug tracker is not a good place for this
stuff.

Bye,
Mosè

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

From: Clément Pit--Claudel <clement.pitclaudel <at> live.com>
To: Mosè Giordano <mose <at> gnu.org>
Cc: 24877 <at> debbugs.gnu.org
Subject: Re: bug#24877: Feature request: let TeX-error-overview show warnings, 
 but ignore them in TeX-next-error
Date: Sun, 6 Nov 2016 19:58:40 -0500

[Message part 1 (text/plain, inline)]

On 2016-11-06 18:20, Mosè Giordano wrote:
> Hi Clément,

Hi Mosè,

>> […] It could say "no errors or warning to show; try
>> customizing `…'"?
>
> That's a sensible suggestion, thanks.  Maybe we can conditionally
> add that suggestion when there are warnings/bad boxes.  I can do
> this, but not right now, probably tomorrow or in the next few days.
> Feel free to ping me here if I miss to do it this week.

Great, thanks!

>> Also, the error list has a menu that allows me to change what kind
>> of errors are displayed but it's useless here: AucTeX errors out
>> instead of showing an empty list when there's nothing to display)
> 
> Sorry, I don't get what you mean here.

I meant that AucTeX could show an empty list instead of raising an error when there's nothing to display.  Then I could use the menu bar in that window to enable display of warnings.

> I could agree on
> changing the defaults of `TeX-debug-warnings' and
> `TeX-debug-bad-boxes' (I do it in my init file),

Please don't :) It's convenient to be able to quickly jump to the next error.

> If instead you truly believe this is a good idea, please start a discussion at
> auctex-devel <at> gnu.org so that other people can participate, bug tracker is not
> a good place for this stuff.

Got it! I have limited bandwidth and advising TeX-errors-overview works for me, so I'll drop the issue :)

Clément.

[signature.asc (application/pgp-signature, attachment)]

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

From: Mosè Giordano <mose <at> gnu.org>
To: Clément Pit--Claudel <clement.pitclaudel <at> live.com>
Cc: 24877 <at> debbugs.gnu.org
Subject: Re: bug#24877: Feature request: let TeX-error-overview show warnings, 
 but ignore them in TeX-next-error
Date: Tue, 8 Nov 2016 00:18:01 +0100

[Message part 1 (text/plain, inline)]

Hi Clément,

2016-11-07 1:58 GMT+01:00 Clément Pit--Claudel <clement.pitclaudel <at> live.com>:
> On 2016-11-06 18:20, Mosè Giordano wrote:
>> Hi Clément,
>
> Hi Mosè,
>
>>> […] It could say "no errors or warning to show; try
>>> customizing `…'"?
>>
>> That's a sensible suggestion, thanks.  Maybe we can conditionally
>> add that suggestion when there are warnings/bad boxes.  I can do
>> this, but not right now, probably tomorrow or in the next few days.
>> Feel free to ping me here if I miss to do it this week.
>
> Great, thanks!

Before applying it, please have a look to the patch attached.  Do you
think it's ok?

Bye,
Mosè

[patch (application/octet-stream, attachment)]

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

From: Clément Pit--Claudel <clement.pitclaudel <at> live.com>
To: Mosè Giordano <mose <at> gnu.org>
Cc: 24877 <at> debbugs.gnu.org
Subject: Re: bug#24877: Feature request: let TeX-error-overview show warnings, 
 but ignore them in TeX-next-error
Date: Mon, 7 Nov 2016 19:40:42 -0500

[Message part 1 (text/plain, inline)]

On 2016-11-07 18:18, Mosè Giordano wrote:
> Before applying it, please have a look to the patch attached.  Do you
> think it's ok?

Looks perfect.  Any reason to use `if` over `when`?

Cheers,
Clément.

[signature.asc (application/pgp-signature, attachment)]

Message #22 received at 24877-done <at> debbugs.gnu.org (full text, mbox):

From: Mosè Giordano <mose <at> gnu.org>
To: Clément Pit--Claudel <clement.pitclaudel <at> live.com>
Cc: 24877-done <at> debbugs.gnu.org
Subject: Re: bug#24877: Feature request: let TeX-error-overview show warnings, 
 but ignore them in TeX-next-error
Date: Mon, 14 Nov 2016 11:29:43 +0100

Hi Clément,

2016-11-08 1:40 GMT+01:00 Clément Pit--Claudel <clement.pitclaudel <at> live.com>:
> On 2016-11-07 18:18, Mosè Giordano wrote:
>> Before applying it, please have a look to the patch attached.  Do you
>> think it's ok?
>
> Looks perfect.  Any reason to use `if` over `when`?

Yes: laziness ;-)  Patch installed, thank you for the suggestion!

Bye,
Mosè

Previous Next

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