mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Weijie Yuan <wy@wyuan.org>
To: Chen-Yu Yeh <chenyou910331@gmail.com>
Cc: Jonathan Corbet <corbet@lwn.net>, Alex Shi <alexs@kernel.org>,
	Dongliang Mu <dzm91@hust.edu.cn>,
	Yanteng Si <si.yanteng@linux.dev>,
	Hu Haowen <2023002089@link.tyut.edu.cn>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 01/16] docs/zh_TW: process: localize terminology in 7.AdvancedTopics.rst
Date: Wed, 22 Jul 2026 16:48:08 +0800	[thread overview]
Message-ID: <amCDyEKqNzkjgu-H@wyuan.org> (raw)
In-Reply-To: <20260721215542.98435-2-chenyou910331@gmail.com>

On Wed, Jul 22, 2026 at 05:55:27AM +0800, Chen-Yu Yeh wrote:
>  審閱補丁
>  --------
>  
> -一些讀者顯然會反對將本節與“高級主題”放在一起,因爲即使是剛開始的內核開發人員
> -也應該審閱補丁。當然,沒有比查看其他人發佈的代碼更好的方法來學習如何在內核環境
> -中編程了。此外,審閱者永遠供不應求;通過審閱代碼,您可以對整個流程做出重大貢獻。
> +一些讀者顯然會反對將本節與“進階主題”放在一起,因為即使是剛開始的核心開發人員
> +也應該審閱補丁。當然,沒有比查看其他人發布的程式碼更好的方法來學習如何在核心環境
> +中撰寫程式了。此外,審閱者永遠供不應求;透過審閱程式碼,您可以對整個流程做出重大貢獻。
>  
> -審查代碼可能是一副令人生畏的圖景,特別是對一個新的內核開發人員來說,他們
> -可能會對公開詢問代碼感到緊張,而這些代碼是由那些有更多經驗的人發佈的。不過,
> -即使是最有經驗的開發人員編寫的代碼也可以得到改進。也許對(所有)審閱者最好
> +審查程式碼可能是一副令人生畏的圖景,特別是對一個新的核心開發人員來說,他們
> +可能會對公開詢問程式碼感到緊張,而這些程式碼是由那些有更多經驗的人發布的。不過,
> +即使是最有經驗的開發人員編寫的程式碼也可以得到改進。也許對(所有)審閱者最好
>  的建議是:把審閱評論當成問題而不是批評。詢問“在這條路徑中如何釋放鎖?”
> -總是比說“這裏的鎖是錯誤的”更好。
> +總是比說“這裡的鎖是錯誤的”更好。
>  
> -不同的開發人員將從不同的角度審查代碼。部分人會主要關注代碼風格以及代碼行是
> -否有尾隨空格。其他人會主要關注補丁作爲一個整體實現的變更是否對內核有好處。
> -同時也有人會檢查是否存在鎖問題、堆棧使用過度、可能的安全問題、在其他地方
> -發現的代碼重複、足夠的文檔、對性能的不利影響、用戶空間ABI更改等。所有類型
> -的檢查,只要它們能引導更好的代碼進入內核,都是受歡迎和值得的。
> +不同的開發人員將從不同的角度審查程式碼。部分人會主要關注程式碼風格以及程式碼行是
> +否有尾隨空格。其他人會主要關注補丁作為一個整體實作的變更是否對核心有好處。
> +同時也有人會檢查是否存在鎖問題、堆疊使用過度、可能的安全問題、在其他地方
> +發現的程式碼重複、足夠的文件、對效能的不利影響、使用者空間ABI更改等。所有類型
> +的檢查,只要它們能引導更好的程式碼進入核心,都是受歡迎和值得的。

Should we wrap lines earlier throughout the series? After the certain
terminology was changed, many lines became longer but were not reflowed.

Just a thought. I'am also fine with current changes.

Thanks.

