On 07.09.2017 07:46, Thomas Huth wrote:
>>  #include "qemu/cutils.h"
>>  #include "cpu.h"
>> -#include "exec/exec-all.h"
> You should maybe mention exec-all.h in the patch description, too.
> (I think it can be dropped here because it is included by
> translate-all.h already).
> 
> If you tweak the patch description:
> Reviewed-by: Thomas Huth <th...@redhat.com>

I'll just drop that hunk although it can also be removed.

Yes, this was not included on purpose - these similar names still
confuse me :)

-- 

Thanks,

David

Reply via email to