[bug #59443] [PATCH] src/preproc/tbl/table.cpp: add "tbl:" to diagnostic messages

[G. Branden Robinson]

Follow-up Comment #1, bug #59443 (project groff):

I somewhat disagree with Bjarni's analysis.

tbl itself can issue diagnostics, and it can produce roff language constructs
that _may_ cause the production of diagnostics

Whether the latter category of diagnostics actually issues depends on
conditions at the time the document is formatted.

Therefore, I think the current arrangement, where in the document basically
"complains about itself", is more correct than the alternative Bjarni
proposes.

However, I do note that of the messages Bjarni proposes to patch, only one is
missing the word "table" in the text of the actual message part.  I think it
would be helpful to mention tables there, too, somewhat like the following.


     prints(".tmc \\n[.F]: around line \\n[.c]: warning:\n"
     prints(".tmc \\n[.F]: around line \\n[.c]: warning:\n"
-           ".tm1 \" column separation set to zero\n"
+           ".tm1 \" table column separation set to zero\n"
            ".nr " SEPARATION_FACTOR_REG " 0\n");


I'll work on this, as a welcome break from the 100+ man page fix-up commits I
have pending.

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?59443>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/