On 03/08/2016 03:19 AM, Kevin Wolf wrote: > Am 08.03.2016 um 09:22 hat Markus Armbruster geschrieben: >> Cc: Kevin, because he added the array in question. >> >> Peter Xu <pet...@redhat.com> writes: >>
>> Unrelated to your patch: I think we've pushed QDict use father than >> sensible. Encoding multiple keys in a string so you can use a flat >> associative array as your catch-all data structure is appropriate in >> AWK, but in C? Not so much... > > We'll always to that, because it's the command line syntax. What you may > criticise is that we convert QAPI objects to the command line > representation instead of the other way around, but changing that (which > I think would be far from trivial, for relatively little use) wouldn't > get rid of this kind of key parsing, but just move it a bit closer to > the command line handling. I actually WANT us to try that conversion (a great GSoC project, if someone wants it) for 2.7. We already did it for SocketAddress, and it makes the code easier to maintain when you can just access foo->data instead of doing qdict_lookup(foo, "data") all over the place. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
