Re: [Qemu-devel] [PATCH v0 00/10]: More QObject conversions

[Markus Armbruster]

Luiz Capitulino <address@hidden> writes:

>  Hi there,
>
>  Here goes another set of conversions, just to keep the ball rolling.
>
>  The first patch introduces a function called qobject_from_fmt(), which can
> be used to build QObjects from a specified printf-like format.
>
>  Most of its code was based on a Python function with similiar functionality,
> as far as I could understand the license is compatible and I've added a
> copyright notice.
>
>  Is this the right procedure?
>
>  Most patches are simple (as we aren't converting errors yet), except the
> last ones. Which use qobject_from_fmt() and are very cool, hopefully they're
> right too. :)
>
> PS: This series apply on top of my last 'Initial QObject conversion' series.
>
>  Thanks.

The separation of presentation from logic seems to work reasonably well
so far.  The separation involves building explicit data to be sent by a
separate presentation function instead of printing right away.  It's key
that this data building code is about as legible and easy to maintain as
the old, straightforward printing.  qobject_from_fmt() helps with that,
in my opinion.

Of course, much "interesting" stuff remains, chiefly errors.  Also any
output that doesn't fit into the "run command to completion, report its
result" mold (not sure we need that).  We'll see how it goes.


PS: Fun to see Greenspun's Tenth Rule in action once more.