[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <9d324f50-e632-c020-04b9-57269ff7bdca@aol.com>
Date: Wed, 12 Dec 2018 01:05:13 +0800
From: Gao Xiang <hsiangkao@....com>
To: Joe Perches <joe@...ches.com>
Cc: Gao Xiang <gaoxiang25@...wei.com>,
Aaron Strahlberger <aaron.strahlberger@...teo.de>,
devel@...verdev.osuosl.org,
Julius Wiedmann <julius.wiedmann@....de>,
linux-kernel@...cs.fau.de,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
linux-kernel@...r.kernel.org, Dominik Huber <domi250@....de>,
linux-erofs@...ts.ozlabs.org
Subject: Re: [PATCH 4/5] drivers: staging: erofs: Fix parentheses error in
macro
Hi Joe,
On 2018/12/11 23:48, Joe Perches wrote:
> This macro is used only once and is merely obfuscation
> for that one use. Please remove the macro and expand
> it in the one place it is used.
I am also considering to replace it in a plain way,
it will be more straight-forward to describe its on-disk format.
Thanks for your suggestion.
Thanks,
Gao Xiang
Powered by blists - more mailing lists