Eric Blake <ebl...@redhat.com> writes: > On 08/31/2015 12:09 PM, Markus Armbruster wrote: > >>>>>> >>>>>> since we don't allow 2D arrays? >>>>> >>>>> If the generators actually rely on it, yes. > >> The appended experimental frontend patch passes "make check". Looks >> like the backends are just fine with arrays of arrays. I'll therefore >> refrain from adding "element type isn't array" assertions to backends. >> >> Since there's plenty of QAPI work on list already, I'll shelve this >> patch for now. We can revisit nested arrays later. > > Another thing to think about if we allow nested arrays: patch 31/32 maps > 'uint8' to 'int', and ['uint8'] to ['int'], but leaves [['uint8']] > untouched. We'd have to add a while loop in qapi-introspect.py to > ensure we drill down through all levels of an array.
Yes, that's broken in my experimental patch.
