Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Date:   Sun, 5 Nov 2023 15:27:00 +0100
From:   Willy Tarreau <w@1wt.eu>
To:     Thomas =?iso-8859-1?Q?Wei=DFschuh?= <linux@weissschuh.net>
Cc:     Shuah Khan <shuah@kernel.org>, linux-kselftest@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] selftests/nolibc: fix testcase status alignment
Message-ID: <20231105142659.GA21217@1wt.eu>
References: <20231105-nolibc-align-v1-1-5bfc542cb6ee@weissschuh.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20231105-nolibc-align-v1-1-5bfc542cb6ee@weissschuh.net>
User-Agent: Mutt/1.10.1 (2018-07-13)
Precedence: bulk

On Sun, Nov 05, 2023 at 03:22:30PM +0100, Thomas Wei?schuh wrote:
> Center-align all possible status reports.
> Before OK and FAIL were center-aligned in relation to each other but
> SKIPPED and FAILED would be left-aligned.
> 
> Before:
> 
> 7 environ_addr = <0x7fffef3e7c50>                                [OK]
> 8 environ_envp = <0x7fffef3e7c58>                               [FAIL]
> 9 environ_auxv                                                  [SKIPPED]
> 10 environ_total                                                [SKIPPED]
> 11 environ_HOME = <0x7fffef3e99bd>                               [OK]
> 12 auxv_addr                                                    [SKIPPED]
> 13 auxv_AT_UID = 1000                                            [OK]
> 
> After:
> 
> 7 environ_addr = <0x7ffff13b00a0>                                 [OK]
> 8 environ_envp = <0x7ffff13b00a8>                                [FAIL]
> 9 environ_auxv                                                  [SKIPPED]
> 10 environ_total                                                [SKIPPED]
> 11 environ_HOME = <0x7ffff13b19bd>                                [OK]
> 12 auxv_addr                                                    [SKIPPED]
> 13 auxv_AT_UID = 1000                                             [OK]
> 
> Signed-off-by: Thomas Wei?schuh <linux@weissschuh.net>

Quite frankly for trivial cleanups like this you should not even bother
with sending a review, and could queue them directly!

Thanks,
Willy