Messages in this thread | | | Subject | Re:Re: [PATCH] [v2] docs/zh CN: add translations in zh CN/dev-tools/kasan | From | Jiabing Wan <> | Date | Thu, 3 Jun 2021 12:50:12 +0800 (GMT+08:00) |
| |
> 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. >> ---
| |