Re:Re: [PATCH v2,RESEND] doc: zh_CN: fix style problems for io_ordering.txt

[çæè]

From: Jonathan Corbet <corbet@xxxxxxx>
Date: 2020-03-21 07:24:26
To:  Wang Wenhu <wenhu.wang@xxxxxxxx>
Cc:  Harry Wei <harryxiyou@xxxxxxxxx>,Alex Shi <alex.shi@xxxxxxxxxxxxxxxxx>,linux-doc@xxxxxxxxxxxxxxx,linux-kernel@xxxxxxxxxxxxxxx,kernel@xxxxxxxx
Subject: Re: [PATCH v2,RESEND] doc: zh_CN: fix style problems for io_ordering.txt>On Sat, 14 Mar 2020 23:08:55 -0700
>Wang Wenhu <wenhu.wang@xxxxxxxx> wrote:
>
>> Problems exist in the Chinese translation of io_ordering.txt.
>> Partly for the difference between Chinese and English character
>> encoding format, and the others are of the failure to comply
>> with the ReST markups.
>
>So I feel like I'm missing something here...
Some style modification, such as "::" field added within a newline indicating
start of a block. Because of the difference between encoding of Chinese and
English, it did not work well for Chinese to just put it after some Chinese
characters within the same line.

The file is currently a txt file(none rst), so it's OK just ignore the modifactions.
>
>> Signed-off-by: Wang Wenhu <wenhu.wang@xxxxxxxx>
>> ---
>> v2: resend for the failure of delivering.
>> 
>>  .../translations/zh_CN/io_ordering.txt        | 72 ++++++++++++-------
>>  1 file changed, 46 insertions(+), 26 deletions(-)
>> 
>> diff --git a/Documentation/translations/zh_CN/io_ordering.txt b/Documentation/translations/zh_CN/io_ordering.txt
>> index 1f8127bdd415..080ed2911db0 100644
>> --- a/Documentation/translations/zh_CN/io_ordering.txt
>> +++ b/Documentation/translations/zh_CN/io_ordering.txt
>> @@ -29,39 +29,59 @@ Documentation/io_ordering.txt çäæçè
>>  èäåääèåéçåæäåååéçåæääååèèåïèéåçääåå
>>  åéæäïmb()ïäèäéçäI/Oïã
>>  
>> +A more concrete example from a hypothetical device driver::
>> +
>> +		...
>> +	CPU A:  spin_lock_irqsave(&dev_lock, flags)
>> +	CPU A:  val = readl(my_status);
>> +	CPU A:  ...
>> +	CPU A:  writel(newval, ring_ptr);
>> +	CPU A:  spin_unlock_irqrestore(&dev_lock, flags)
>> +		...
>> +	CPU B:  spin_lock_irqsave(&dev_lock, flags)
>> +	CPU B:  val = readl(my_status);
>> +	CPU B:  ...
>> +	CPU B:  writel(newval2, ring_ptr);
>> +	CPU B:  spin_unlock_irqrestore(&dev_lock, flags)
>> +		...
>> +
>> +
>>  åèääèåéåççåääåï
>> +::
>>  
>> +		...
>> +	CPU A:  spin_lock_irqsave(&dev_lock, flags)
>> +	CPU A:  val = readl(my_status);
>> +	CPU A:  ...
>> +	CPU A:  writel(newval, ring_ptr);
>> +	CPU A:  spin_unlock_irqrestore(&dev_lock, flags)
>> +		...
>> +	CPU B:  spin_lock_irqsave(&dev_lock, flags)
>> +	CPU B:  val = readl(my_status);
>> +	CPU B:  ...
>> +	CPU B:  writel(newval2, ring_ptr);
>> +	CPU B:  spin_unlock_irqrestore(&dev_lock, flags)
>
>It sure looks like you're adding the same text twice here...?
As I mentioned below the sign-off area, delivery failure happened so
this is just a resent version.
>
>Thanks,
>
>jon
>