[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANiq72kghG4i35Jt2cY0-2CJkzKe-jEfSyU2pJFFqKgfkV7CZg@mail.gmail.com>
Date: Wed, 8 Feb 2023 17:56:43 +0100
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: Boqun Feng <boqun.feng@...il.com>
Cc: linux-kernel@...r.kernel.org, rust-for-linux@...r.kernel.org,
Miguel Ojeda <ojeda@...nel.org>,
Alex Gaynor <alex.gaynor@...il.com>,
Wedson Almeida Filho <wedsonaf@...il.com>,
Gary Guo <gary@...yguo.net>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>,
Vincenzo Palazzo <vincenzopalazzodev@...il.com>,
Will Deacon <will@...nel.org>,
Peter Zijlstra <peterz@...radead.org>,
Mark Rutland <mark.rutland@....com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Sergio González Collado <sergio.collado@...il.com>,
Finn Behrens <fin@...ntec.com>
Subject: Re: [PATCH v2 2/2] sample: rust: print: Add sampe code for Arc printing
On Wed, Feb 8, 2023 at 5:34 PM Boqun Feng <boqun.feng@...il.com> wrote:
>
> Hmm.. I'm OK with this change, but it's not a typo ;-)
By typo I meant the "sampe", not the other changes -- sorry, I should
have been more clear.
> 1) capitalize the first letter after subsystem tags in the title
> since that's kinda the rule for a few subsystems I usually work
> on, I don't have my own preference, just something I'm used to
> ;-)
Yeah, I don't mind one way or the other (in fact, personally I prefer
uppercase slightly because it is a bit easier to "scan" visually to
see where it starts after the prefixes). The lowercase one is just the
one we have used so far for Rust (which I picked looking at what Linus
et al. usually do).
> 2) avoid using "`" in the title to save space because title space
> is precious.
I see, makes sense, thanks!
Cheers,
Miguel
Powered by blists - more mailing lists