[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v2 1/5] Use #include "..." exactly for our o
Re: [Qemu-devel] [PATCH RFC v2 1/5] Use #include "..." exactly for our own headers
Fri, 24 Jun 2016 08:54:34 -0600
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0
On 06/24/2016 08:19 AM, Markus Armbruster wrote:
> Signed-off-by: Markus Armbruster <address@hidden>
> Reviewed-by: Peter Maydell <address@hidden>
Worth mentioning how you found the culprits in the commit message, so
that someone could repeat the task when backporting this patch or
dealing with future cruft that inevitably creeps back in without an
automated checkin validation tool?
> +++ b/crypto/pbkdf-nettle.c
> @@ -19,9 +19,9 @@
> #include "qemu/osdep.h"
> +#include <nettle/pbkdf2.h>
> #include "qapi/error.h"
> #include "crypto/pbkdf.h"
> -#include "nettle/pbkdf2.h"
You're not just converting <> to "" (when the header is internal) or ""
to <> (when the header is 3rd-party), but also rearranging things to put
<> before "" (except for osdep.h which must be first). I like that
paradigm, but again, might be worth a mention in the commit message as
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Description: OpenPGP digital signature
[Qemu-devel] [PATCH RFC v2 3/5] tests: Make check-block a phony target, Markus Armbruster, 2016/06/24
[Qemu-devel] [PATCH RFC v2 5/5] include: Include exec/hwaddr.h where hwaddr is used, Markus Armbruster, 2016/06/24