grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 5/9] btrfs: move logging code in grub_btrfs_read_logical()


From: Daniel Kiper
Subject: Re: [PATCH 5/9] btrfs: move logging code in grub_btrfs_read_logical()
Date: Wed, 30 May 2018 12:55:15 +0200
User-agent: Mutt/1.3.28i

On Wed, May 16, 2018 at 08:48:15PM +0200, Goffredo Baroncelli wrote:
> A portion of the logging code is moved outside a for(;;). The part that is

s/outside a for/outside of for/

> left inside is the one which depends by the for(;;) index.

s/depends by the for/depends on the for/

By the way, there are 2 for(;;). You have to be clear which one.

> This is a preparatory patch: in the next one it will be possible to refactor
> the code inside the for(;;) in an another function.

I think that you can drop these sentences. However, it seems to me that you
are no giving the reason why this move is needed for subsequent patches.
Please explain that in the commit message.

And missing SOB.

The code itself LGTM.

Daniel



reply via email to

[Prev in Thread] Current Thread [Next in Thread]