Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <043f8d6c-ecae-bbb9-922a-93422d9a21c7@gmail.com>
Date:   Tue, 26 Apr 2022 16:39:01 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.7.0
Subject: Re: [PATCH 2/2] doc/vm/page_owner.rst: Fix table display confusion
Content-Language: en-US
To:     Akira Yokosawa <akiyks@gmail.com>, baihaowen@meizu.com
Cc:     alexs@kernel.org, corbet@lwn.net, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, siyanteng01@gmail.com,
        siyanteng@loongson.cn
References: <1650424016-7225-3-git-send-email-baihaowen@meizu.com>
 <3ce4feb9-5477-e7fb-e98c-3d6342ddc7b5@gmail.com>
From:   Alex Shi <seakeel@gmail.com>
In-Reply-To: <3ce4feb9-5477-e7fb-e98c-3d6342ddc7b5@gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk


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.