On Tue, 9 Apr 2019 at 16:58, Aruna Jayasena <aruna...@cse.mrt.ac.lk> wrote: > > Removed unwanted includes from cpu-common.h > This task was under https://wiki.qemu.org/Contribute/BiteSizedTasks > > Signed-off-by: Aruna Jayasena <aruna...@cse.mrt.ac.lk> > > --- > include/exec/cpu-common.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/include/exec/cpu-common.h b/include/exec/cpu-common.h > index cef8b88a2a..a3594f3f50 100644 > --- a/include/exec/cpu-common.h > +++ b/include/exec/cpu-common.h > @@ -7,8 +7,6 @@ > #include "exec/hwaddr.h" > #endif > > -#include "qemu/bswap.h" > -#include "qemu/queue.h" > #include "qemu/fprintf-fn.h" >
I was surprised that no .c files are implicitly relying on these includes, but apparently none are, so we can safely remove them. A couple of notes: * this doesn't need to go to qemu-stable as it's just a code cleanup * did something go wrong with your sending of this patch? Its subject is "PATCH 0/4" which would usually indicate a cover letter for a four-patch series, but this seems to be a stand-alone patch. Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM