From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH 00/10] Tweaks to follow guidelines in style guide
Date: Thu, 5 Oct 2017 13:48:31 -0700 [thread overview]
Message-ID: <20171005204831.GX3521@linux.vnet.ibm.com> (raw)
In-Reply-To: <d36d262c-2fe4-284c-7818-516a4ad59ec7@gmail.com>
On Fri, Oct 06, 2017 at 12:47:21AM +0900, Akira Yokosawa wrote:
> >From 2890e0069882321553c16aac213d4bb8d0a06fb7 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Wed, 5 Oct 2017 22:57:46 +0900
> Subject: [PATCH 00/10] Tweaks to follow guidelines in style guide
>
> Hi Paul,
>
> This patch set consists of minor tweaks in regard to the suggestions
> having been presented in style guide for a while.
>
> Patches #1 -- #5 are trivial changes.
>
> Patch #6 attempts to improve consistency in denoting POWER series CPU
> by defining a macro "\Power{}".
>
> Patch #7 substitutes "GCC" for "gcc". There are a few exceptions as
> mentioned in commit log.
>
> Patch #8 substitutes "IRQ" for "irq" in the same way. You might like
> to skip this one, as I see "irq" more often than "IRQ" in Linux
> documentations.
>
> Patch #9 is somewhat invasive. It switches Times font to that of
> "newtxtext" and "newtxmath" packages. The reason of the change
> is to have access to both upright and slated glyphs of Greek letters.
> Recent versions of these font packages give better looking result,
> especially in math mode. As noted in the commit log, newtxmath in
> TeX Live 2013/Debian has a few issues which have been fixed in later
> versions. It also switches font choice for the experimental target "1csf".
>
> Patch #10 updates style guide to reflect the changes made in this
> patch set.
They look fine, so I applied them, thank you! We might want to go with
"irq", but let's see how people react. Easy to change, aside from
beginnings of sentences!
Thanx, Paul
> Thanks, Akira
> --
> Akira Yokosawa (10):
> debugging: Insert narrow space in front of percent symbol
> debugging: Use upright font for Euler's number
> future/QC: Insert narrow space in front of percent symbol
> future/QC: Use non-breakable hyphen for axis names
> treewide: Insert narrow space in front of percent symbol
> treewide: Use \Power{} macro for POWER CPU family
> treewide: Call GNU C compiler as "GCC"
> treewide: Use "IRQ" instead of "irq" used as abbreviation
> future/QC: Use upright glyph for math constant and descriptive suffix
> styleguide: Reflect recent style improvements
>
> FAQ-BUILD.txt | 4 +-
> Makefile | 2 +-
> SMPdesign/SMPdesign.tex | 2 +-
> SMPdesign/beyond.tex | 14 ++---
> advsync/advsync.tex | 2 +-
> appendix/styleguide/styleguide.tex | 64 ++++++++---------------
> appendix/toyrcu/toyrcu.tex | 26 +++++-----
> count/count.tex | 28 +++++-----
> cpu/hwfreelunch.tex | 4 +-
> datastruct/datastruct.tex | 2 +-
> debugging/debugging.tex | 104 ++++++++++++++++++-------------------
> defer/rcuapi.tex | 6 +--
> defer/rcuusage.tex | 4 +-
> formal/dyntickrcu.tex | 52 +++++++++----------
> formal/formal.tex | 2 +-
> formal/spinhint.tex | 2 +-
> future/QC.tex | 92 ++++++++++++++++----------------
> future/htm.tex | 2 +-
> future/tm.tex | 4 +-
> intro/intro.tex | 8 +--
> memorder/memorder.tex | 32 ++++++------
> perfbook.tex | 20 +++++--
> rt/rt.tex | 22 ++++----
> toolsoftrade/toolsoftrade.tex | 24 ++++-----
> 24 files changed, 257 insertions(+), 265 deletions(-)
>
> --
> 2.7.4
>
>
prev parent reply other threads:[~2017-10-05 20:48 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-05 15:47 [PATCH 00/10] Tweaks to follow guidelines in style guide Akira Yokosawa
2017-10-05 15:48 ` [PATCH 01/10] debugging: Insert narrow space in front of percent symbol Akira Yokosawa
2017-10-05 15:49 ` [PATCH 02/10] debugging: Use upright font for Euler's number Akira Yokosawa
2017-10-05 15:51 ` [PATCH 03/10] future/QC: Insert narrow space in front of percent symbol Akira Yokosawa
2017-10-05 15:52 ` [PATCH 04/10] future/QC: Use non-breakable hyphen for axis names Akira Yokosawa
2017-10-05 15:53 ` [PATCH 05/10] treewide: Insert narrow space in front of percent symbol Akira Yokosawa
2017-10-05 15:54 ` [PATCH 06/10] treewide: Use \Power{} macro for POWER CPU family Akira Yokosawa
2017-10-05 15:55 ` [PATCH 07/10] treewide: Call GNU C compiler as "GCC" Akira Yokosawa
2017-10-05 15:56 ` [PATCH 08/10] treewide: Use "IRQ" instead of "irq" used as abbreviation Akira Yokosawa
2017-10-05 15:59 ` [PATCH 09/10] future/QC: Use upright glyph for math constant and descriptive suffix Akira Yokosawa
2017-10-05 16:00 ` [PATCH 10/10] styleguide: Reflect recent style improvements Akira Yokosawa
2017-10-05 20:48 ` Paul E. McKenney [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171005204831.GX3521@linux.vnet.ibm.com \
--to=paulmck@linux.vnet.ibm.com \
--cc=akiyks@gmail.com \
--cc=perfbook@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.