Re: [Qemu-devel] [PATCH 01/10] Introduce qmisc module

[Anthony Liguori]

Luiz Capitulino wrote:
> > > +/**
> > > + * qobject_from_fmt(): build QObjects from a specified format.
> > > + *
> > > + * Valid characters of the format:
> > > + *
> > > + * i   integer, map to QInt
> > > + * s   string, map to QString
> > > + * []  list, map to QList
> > > + * {}  dictionary, map to QDict
> > > + *
> > > + * Examples:
> > > + *
> > > + * - Create a QInt
> > > + *
> > > + *      qobject_from_fmt("i", 42);
> > > + *
> > > + * - Create a QList of QStrings
> > > + *
> > > + *      qobject_from_fmt("[ i, i, i ]", 0, 1 , 2);
> > > + *
> > > + * - Create a QDict with mixed data-types
> > > + *
> > > + *      qobject_from_fmt("{ s: [ i, s ], s: i }", ... );
> > > + *
> > > + * Return a strong reference to a QObject on success, NULL otherwise.
> > > + */
> > >   
> > >       
> > But my real objection is that we should make this "{%s: [%d, %s], %s: 
> > %d}" so that we can mark it as a printf formatted function and get type 
> > checking.  You'll probably have to support both "%d" and "%" PRId64 for 
> > sanity sake.
> >     
>
>  Trivial to do if we ignore the '%' characters. :))
>   
Okay, I'd like to see that and I'd like to see string parsing.  We can 
rewrite the parser later.

Regards,

Anthony Liguori