Hello, Jonathan! (Resending for I accidentally didn't copy the lists) On Tue, Mar 07, 2023 at 10:13:44AM -0700, Jonathan Corbet wrote: > Bru Moreira-Guedes <codeagain@xxxxxxxxxxxxx> writes: > > > It might be misleading, especially for some autistic readers for whom > > English is not the first language and whose tendency to interpret things > > more literally might be present (i.e., my very own self). Perhaps I clearly made a poor description of the problem this patch solves. I'll address it in details when I respond to your points. I could certainly improve it for a v2. > > > > The present patch clarifies the tabs text on such aspects. > > > > Signed-off-by: Bru Moreira-Guedes <codeagain@xxxxxxxxxxxxx> > > --- > > Documentation/process/coding-style.rst | 14 ++++++++------ > > 1 file changed, 8 insertions(+), 6 deletions(-) > > > > diff --git a/Documentation/process/coding-style.rst b/Documentation/process/coding-style.rst > > index 007e49ef6cec..9a0205fa7ed3 100644 > > --- a/Documentation/process/coding-style.rst > > +++ b/Documentation/process/coding-style.rst > > @@ -18,23 +18,25 @@ Anyway, here goes: > > 1) Indentation > > -------------- > > > > -Tabs are 8 characters, and thus indentations are also 8 characters. > > -There are heretic movements that try to make indentations 4 (or even 2!) > > -characters deep, and that is akin to trying to define the value of PI to > > -be 3. > > +The Linux Kernel uses the TAB character for indentation, as in the K&R > > +style. Tabs are meant to be viewed as 8-characters, and thus > > +indentations are meant to be seen as 8 characters as well. There are > > +heretic movements that try to use spaces for indentation, or make tabs > > +viewed as 4 (or even 2!) characters, and that is akin to trying to > > +define the value of PI to be 5. > > I guess I really don't see how this kind of tweaking helps things; this > text has stood for a couple of decades or so without evident > misunderstandings. Well, there was my own misunderstanding that motivated this patch :-} so, there is at least one evidenced case. Anyway, I think this is more of an improvement to make the text accessible for everyone, including autistic readers who might have a tendency towards literal interpretation. Just like me :D I don't actually think we would have a way to find out if everyone else who read this text throughout all those decades got it properly. Historically, the linux kernel hasn't been overall a much accessible project in a general sense. I think this kind of change is important if we want to move to the right direction. > ...and I'm totally mystified as to why we've redefined pi from 3 to 5? Ohhh, never mind this part, it shouldn't be here. At first I was undecided on whether to change it or not. I actually thought I had removed it. It just came to my mind that the value of pi is a constant, yet its representation is always bound to a certain precision level. Since the text didn't use decimal places, assuming no decimal precision the pi is actually near 3. Since the text appears to be trying to show an example of something "absurd", I think another value could be more appropriate. But yea, this change would not affect anyone and I think it shouldn't be done. > > > > -Now, some people will claim that having 8-character indentations makes > > +Now, some people will claim that viewint 8-character tab indentations makes > > Adding typos doesn't really help either. Sorry, I clearly had a poor review of the changed text (and I failed to setup spell-checking on checkpatch.pl). Yet Bagas did an excellent job in the other thread pointing me out all the details I missed. I'll be doing a v2 soon! Best, Bru Moreira-Guedes they/them
Attachment:
signature.asc
Description: PGP signature
