|
From: | Thomas Huth |
Subject: | Re: [PATCH 10/10] exec/exec-all: Move 'qemu/log.h' include in units requiring it |
Date: | Fri, 4 Feb 2022 16:33:29 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 |
On 03/02/2022 20.38, Philippe Mathieu-Daudé wrote:
Many files use "qemu/log.h" declarations but neglect to include it (they inherit it via "exec/exec-all.h"). "exec/exec-all.h" is a core component and shouldn't be used that way. Move the "qemu/log.h" inclusion locally to each unit requiring it. Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> ---
This seems to be a good idea, thanks for cleaning this up! Acked-by: Thomas Huth <thuth@redhat.com>
[Prev in Thread] | Current Thread | [Next in Thread] |