Discussions of the Parallel Programming book
 help / color / mirror / Atom feed
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
> 
> 


      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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox