linux-kernel - Re: [patch] console

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <fd0b3fea-4b40-ffba-442f-00908a5335a9@gmail.com>
Date:   Sun, 20 Mar 2022 19:15:02 +0100
From:   "Alejandro Colomar (man-pages)" <alx.manpages@...il.com>
To:     "G. Branden Robinson" <g.branden.robinson@...il.com>
Cc:     nick black <dankamongmen@...il.com>,
        Michael Kerrisk <mtk.manpages@...il.com>,
        linux-kernel@...r.kernel.org, linux-man@...r.kernel.org
Subject: Re: [patch] console_codes.4: ffix

Hi, nick, and Branden!

On 3/20/22 17:02, G. Branden Robinson wrote:
> Hi Nick,
> 
> Alex has been waiting on me for a review of this patch.  I took longer
> than I should have because I thought he was expecting a technical
> evaluation of the accuracy of the sequences documented.[1]  Now I see it
> was just a matter of man(7) and tbl(1) syntactical and style review.

No, you were originally right.  I did mean both, but especially
"a technical evaluation of the accuracy of the sequences documented".
You did it anyway, so thanks! :)

> 
> Easy bits first.
> 
>> Remove CSI prefix from the list of non-CSI escapes.
> 
> +1
> 
>> End all items of said list with periods, matching other sections of
>> the page.
> 
> +1
> 
>> Fix up the busted OSC command list (reset palette and set palette).
> 
>>  ESC ]  OSC     T{
>> -(Should be: Operating system command)
>> -ESC ] P \fInrrggbb\fP: set palette, with parameter
>> -given in 7 hexadecimal digits after the final P :-(.
>> -Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates
>> +Operating System Command prefix.
>> +T}
>> +ESC ] R                Reset palette.
>> +ESC ] P                T{
>> +Set palette, with parameter given in 7 hexadecimal digits \fInrrggbb\fP after
>> +the final P. Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates
>>  the red/green/blue values (0\(en255).
>> -ESC ] R: reset palette
>>  T}
> 
> Yes, this was majorly hosed up, syntactically.  What you have is good
> repair to obvious damage.  +1.
> 
> I have some suggestions for further improvement, but please don't gate
> the patch application on these.
[...]

Okay, thanks!

nick, can you please resend the patch?  I've lost the original email.

Cheers,

Alex

-- 
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/