[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.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH] fixup! char: Convert to new qapi union layout,
Markus Armbruster <=