qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v4 28/29] sysemu: Move the VMChangeStateEntry ty


From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH v4 28/29] sysemu: Move the VMChangeStateEntry typedef to qemu/typedefs.h
Date: Mon, 12 Aug 2019 15:54:54 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux)

Alex Bennée <address@hidden> writes:

> Markus Armbruster <address@hidden> writes:
>
>> In my "build everything" tree, changing sysemu/sysemu.h triggers a
>> recompile of some 1800 out of 6600 objects (not counting tests and
>> objects that don't depend on qemu/osdep.h, down from 5400 due to the
>> previous commit).
>>
>> Several headers include sysemu/sysemu.h just to get typedef
>> VMChangeStateEntry.  Move it from sysemu/sysemu.h to qemu/typedefs.h.
>> Spell its structure tag the same while there.  Drop the now
>> superfluous includes of sysemu/sysemu.h from headers.
>
> You should probably mention you also fix the struct definition to meet
> our coding style. Otherwise:

I did: "Spell its structure tag the same while there."  Would you like
to suggest a better wording?

> Reviewed-by: Alex Bennée <address@hidden>

Thanks!



reply via email to

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