2022-03-23 12:19:11

by Johannes Thumshirn

[permalink] [raw]
Subject: Re: [PATCH] fs: btrfs: Remove redundant condition

On 23/03/2022 02:47, Haowen Bai wrote:
> The logic !A || A && B is equivalent to !A || B. so we can
> make code clear.
>
> Signed-off-by: Haowen Bai <[email protected]>
> ---

Well I guess as this is the 3rd time someone wants to do this:
Reviewed-by: Johannes Thumshirn <[email protected]>


2022-03-25 19:56:08

by David Sterba

[permalink] [raw]
Subject: Re: [PATCH] fs: btrfs: Remove redundant condition

On Wed, Mar 23, 2022 at 07:54:39AM +0000, Johannes Thumshirn wrote:
> On 23/03/2022 02:47, Haowen Bai wrote:
> > The logic !A || A && B is equivalent to !A || B. so we can
> > make code clear.
> >
> > Signed-off-by: Haowen Bai <[email protected]>
> > ---
>
> Well I guess as this is the 3rd time someone wants to do this:
> Reviewed-by: Johannes Thumshirn <[email protected]>

Ok then, added to misc-next.