Re: [Qemu-devel] [PATCH v19 07/11] qapi: make string output visitor parse int list

[Paolo Bonzini]

Il 04/03/2014 10:23, Hu Tao ha scritto:
> > >
> > > > >+            } else {
> > > > >+                out = g_strdup_printf("%lld-%lld",
> > > > >+                                      (long long) sov->range_start.s,
> > > > >+                                      (long long) sov->range_end.s);
> > > > >+            }
> > > > >+        }
> > >
> > > This looks wrong.  You do not insert any separator, and you do not
> > > handle things like "0-3,8-11".  You probably should use a GString
> > > instead of string_output_set.
>
> Right. We should also handle "0-3,8-11"-like lists in string input
> visitor and opts visitor.

Ok, no big deal; let's do one thing at a time.

Regarding string input visitor, I believe we can avoid the code 
duplication by:

(1) using Qmp input/output visitors in query-memdev

(2) using OptsVisitor in -object


The string output visitor becomes more important than the string input 
visitor, but it is only needed for HMP so it is not urgent.

We can then work on making OptsVisitor use the string input visitor 
internally.

But I'm already applying patches 1-5, 8, 9, 10 (the latter with the 
above changes) to the numa tree.

Paolo