|
From: | Philippe Mathieu-Daudé |
Subject: | Re: [Qemu-devel] [PATCH 1/2] vl: Factor object_create() out of main() |
Date: | Fri, 11 Aug 2017 00:33:54 -0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 08/10/2017 01:28 PM, Markus Armbruster wrote:
Eric Blake <address@hidden> writes:On 08/10/2017 07:25 AM, Markus Armbruster wrote:Signed-off-by: Markus Armbruster <address@hidden>
for the refactor: Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
--- vl.c | 21 ++++++++++----------- 1 file changed, 10 insertions(+), 11 deletions(-)+++ b/vl.c @@ -2845,7 +2845,6 @@ static bool object_create_initial(const char *type) return true; }-/*Spurious whitespace change? I can live with it because it adds consistency, but it's not on a function directly touched by this patch.Not intentional (alternatively: I've since forgotten). I'll drop it.
If you ever send the whitespace change as another patch with at least some funny message:
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Eric Blake <address@hidden>Thanks!
[Prev in Thread] | Current Thread | [Next in Thread] |