bug-ncurses
[Top][All Lists]
Advanced

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

Re: [PATCH 4/9] man/clear.1: Migrate macro usage conventions.


From: Thomas Dickey
Subject: Re: [PATCH 4/9] man/clear.1: Migrate macro usage conventions.
Date: Sun, 1 Oct 2023 10:32:25 -0400

On Sun, Oct 01, 2023 at 11:34:47AM +0200, Ingo Schwarze wrote:
> Hi Branden,
> 
> note that i'm not planning to subscribe to bug-ncurses@, so please keep
> me in Cc: if you desire feedback from my side.
> 
> G. Branden Robinson wrote on Sat, Sep 30, 2023 at 04:06:24PM -0500:
> 
> > Argh.  I didn't test with mandoc, and I guess I should have.  It's just
> > refusing to accept valid *roff syntax here.  I wonder if I can talk Ingo
> > into supporting it.
> > 
> > How about it, Ingo?  Will you support people defining macros inside
> > conditional block brace scopes?  ;-)
> 
> In general, it appears that mandoc supports that, see the example
> at the end of this message - but you are aware that both defining
> macros and using conditionals in manual page source code is strongly
> discouraged, right?  Let alone combining both features...
> 
> So if people do that, they should be aware that they are ruining the
> robustness and likely the portability of their pages, and that fixing
> any bugs or adding any missing features related to such stunts is
> likely to be high effort and low priority while i often have bugs
> open that are high priority and low effort.  So such bugs, if any,
> are unlikely to be addressed in any reasonable time frame and more
> likely to rot for years before getting addressed.
> 
> That said, i tried to have a look what is *actually* going on,
> but so far without success.  I believe the patch we are talking
> about is
> 
>   https://lists.gnu.org/archive/html/bug-ncurses/2023-09/msg00088.html
> 
> I tried to apply that patch to both

That's not the first patch which Branden made to that file.
Applying each patch, I had no rejects.

See attached

-- 
Thomas E. Dickey <dickey@invisible-island.net>
https://invisible-island.net

Attachment: clear.1
Description: Text document

Attachment: mandoc.out
Description: Text document

Attachment: signature.asc
Description: PGP signature


reply via email to

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