lkml.org 
[lkml]   [2021]   [Jun]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe:Re: [PATCH] [v2] docs/zh CN: add translations in zh CN/dev-tools/kasan
From
Date

> Re: [PATCH] [v2] docs/zh_CN: add translations in zh_CN/dev-tools/kasan>>Hi Jiabing,
>
>Thanks for translation! The contents are generally fine.
>I tried the patch with scripts/checkpatch.pl found there are lots errors

Hi,Alex,

Oh, that is strange.

In my ubuntu server, I ran scripts/checkpatch.pl and got:

When Documentation/translations/zh_CN/dev-tools/kasan.rst
"total: 0 errors, 0 warnings, 431 lines checked"

When 0001-v2-docs-zh_CN-add-translations-in-zh_CN-dev-tools-ka.patch.
"WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#41:
new file mode 100644
total: 0 errors, 1 warnings, 445 lines checked"

No errors and just one warning.
So I think there was a problem with your platform or downloading?

If there were no problem, I would check my email client again ;)

>---
>...
>ERROR: DOS line endings
>#552: FILE: Documentation/translations/zh_CN/dev-tools/kasan.rst:430:
>+^M$
>
>ERROR: trailing whitespace
>#553: FILE: Documentation/translations/zh_CN/dev-tools/kasan.rst:431:
>+.. _KUnit: https://www.kernel.org/doc/html/latest/dev-tools/kunit/index.html^M$
>
>total: 432 errors, 1 warnings, 445 lines checked
>---
>
>and the lines in the patch are not aligned on 80 chars, (40 chars for Chinese,
>since one Chinese char could take 2 positions. )
>Thanks
>Alex

OK, great tips for me.
I'll fix them in v3.

Thanks,
Jiabing

>
>On 2021/6/2 下午8:56, Wan Jiabing wrote:
>> Add new zh translations
>> * zh_CN/dev-tools/kasan.rst
>> and link it to zh_CN/dev-tools/index.rst
>>
>> Signed-off-by: Wan Jiabing <wanjiabing@vivo.com>
>> ---
>> Changelog:
>> v2:
>> - Delete spaces surround with English words.
>> ---

\
 
 \ /
  Last update: 2021-06-03 06:51    [W:0.202 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site