[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/14] More #include cleanups
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 00/14] More #include cleanups |
Date: |
Tue, 9 Feb 2016 09:11:04 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 |
On 02/09/2016 09:08 AM, Eric Blake wrote:
> On 02/09/2016 08:24 AM, Peter Maydell wrote:
>> This set of patches does a bunch more cleaning of includes.
>> Specifically it does everything necessary to be able to
>> remove osdep.h includes from header files including qemu-common.h
>>
>> Most of these patches are small, except for the one which
>> deals with tests/ (that one has already been reviewed).
>> There are several patches which fix code generators to
>> generate correct #includes, and some which do manual changes
>> to C++ or other files that Coccinelle barfs on.
>>
>> The "all:" patch is running clean-includes on the whole
>> tree to pick up some new files which have been added
>> recently (mostly via target-arm).
>>
>> I've thrown in a couple of patches that have already appeared
>> on list, just to get the whole set into one series.
>>
>> I have some pending patches which:
>> * enhance clean-includes to handle header files
>> * clean up header files
>> * add a --all option to clean-includes to do a "run clean
>> on whole tree"
>>
>> but I'd like to get the "add missing includes to .c files"
>> parts into master before moving onto "remove includes from .h
>> files" part...
>
> Reviewed-by: Eric Blake <address@hidden>
But failed testing:
CC disas/arm.o
CXX disas/arm-a64.o
In file included from /usr/include/c++/5.3.1/cmath:41:0,
from /home/eblake/qemu-tmp/disas/libvixl/vixl/utils.h:31,
from
/home/eblake/qemu-tmp/disas/libvixl/vixl/a64/disasm-a64.h:31,
from disas/arm-a64.cc:25:
/usr/include/c++/5.3.1/x86_64-redhat-linux/bits/c++config.h:1991:10:
error: expected unqualified-id before ‘namespace’
inline namespace __cxx11 __attribute__((__abi_tag__ ("cxx11"))) { }
^
Not sure what went wrong there, but 3/14 probably needs tweaking.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 04/14] hw/block/nand.c: Include osdep.h first, (continued)
- [Qemu-devel] [PATCH 04/14] hw/block/nand.c: Include osdep.h first, Peter Maydell, 2016/02/09
- [Qemu-devel] [PATCH 02/14] ui/cocoa.m: Include qemu/osdep.h, Peter Maydell, 2016/02/09
- [Qemu-devel] [PATCH 05/14] target-cris: Remove unnecessary ifdef from mmu.c, Peter Maydell, 2016/02/09
- [Qemu-devel] [PATCH 08/14] tests/i440fx-test: Don't define ARRAY_SIZE locally, Peter Maydell, 2016/02/09
- [Qemu-devel] [PATCH 09/14] tests: Clean up includes, Peter Maydell, 2016/02/09
- [Qemu-devel] [PATCH 06/14] cris: Clean up includes, Peter Maydell, 2016/02/09
- [Qemu-devel] [PATCH 11/14] scripts/feature_to_c.sh: Include qemu/osdep.h rather than config.h, Peter Maydell, 2016/02/09
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Eric Blake, 2016/02/09
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups,
Eric Blake <=
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Peter Maydell, 2016/02/09
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Peter Maydell, 2016/02/09
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Eric Blake, 2016/02/09
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Eric Blake, 2016/02/09
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Peter Maydell, 2016/02/09
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Paolo Bonzini, 2016/02/09
- Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Eric Blake, 2016/02/09
Re: [Qemu-devel] [PATCH 00/14] More #include cleanups, Peter Maydell, 2016/02/16