[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block: Clean up some bad code in the vvfat driv
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] block: Clean up some bad code in the vvfat driver |
Date: |
Wed, 13 Sep 2017 08:57:27 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 09/13/2017 05:21 AM, Thomas Huth wrote:
> Remove the unnecessary home-grown redefinition of the assert() macro here,
> and remove the unusable debug code at the end of the checkpoint() function.
> The code there uses assert() with side-effects (assignment to the "mapping"
> variable), which should be avoided. Looking more closely, it seems as it is
> apparently also only usable for one certain directory layout (with a file
> named USB.H in it) and thus is of no use for the rest of the world.
>
> Signed-off-by: Thomas Huth <address@hidden>
> ---
> block/vvfat.c | 26 ++------------------------
> 1 file changed, 2 insertions(+), 24 deletions(-)
>
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
Re: [Qemu-devel] [PATCH] block: Clean up some bad code in the vvfat driver, Kevin Wolf, 2017/09/19