[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 0/6] decouple board headers from cpu.h
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [RFC PATCH 0/6] decouple board headers from cpu.h |
Date: |
Tue, 15 Dec 2015 16:06:19 +0000 |
On 15 December 2015 at 14:28, Paolo Bonzini <address@hidden> wrote:
> These patches are perhaps the only somewhat controversial part.
> Generally, we add opaque typedefs to typedefs.h in order to avoid
> indirect header inclusions. However, this might not be desirable
> for target specific types such as FooCPU. These types are used
> mostly in headers for boards, and thus this series uses struct
> explicitly in those headers. Adding a typedef breaks on older
> compilers that do not like redefinitions of typedefs.
I would prefer us to provide the typedef. There's no problem
with older compilers because you just only define the typedef
in one place (typically in typedefs.h).
thanks
-- PMM
- [Qemu-devel] [RFC PATCH 0/6] decouple board headers from cpu.h, Paolo Bonzini, 2015/12/15
- [Qemu-devel] [PATCH 2/6] mips: use struct CPUMIPSState to avoid need for cpu.h, Paolo Bonzini, 2015/12/15
- [Qemu-devel] [PATCH 3/6] ppc: use struct to avoid the need for cpu.h, Paolo Bonzini, 2015/12/15
- [Qemu-devel] [PATCH 5/6] coldfire: use "struct M68kCPU" in header files, Paolo Bonzini, 2015/12/15
- [Qemu-devel] [PATCH 4/6] alpha: use AlphaCPU as an opaque type, Paolo Bonzini, 2015/12/15
- [Qemu-devel] [PATCH 1/6] arm: use "struct ARMCPU" in header files, Paolo Bonzini, 2015/12/15
- [Qemu-devel] [PATCH 6/6] sh: use SuperHCPU struct to avoid cpu.h dependency, Paolo Bonzini, 2015/12/15
- Re: [Qemu-devel] [RFC PATCH 0/6] decouple board headers from cpu.h,
Peter Maydell <=