Hi Günther, Jared, On Sun, Feb 23, 2025 at 03:13:35PM -0800, Jared Finder wrote: > I'm not a regular kernel developer, so feel free to ignore my > recommendations if they are not aligned with the level of documentation > usually provided. > > On 2025-02-23 13:36, Günther Noack wrote: > > * Indent the documented selection modes into tagged paragraphs. > > * Document constants from the header file (tiocl.h) instead of numbers. > > > > Signed-off-by: Günther Noack <gnoack3000@xxxxxxxxx> I have no comments of my own. What do you want to do about the ones from Jared? Have a lovely day! Alex > > --- > > man/man2const/TIOCLINUX.2const | 23 ++++++++++++++++++----- > > 1 file changed, 18 insertions(+), 5 deletions(-) > > > > diff --git a/man/man2const/TIOCLINUX.2const > > b/man/man2const/TIOCLINUX.2const > > index c0acdd0ea..5db3f6ea6 100644 > > --- a/man/man2const/TIOCLINUX.2const > > +++ b/man/man2const/TIOCLINUX.2const > > @@ -65,11 +65,24 @@ are the ending > > column and row. > > (Upper left corner is row=column=1.) > > .I sel_mode > > -is 0 for character-by-character selection, > > -1 for word-by-word selection, > > -or 2 for line-by-line selection. > > -The indicated screen characters are highlighted and saved > > -in a kernel buffer. > > +may be one of the following operations: > > +.RS > > +.TP > > +.B TIOCL_SELCHAR > > +Select character-by-character. > > +The indicated screen characters are highlighted > > +and saved in a kernel buffer. > > I would recommend also saying > > > +.TP > > +.B TIOCL_SELWORD > > +Select word-by-word. > > +The indicated screen characters are highlighted > > +and saved in a kernel buffer. > > I would recommend also saying something like "The start is expanded backward > and the end is expanded forwards to select entire words." > > > +.TP > > +.B TIOCL_SELLINE > > +Select line-by-line. > > +The indicated screen characters are highlighted > > +and saved in a kernel buffer. > > +.RE > > Similarly, mention how the start / end is expanded here too. > > > .IP > > Since Linux 6.7, using this subcode requires the > > .B CAP_SYS_ADMIN > > -- MJF -- <https://www.alejandro-colomar.es/>
Attachment:
signature.asc
Description: PGP signature
