On 18.08.20 09:19, Stefano Garzarella wrote: > Hi Yi Li, > thanks for this patch! Just a comment below: > > On Mon, Aug 17, 2020 at 07:01:13PM +0800, Yi Li wrote: >> Signed-off-by: Yi Li <y...@winhong.com> >> --- >> qemu-img.c | 12 ++++++------ >> 1 file changed, 6 insertions(+), 6 deletions(-) >> >> diff --git a/qemu-img.c b/qemu-img.c >> index 5308773811..a0fbc2757c 100644 >> --- a/qemu-img.c >> +++ b/qemu-img.c >> @@ -1181,7 +1181,7 @@ static int64_t find_nonzero(const uint8_t *buf, >> int64_t n) >> } >> >> /* >> - * Returns true iff the first sector pointed to by 'buf' contains at least >> + * Returns true if the first sector pointed to by 'buf' contains at least > > This change seems unrelated, please can you put this in a separate patch?
It’s also just not a correct change, as “iff” means “if and only if”. Max
signature.asc
Description: OpenPGP digital signature
