[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/4] Header cleanups
From: |
Aruna Jayasena |
Subject: |
Re: [Qemu-devel] [PATCH 0/4] Header cleanups |
Date: |
Tue, 9 Apr 2019 22:12:59 +0530 |
Thank you
On Tue, Apr 9, 2019 at 10:12 PM, Aruna Jayasena <address@hidden>
wrote:
> Yeah this is a stand alone patch.
>
> On Tue, Apr 9, 2019 at 10:10 PM, Peter Maydell <address@hidden>
> wrote:
>
>> On Tue, 9 Apr 2019 at 16:58, Aruna Jayasena <address@hidden>
>> wrote:
>> >
>> > Removed unwanted includes from cpu-common.h
>> > This task was under https://wiki.qemu.org/Contribute/BiteSizedTasks
>> >
>> > Signed-off-by: Aruna Jayasena <address@hidden>
>> >
>> > ---
>> > 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 <address@hidden>
>>
>> thanks
>> -- PMM
>>
> --
> *Aruna* Jayasena
> Embedded Systems and Computer Science Engineering Undergraduate
> University of Moratuwa
> *✆ **+94719872152 *
> ✉ <address@hidden> ⌨ <http://archfx.github.io> in
> <https://www.linkedin.com/in/arunajayasena/>
>
>
> --
*Aruna* Jayasena
Embedded Systems and Computer Science Engineering Undergraduate
University of Moratuwa
*✆ **+94719872152 *
✉ <address@hidden> ⌨ <http://archfx.github.io> in
<https://www.linkedin.com/in/arunajayasena/>
- [Qemu-devel] [PATCH 0/4] Header cleanups, Aruna Jayasena, 2019/04/09
- Re: [Qemu-devel] [PATCH 0/4] Header cleanups, Peter Maydell, 2019/04/09
- Re: [Qemu-devel] [PATCH 0/4] Header cleanups, Aruna Jayasena, 2019/04/09
- Re: [Qemu-devel] [PATCH 0/4] Header cleanups,
Aruna Jayasena <=
- Re: [Qemu-devel] [PATCH 0/4] Header cleanups, Peter Maydell, 2019/04/09
- Re: [Qemu-devel] [PATCH 0/4] Header cleanups, Aruna Jayasena, 2019/04/09
- Re: [Qemu-devel] [PATCH 0/4] Header cleanups, Markus Armbruster, 2019/04/10
- Re: [Qemu-devel] [Qemu-trivial] [PATCH 0/4] Header cleanups, Laurent Vivier, 2019/04/11
- Re: [Qemu-devel] [Qemu-trivial] [PATCH 0/4] Header cleanups, Aruna Jayasena, 2019/04/11
Re: [Qemu-devel] [PATCH 0/4] Header cleanups, Thomas Huth, 2019/04/09