lkml.org 
[lkml]   [2022]   [Apr]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 2/2] doc/vm/page_owner.rst: Fix table display confusion
From


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.

\
 
 \ /
  Last update: 2022-04-26 11:09    [W:0.102 / U:0.312 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site