qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] fixup! char: Convert to new qapi union layout


From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH] fixup! char: Convert to new qapi union layout
Date: Mon, 02 Nov 2015 10:10:38 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Eric Blake <address@hidden> writes:

> Fix build failure on w32
>
> Signed-off-by: Eric Blake <address@hidden>

Squashed, thanks!

> ---
>
> Hopefully, this is the only use of qapi union types hiding
> behind #ifdefs (I relied on the compiler to tell me which spots
> need conversion, but that doesn't work for spots that aren't
> compiled in my setup)

We could've generated a semantic patch for Coccinelle.  Not foolproof,
either, as Coccinelle can miss things when its parser heuristics fail.
Anyway, not worth it now.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]