Messages in this thread | | | Date | Tue, 26 Apr 2022 16:39:01 +0800 | Subject | Re: [PATCH 2/2] doc/vm/page_owner.rst: Fix table display confusion | From | Alex Shi <> |
| |
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@meizu.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.
| |