Messages in this thread | | | From | Miguel Ojeda <> | Date | Wed, 8 Feb 2023 17:56:43 +0100 | 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@gmail.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
| |