[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug #57592] [PATCH] tmac/<files containing a ".nx" request>: Add a warn
G. Branden Robinson
[bug #57592] [PATCH] tmac/<files containing a ".nx" request>: Add a warning
Thu, 16 Jan 2020 01:06:41 -0500 (EST)
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0 PureBrowser/60.9.0
Follow-up Comment #4, bug #57592 (project groff):
To be fair to Bjarni, he wasn't requesting a change in the semantics of `.nx`,
but asking us to add include guards to some of the macro packages.
But as Ingo pointed out, the macro files should be written idempotently.
Having them skip to the next file of input when a symbol (string, macro,
diversion, request) that the package defines is already defined is a pretty
conventional way to do this and I do not think requires a warning.
I regret my lack of clarity in characterizing my objection earlier. In fact I
think our "include guards" could be improved by keying of a simple that is
_only_ there for the purpose of detecting multiple inclusion.
But better still would be to simply have "reentrant" macro packages. :P
Reply to this item at:
Message sent via Savannah