See:

  https://docs.kernel.org/translations/zh_CN/how-to.html#id9
  https://docs.kernel.org/translations/zh_CN/how-to.html#id10

  parent reply	other threads:[~2026-07-22  8:48 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-21 21:55 [PATCH 00/16] docs/zh_TW: localize terminology and sync process/ documents Chen-Yu Yeh
2026-07-21 21:55 ` [PATCH 01/16] docs/zh_TW: process: localize terminology in 7.AdvancedTopics.rst Chen-Yu Yeh
2026-07-22  5:56   ` Weijie Yuan
2026-07-22  8:48   ` Weijie Yuan [this message]
2026-07-22 13:32     ` 葉宸佑
2026-07-21 21:55 ` [PATCH 02/16] docs/zh_TW: process: localize terminology in 1.Intro.rst Chen-Yu Yeh
2026-07-22  7:42   ` Weijie Yuan
2026-07-21 21:55 ` [PATCH 03/16] docs/zh_TW: process: localize terminology in code-of-conduct-interpretation.rst Chen-Yu Yeh
2026-07-21 21:55 ` [PATCH 04/16] docs/zh_TW: process: localize terminology in license-rules.rst Chen-Yu Yeh
2026-07-22 11:35   ` Weijie Yuan
2026-07-21 21:55 ` [PATCH 05/16] docs/zh_TW: process: localize terminology in email-clients.rst Chen-Yu Yeh
2026-07-22 11:35   ` Weijie Yuan
2026-07-24  6:48     ` 葉宸佑
2026-07-21 21:55 ` [PATCH 06/16] docs/zh_TW: process: localize terminology in programming-language.rst Chen-Yu Yeh
2026-07-22 14:51   ` Weijie Yuan
2026-07-21 21:55 ` [PATCH 07/16] docs/zh_TW: process: localize terminology in coding-style.rst Chen-Yu Yeh
2026-07-22 14:48   ` Weijie Yuan
2026-07-24  6:38     ` 葉宸佑
2026-07-21 21:55 ` [PATCH 08/16] docs/zh_TW: process: localize terminology in stable-kernel-rules.rst Chen-Yu Yeh
2026-07-21 21:55 ` [PATCH 09/16] docs/zh_TW: process: localize terminology in 5.Posting.rst Chen-Yu Yeh
2026-07-23  7:02   ` Weijie Yuan
2026-07-24  6:40     ` 葉宸佑
2026-07-24 14:11       ` Weijie Yuan
2026-07-25  0:46         ` 葉宸佑
2026-07-21 21:55 ` [PATCH 10/16] docs/zh_TW: process: localize terminology in 2.Process.rst Chen-Yu Yeh
2026-07-23  6:30   ` Weijie Yuan
2026-07-24  6:41     ` 葉宸佑
2026-07-24 13:57       ` Weijie Yuan
2026-07-26  8:56     ` Weijie Yuan
2026-07-21 21:55 ` [PATCH 11/16] docs/zh_TW: process: localize terminology in howto.rst Chen-Yu Yeh
2026-07-23 10:53   ` Weijie Yuan
2026-07-24  6:41     ` 葉宸佑
2026-07-21 21:55 ` [PATCH 12/16] docs/zh_TW: process: localize terminology in embargoed-hardware-issues.rst Chen-Yu Yeh
2026-07-23 12:43   ` Weijie Yuan
2026-07-24  6:42     ` 葉宸佑
2026-07-21 21:55 ` [PATCH 13/16] docs/zh_TW: process: localize terminology in submitting-patches.rst Chen-Yu Yeh
2026-07-24 11:06   ` Weijie Yuan
2026-07-25  0:36     ` 葉宸佑
2026-08-05  7:00       ` Weijie Yuan
2026-07-21 21:55 ` [PATCH 14/16] docs/zh_TW: process: localize terminology in 8.Conclusion.rst Chen-Yu Yeh
2026-07-24 12:08   ` Weijie Yuan
2026-07-25  0:43     ` 葉宸佑
2026-07-21 21:55 ` [PATCH 15/16] docs/zh_TW: process: localize terminology in index.rst Chen-Yu Yeh
2026-07-24 12:27   ` Weijie Yuan
2026-07-25  0:44     ` 葉宸佑
2026-07-21 21:55 ` [PATCH 16/16] docs/zh_TW: Add a glossary for Traditional Chinese translations Chen-Yu Yeh
2026-07-24 12:56   ` Weijie Yuan
2026-07-25  0:41     ` 葉宸佑
2026-07-22 15:09 ` [PATCH 00/16] docs/zh_TW: localize terminology and sync process/ documents Weijie Yuan
2026-07-24  6:36   ` 葉宸佑

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=amCDyEKqNzkjgu-H@wyuan.org \
    --to=wy@wyuan.org \
    --cc=2023002089@link.tyut.edu.cn \
    --cc=alexs@kernel.org \
    --cc=chenyou910331@gmail.com \
    --cc=corbet@lwn.net \
    --cc=dzm91@hust.edu.cn \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=si.yanteng@linux.dev \
    /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

all inboxes | Powered by JetHome®