On 06/24/2016 08:19 AM, Markus Armbruster wrote:
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

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
being intentional.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org