|
From: | Philippe Mathieu-Daudé |
Subject: | Re: [Qemu-devel] [PATCH] qemu/units: Move out QCow2 specific definitions |
Date: | Fri, 2 Nov 2018 13:37:36 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 |
Hi Kevin, On 2/11/18 12:07, Kevin Wolf wrote:
Am 02.11.2018 um 09:58 hat Philippe Mathieu-Daudé geschrieben:This definitions are QCow2 specific, there is no need to expose them in the global namespace. This partially reverts commit 540b8492618eb. Signed-off-by: Philippe Mathieu-Daudé <address@hidden>If we don't want this globally, I think we also don't want it in qcow2.
I only see this definitions used by block/qcow2.h (b6a95c6d1007).Per 540b8492618eb description "This is needed when a size has to be stringified" but I can't find other code requiring these definitions in the codebase.
Or at least reduce it to only those constants that qcow2 actually uses.
Fine by me, I'll let Leonid opine first. Regards, Phil.
[Prev in Thread] | Current Thread | [Next in Thread] |