[Qemu-devel] Re: [PATCH 02/29] Introduce QInt

[Luiz Capitulino <lcapitulino@redhat.com>]

On Thu, 13 Aug 2009 16:55:50 +0300
Avi Kivity <avi@redhat.com> wrote:

> On 08/13/2009 04:50 PM, Luiz Capitulino wrote:
> > +
> > +/**
> > + * qobject_to_qint(): Convert a QObject into a QInt
> > + */
> > +static inline QInt *qobject_to_qint(const QObject *obj)
> > +{
> > +    return container_of(obj, QInt, base);
> > +}
> > +
> >    
> 
> It's worthwhile to check the type here and return NULL if not an 
> integer.  That allows users to
> 
>     QInt *qi = qobject_to_qint(...);
> 
>     if (qi) {
>         do something
>     } else {
>         error handling
>     }
> 
> (this is equivalent to C++'s static_cast vs. dynamic_cast)

 OK.

 Note that QDict has high-level helpers like qdict_get_int()
that assumes that the stored key is a QInt.

 I was in doubt about how to fail. I chose to use assert() and
consider this a programming error.