Re: [PATCH man-pages] man/man2const: document the new F_SETDELEG and F_GETDELEG constants

["G. Branden Robinson" <g.branden.robinson@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1310 bytes --]

Hi Jeff and Alex,

At 2026-01-13T09:45:11-0500, Jeff Layton wrote:
> On Tue, 2026-01-13 at 15:13 +0100, Alejandro Colomar wrote:
> > On Mon, Jan 12, 2026 at 01:47:11PM -0500, Jeff Layton wrote:
[...]
> > > +.br
> > 
> > This .br seems superfluous.
> > 
> 
> You would think, no? But when I remove it, man seems to stick both
> lines togther. I really do not grok groff at all.

Try the introduction in groff_man_style(7).

https://man7.org/linux/man-pages/man7/groff_man_style.7.html

> I'm happy to accept other suggestions on how to fix that though.

Here's the hunk with context:

+.EE
+.P
+.BI "int fcntl(int " fd ", F_SETDELEG, struct delegation *" deleg );
+.br
+.BI "int fcntl(int " fd ", F_GETDELEG, struct delegation *" deleg );
+.fi

The `br` request really is necessary there, as Jeff says.  This is an
ordinary paragraph.  The `EE` (end example) restores filling.  So you
_do_ need the `br` request here _but_ you can discard the `fi` request
to enable filling afterward, because it's already on.

An alternative will become available in groff 1.24.0; you'll be able to
use groff man's syopsis macros to set C function protoypes.

https://cgit.git.savannah.gnu.org/cgit/groff.git/tree/NEWS?id=cfcfe1dde5b1752484dcd94dde1d65c65a5540fb#n409

Regards,
Branden

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]