[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/10] qdev: move bulk of qdev-properties.c to q
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 07/10] qdev: move bulk of qdev-properties.c to qom/object.c |
Date: |
Fri, 25 May 2012 15:36:07 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 |
Il 25/05/2012 15:32, Andreas Färber ha scritto:
> Am 23.05.2012 17:44, schrieb Paolo Bonzini:
>> Signed-off-by: Paolo Bonzini <address@hidden>
>> ---
>> hw/qdev-properties.c | 487
>> +++++--------------------------------------------
>> hw/qdev.c | 47 +----
>> hw/qdev.h | 87 ---------
>> include/qemu/object.h | 98 ++++++++++
>> qom/object.c | 459 ++++++++++++++++++++++++++++++++++++++++++++++
>> 5 files changed, 604 insertions(+), 574 deletions(-)
>
> I'm not so fond of blowing up object.c with stuff that was originally in
> its own file. Had you considered a new qom/object-properties.c?
I hadn't, but it can be done. I can do it next week, or review the
result, as you prefer.
Paolo
- Re: [Qemu-devel] [PATCH 01/10] qdev: remove PropertyInfo range checking, (continued)
- [Qemu-devel] [PATCH 02/10] qdev: remove qdev_prop_exists, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 05/10] qdev: push state up to Object, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 06/10] qdev: generalize properties to Objects, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 07/10] qdev: move bulk of qdev-properties.c to qom/object.c, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 09/10] qom: add realized property, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 10/10] qom: Add QERR_PROPERTY_SET_AFTER_REALIZE, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 04/10] qom: add get_id, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 08/10] qom: push static properties to Object, Paolo Bonzini, 2012/05/23