[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 04/24] qom: make Object a type
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v2 04/24] qom: make Object a type |
Date: |
Thu, 12 Apr 2012 18:18:42 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120312 Thunderbird/11.0 |
Am 12.04.2012 18:11, schrieb Paolo Bonzini:
> Il 12/04/2012 18:09, Andreas Färber ha scritto:
>> I think I had suggested static const and moving them to before the
>> register_types() function?
>
> I'm not sure why that would be useful...
Well, I'm not sure why Anthony is doing it differently from everywhere
else here. :)
Outside function:
Since you move it to the bottom of the file now (which I appreciate)
there's no need to shield it against misuse.
const:
Moves it to different ELF section iiuc and assures no unintentional
modification. We do that for the MemoryRegionOps and I am planning to
post a patch to clean up all old TypeInfo uses (in an intermediate step
by Anthony it was being used in a non-const way) so that they don't
propagate further in the code.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- [Qemu-devel] [PATCH 00/24] qdev properties final installment: push, push!, Paolo Bonzini, 2012/04/11
- [Qemu-devel] [PATCH v2 01/24] qom: add object_class_get_parent, Paolo Bonzini, 2012/04/11
- [Qemu-devel] [PATCH v2 02/24] qom: add object_child_foreach, Paolo Bonzini, 2012/04/11
- [Qemu-devel] [PATCH v2 03/24] qom: add class_base_init, Paolo Bonzini, 2012/04/11
- [Qemu-devel] [PATCH v2 04/24] qom: make Object a type, Paolo Bonzini, 2012/04/11
- Re: [Qemu-devel] [PATCH v2 04/24] qom: make Object a type, Andreas Färber, 2012/04/12
- [Qemu-devel] [PATCH v2 05/24] qdev: push "type" property up to Object, Paolo Bonzini, 2012/04/11
- [Qemu-devel] [PATCH v2 06/24] qdev: fix -device foo,?, Paolo Bonzini, 2012/04/11
[Qemu-devel] [PATCH v2 08/24] qdev: move bus properties to a separate global, Paolo Bonzini, 2012/04/11