[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <043f8d6c-ecae-bbb9-922a-93422d9a21c7@gmail.com>
Date: Tue, 26 Apr 2022 16:39:01 +0800
From: Alex Shi <seakeel@...il.com>
To: Akira Yokosawa <akiyks@...il.com>, baihaowen@...zu.com
Cc: alexs@...nel.org, corbet@....net, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, siyanteng01@...il.com,
siyanteng@...ngson.cn
Subject: Re: [PATCH 2/2] doc/vm/page_owner.rst: Fix table display confusion
On 4/24/22 11:38, Akira Yokosawa wrote:
> This patch looks like against akpm's tree (or linux-next).
> So it is not applicable to docs-next of Jonathan's tree at the
> moment.
>
> You need to wait until docs-next catches up akpm's tree.
>
> Patch 1/2 should wait until the English doc is updated,
> I suppose.
>
> By the way, I don't think the list of options need tables.
> Literal blocks should suffice.
>
> I mean, How about the simpler changes:
>
>> For --sort option:
> For --sort option::
>
>> For --curl option:
> For --curl option::
>
> Thoughts?
Hi haowen,
Don't ignore review suggestions, Akira is right here, then
you don't repeat the same issue many times:
$ g am \[PATCH\ 2_2\]\ doc_vm_page_owner.rst\:\ Fix\ table\ display\ confusion\ -\ Haowen\ Bai\ \<baihaowen@...zu.com\>\ -\ 2022-04-26\ 1507.eml
Applying: doc/vm/page_owner.rst: Fix table display confusion
.git/rebase-apply/patch:21: indent with spaces.
==== ========== ============================================
.git/rebase-apply/patch:22: indent with spaces.
KEY LONG DESCRIPTION
.git/rebase-apply/patch:23: indent with spaces.
==== ========== ============================================
.git/rebase-apply/patch:24: indent with spaces.
p pid process ID
.git/rebase-apply/patch:25: indent with spaces.
tg tgid thread group ID
warning: squelched 15 whitespace errors
warning: 20 lines add whitespace errors.
Powered by blists - more mailing lists