Re: [PATCH 0/3] Patches for recent updates

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Dec 23, 2016 at 05:07:17PM +0900, Akira Yokosawa wrote:
> >From cbb7797d6b307850a280248589bc09b8852e8a68 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@xxxxxxxxx>
> Date: Fri, 23 Dec 2016 16:46:06 +0900
> Subject: [PATCH 0/3] Patches for recent updates
> 
> Hi Paul,
> 
> This short patch set consists of fixes to recent updates in
> advsync/memorybarriers.
> 
> I'm not sure what is your preference of quotation of short variable
> names and keywords. In the 1st patch, I chose the form of ~~\co{a}''.
> If you prefer other one, please let me know.
> 
> Ya, I know a variety of styles are mixed-used in perfbook,
> reflecting various original publications...

My current rule is that an identifier can be mistaken for an English
word or letter of the alphabet, then it should be in double quotation
marks and under \co{}, consistent with your changes.  However, something
like "rcu_node" cannot reasonably be mistaken for English, so it only
needs \co{}.

Thoughts?

> But at least within a (sub)section, it would be better to keep
> consistency.

Applied and pushed, thank you!

							Thanx, Paul

>                                                 Thanks, Akira
> -- 
> Akira Yokosawa (3):
>   advsync/memorybarriers: Use consistent quotation marks
>   advsync/memorybarriers: Fix typo (READ_ONCE -> WRITE_ONCE)
>   advsync/memorybarriers: Fix trivial typo
> 
>  advsync/memorybarriers.tex | 30 +++++++++++++++---------------
>  1 file changed, 15 insertions(+), 15 deletions(-)
> 
> -- 
> 2.7.4
> 

--
To unsubscribe from this list: send the line "unsubscribe perfbook" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux