From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH 0/3] Employ auto-numbering in litmus tests
Date: Sat, 5 Aug 2017 08:50:24 -0700 [thread overview]
Message-ID: <20170805155024.GL3730@linux.vnet.ibm.com> (raw)
In-Reply-To: <a69ac710-6be5-0969-c092-8008a6dd764d@gmail.com>
On Sun, Aug 06, 2017 at 12:35:36AM +0900, Akira Yokosawa wrote:
> On 2017/08/06 0:09, Paul E. McKenney wrote:
> > On Sat, Aug 05, 2017 at 12:41:30PM +0900, Akira Yokosawa wrote:
> >> >From ba0af5ad755ab4e175b07abb71890077e4d54a20 Mon Sep 17 00:00:00 2001
> >> From: Akira Yokosawa <akiyks@gmail.com>
> >> Date: Sat, 5 Aug 2017 12:29:10 +0900
> >> Subject: [PATCH 0/3] Employ auto-numbering in litmus tests
> >>
> >> Hi Paul,
> >>
> >> As I mentioned earlier, we need to revert commits related to floatrow.
> >> Patch #1 does the revert.
> >> Patch #2 converts litmus tests in Memory Barriers section to use
> >> auto-numbering.
> >> Patch #3 updates style guide to reflect the removal of floatrow and
> >> a typo fix.
> >>
> >> I'm considering "subfig" package as an alternative to floatrow.
> >> It predates floatrow and has less capability.
> >> I'll see if it works in two-column layout, and update the style guide
> >> if it goes well.
> >
> > Life is like that sometimes. ;-)
> >
> > Applied, thank you! I did a couple of updates on 3/3 for "nor" and
> > for subject-verb agreement.
>
> Thanks!
> It is hard for me to choose "and" or "nor" in negative sentences...
> So, "such as x, y, and z" is right in the negative sentence. Hmm.
Well, what is there is OK. In more ambiguous cases, one approach is
to parenthesize the list, or to reverse the order: "these options,
which include x, y, and z, do not have capability c."
> Uh-oh, in the commit log, the line "Also fix a typo (and -> nor)." remains.
Good point, fixed.
> > Do we want to "git rm" floatrowpf.sty?
>
> I think it has been already removed by the reverting commit d03363ff3493,
> hasn't it?
My confusion, ran "git diff" backwards, apologies!
Thanx, Paul
> Thanks, Akira
>
>
> >
> > Thanx, Paul
> >
> >> Thanks, Akira
> >>
> >> Akira Yokosawa (3):
> >> Revert commits after "Localize floatrow.sty as floatrowpf.sty"
> >> advsync: Employ auto-numbering in litmus tests
> >> styleguide: Remove references to 'floatrow' and adjust context
> >>
> >> advsync/memorybarriers.tex | 502 ++++++------
> >> appendix/styleguide/hello.c | 9 -
> >> appendix/styleguide/styleguide.tex | 205 +----
> >> defer/rcuusage.tex | 6 +-
> >> floatrowpf.sty | 1482 ------------------------------------
> >> howto/howto.tex | 4 +-
> >> perfbook.tex | 4 +-
> >> 7 files changed, 280 insertions(+), 1932 deletions(-)
> >> delete mode 100644 appendix/styleguide/hello.c
> >> delete mode 100644 floatrowpf.sty
> >>
> >> --
> >> 2.7.4
> >>
> >>
> >
> >
>
prev parent reply other threads:[~2017-08-05 15:50 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-28 9:53 [PATCH 00/10] Add style guide Akira Yokosawa
2017-07-28 9:55 ` [PATCH 01/10] appendix: " Akira Yokosawa
2017-07-28 9:57 ` [PATCH 02/10] Specify required revision of 'verbatimbox' Akira Yokosawa
2017-07-28 9:58 ` [PATCH 03/10] Localize floatrow.sty Akira Yokosawa
2017-07-31 3:26 ` Paul E. McKenney
2017-07-31 4:05 ` Akira Yokosawa
2017-07-31 5:01 ` Akira Yokosawa
2017-07-31 12:45 ` Paul E. McKenney
2017-07-28 10:00 ` [PATCH 04/10] Apply workaround to floatrow.sty and rename it floatrowpf.sty Akira Yokosawa
2017-07-28 10:01 ` [PATCH 05/10] Define 'listing' environment for style guide Akira Yokosawa
2017-07-28 10:02 ` [PATCH 06/10] styleguide: Add listing environment examples Akira Yokosawa
2017-07-28 10:03 ` [PATCH 07/10] Disable 'floatrow' layout in manually aligned code snippets Akira Yokosawa
2017-07-28 10:04 ` [PATCH 08/10] styleguide: Add example of grouping " Akira Yokosawa
2017-07-28 10:05 ` [PATCH 09/10] styleguide: Add example of preferred table layout using 'booktabs' Akira Yokosawa
2017-07-28 10:06 ` [PATCH 10/10] styleguide: Tweak layout of 'Limitation' table Akira Yokosawa
2017-07-31 14:43 ` [PATCH v2 0/8] Add style guide Akira Yokosawa
2017-07-31 14:44 ` [PATCH v2 1/8] Localize floatrow.sty as floatrowpf.sty Akira Yokosawa
2017-07-31 14:46 ` [PATCH v2 2/8] Apply workaround to floatrowpf.sty Akira Yokosawa
2017-07-31 14:47 ` [PATCH v2 3/8] Define 'listing' environment for style guide Akira Yokosawa
2017-07-31 14:49 ` [PATCH v2 4/8] styleguide: Add listing environment examples Akira Yokosawa
2017-07-31 14:50 ` [PATCH v2 5/8] Disable 'floatrow' layout in manually aligned code snippets Akira Yokosawa
2017-07-31 14:51 ` [PATCH v2 6/8] styleguide: Add example of grouping " Akira Yokosawa
2017-07-31 14:52 ` [PATCH v2 7/8] styleguide: Add example of preferred table layout using 'booktabs' Akira Yokosawa
2017-07-31 14:52 ` [PATCH v2 8/8] styleguide: Tweak layout of 'Limitation' table Akira Yokosawa
2017-07-31 15:24 ` [PATCH v2 0/8] Add style guide Akira Yokosawa
2017-07-31 15:55 ` Paul E. McKenney
2017-07-31 15:57 ` Paul E. McKenney
2017-08-01 20:40 ` Paul E. McKenney
2017-08-02 14:57 ` Akira Yokosawa
2017-08-02 16:29 ` Paul E. McKenney
2017-08-02 22:22 ` Akira Yokosawa
2017-08-02 22:48 ` Paul E. McKenney
2017-08-05 2:09 ` Akira Yokosawa
2017-08-05 3:41 ` [PATCH 0/3] Employ auto-numbering in litmus tests Akira Yokosawa
2017-08-05 3:43 ` [PATCH 1/3] Revert commits after "Localize floatrow.sty as floatrowpf.sty" Akira Yokosawa
2017-08-05 3:44 ` [PATCH 2/3] advsync: Employ auto-numbering in litmus tests Akira Yokosawa
2017-08-05 3:46 ` [PATCH 3/3] styleguide: Remove references to 'floatrow' and adjust context Akira Yokosawa
2017-08-05 15:09 ` [PATCH 0/3] Employ auto-numbering in litmus tests Paul E. McKenney
2017-08-05 15:35 ` Akira Yokosawa
2017-08-05 15:50 ` 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=20170805155024.GL3730@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