diff options
author | Alex Bennée <alex.bennee@linaro.org> | 2023-04-24 10:22:48 +0100 |
---|---|---|
committer | Alex Bennée <alex.bennee@linaro.org> | 2023-04-27 14:58:51 +0100 |
commit | 067109a11c83e0b461d7a1a9b531b4c738323477 (patch) | |
tree | 7d555f29da3d54fd40c048790b49cddcc66bf714 /docs/devel/style.rst | |
parent | 6a0057aa22fedf4d6d39b749923e1d6046803fd8 (diff) | |
download | qemu-067109a11c83e0b461d7a1a9b531b4c738323477.zip qemu-067109a11c83e0b461d7a1a9b531b4c738323477.tar.gz qemu-067109a11c83e0b461d7a1a9b531b4c738323477.tar.bz2 |
docs/devel: mention the spacing requirement for QOM
We have a more complete document on QOM but we should at least mention
the style requirements in the style guide.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Message-Id: <20230424092249.58552-18-alex.bennee@linaro.org>
Diffstat (limited to 'docs/devel/style.rst')
-rw-r--r-- | docs/devel/style.rst | 37 |
1 files changed, 37 insertions, 0 deletions
diff --git a/docs/devel/style.rst b/docs/devel/style.rst index 5bc6f2f..ac2ce42 100644 --- a/docs/devel/style.rst +++ b/docs/devel/style.rst @@ -628,6 +628,43 @@ are still some caveats to beware of QEMU Specific Idioms ******************** +QEMU Object Model Declarations +============================== + +The QEMU Object Model (QOM) provides a framework for handling objects +in the base C language. The first declaration of a storage or class +structure should always be the parent and leave a visual space between +that declaration and the new code. It is also useful to separate +backing for properties (options driven by the user) and internal state +to make navigation easier. + +For a storage structure the first declaration should always be called +"parent_obj" and for a class structure the first member should always +be called "parent_class" as below: + +.. code-block:: c + + struct MyDeviceState { + DeviceState parent_obj; + + /* Properties */ + int prop_a; + char *prop_b; + /* Other stuff */ + int internal_state; + }; + + struct MyDeviceClass { + DeviceClass parent_class; + + void (*new_fn1)(void); + bool (*new_fn2)(CPUState *); + }; + +Note that there is no need to provide typedefs for QOM structures +since these are generated automatically by the QOM declaration macros. +See :ref:`qom` for more details. + Error handling and reporting ============================ |