Re: [Qemu-devel] [PULL 09/10] monitor: add query-vnc2 command

2015-01-12 Thread Markus Armbruster
Gerd Hoffmann writes: > Hi, > >> > If the schema let us specify the enumeration values, we could avoid the >> > mapping altogether. >> >> Ooh, cool thought: what if we allowed: >> >> { 'enum': 'Foo', 'data': [ >> { 'name': 'One', 'value': 1 }, >> { 'name': 'Two', 'value': 2 } ] } > > Look

Re: [Qemu-devel] [PULL 09/10] monitor: add query-vnc2 command

2015-01-08 Thread Gerd Hoffmann
On Fr, 2014-12-19 at 14:04 +0100, Markus Armbruster wrote: > Gerd Hoffmann writes: > > > Add new query vnc qmp command, for the lack of better ideas just name it > > "query-vnc2". Changes over query-vnc: > > > > * It returns a list of vnc servers, so multiple vnc server instances > >are cov

Re: [Qemu-devel] [PULL 09/10] monitor: add query-vnc2 command

2015-01-06 Thread Gerd Hoffmann
Hi, > > If the schema let us specify the enumeration values, we could avoid the > > mapping altogether. > > Ooh, cool thought: what if we allowed: > > { 'enum': 'Foo', 'data': [ > { 'name': 'One', 'value': 1 }, > { 'name': 'Two', 'value': 2 } ] } Looks nice. Where is the patch? cheers,

Re: [Qemu-devel] [PULL 09/10] monitor: add query-vnc2 command

2015-01-06 Thread Gerd Hoffmann
> > Add new query vnc qmp command, for the lack of better ideas just name it > > "query-vnc2". Changes over query-vnc: > Call it query-vnc-servers? Fine with me, done. > Maybe just VncPrimaryAuth, since there's precedence for abbreviating > authentication that way. Done. > > ## > > +# @quer

Re: [Qemu-devel] [PULL 09/10] monitor: add query-vnc2 command

2014-12-19 Thread Eric Blake
On 12/19/2014 06:04 AM, Markus Armbruster wrote: > Gerd Hoffmann writes: > >> Add new query vnc qmp command, for the lack of better ideas just name it >> "query-vnc2". Changes over query-vnc: >> >> * It returns a list of vnc servers, so multiple vnc server instances >>are covered. >> * Eac

Re: [Qemu-devel] [PULL 09/10] monitor: add query-vnc2 command

2014-12-19 Thread Markus Armbruster
Gerd Hoffmann writes: > Add new query vnc qmp command, for the lack of better ideas just name it > "query-vnc2". Changes over query-vnc: > > * It returns a list of vnc servers, so multiple vnc server instances >are covered. > * Each vnc server returns a list of server sockets. Followup pa

[Qemu-devel] [PULL 09/10] monitor: add query-vnc2 command

2014-12-19 Thread Gerd Hoffmann
Add new query vnc qmp command, for the lack of better ideas just name it "query-vnc2". Changes over query-vnc: * It returns a list of vnc servers, so multiple vnc server instances are covered. * Each vnc server returns a list of server sockets. Followup patch will use that to also report