[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/10] qapi: qapi-types.py, native list support
From: |
mdroth |
Subject: |
Re: [Qemu-devel] [PATCH 01/10] qapi: qapi-types.py, native list support |
Date: |
Fri, 10 May 2013 06:32:21 -0500 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, May 10, 2013 at 11:04:22AM +0800, Amos Kong wrote:
> On Thu, May 09, 2013 at 09:20:53PM -0500, Michael Roth wrote:
> > Teach type generators about native types so they can generate the
> > appropriate linked list types.
> >
> > Signed-off-by: Michael Roth <address@hidden>
> > ---
> > scripts/qapi-types.py | 43 ++++++++++++++++++++++++++++++++++++++++---
> > scripts/qapi.py | 21 +++++++++++++++++++++
> > 2 files changed, 61 insertions(+), 3 deletions(-)
> >
> > diff --git a/scripts/qapi-types.py b/scripts/qapi-types.py
> > index 9e19920..96cb26d 100644
> > --- a/scripts/qapi-types.py
> > +++ b/scripts/qapi-types.py
> > @@ -16,7 +16,18 @@ import os
> > import getopt
> > import errno
>
> <snip>
>
> > diff --git a/scripts/qapi.py b/scripts/qapi.py
> > index afc5f32..0ac8c2b 100644
> > --- a/scripts/qapi.py
> > +++ b/scripts/qapi.py
> > @@ -11,6 +11,10 @@
> >
> > from ordereddict import OrderedDict
> >
> > +builtin_types = [
> > + 'str', 'int', 'number', 'bool'
>
> Will we add size, int8, uint32, ... in future when they are needed?
Well, that was the plan, but I hadn't recalled that support for these
was already been added to the code generators as part of the Netdev
opts-visitor stuff, so I'll go ahead and do a re-spin with those added.
>
> > +]
> > +
>
> --
> Amos.
>
- [Qemu-devel] [PATCH v2 00/10] qapi: add support for lists of native types, Michael Roth, 2013/05/09
- [Qemu-devel] [PATCH 01/10] qapi: qapi-types.py, native list support, Michael Roth, 2013/05/09
- [Qemu-devel] [PATCH 02/10] qapi: qapi-visit.py, fix list handling for union types, Michael Roth, 2013/05/09
- [Qemu-devel] [PATCH 03/10] qapi: qapi-visit.py, native list support, Michael Roth, 2013/05/09
- [Qemu-devel] [PATCH 04/10] qapi: enable generation of native list code, Michael Roth, 2013/05/09
- [Qemu-devel] [PATCH 05/10] qapi: fix leak in unit tests, Michael Roth, 2013/05/09