Re: [Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entries()

2015-05-11 Thread Eric Blake
On 05/11/2015 08:40 AM, Kevin Wolf wrote: >>> +char indexstr[slen], prefix[slen]; >> >> And more dependence on a working C99 compiler, thanks to variable length >> array (VLA). >> >>> +size_t snprintf_ret; >>> + >>> +snprintf_ret = snprintf(indexstr, slen, "%s%u", subqdict,

Re: [Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entries()

2015-05-11 Thread Kevin Wolf
Am 08.05.2015 um 22:06 hat Eric Blake geschrieben: > On 05/08/2015 11:21 AM, Kevin Wolf wrote: > > Signed-off-by: Kevin Wolf > > --- > > Might want to include mention of what it will be used for in the commit > body. You're right. This is the new commit message: This counts the entries in a

Re: [Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entries()

2015-05-11 Thread Max Reitz
On 08.05.2015 19:21, Kevin Wolf wrote: Signed-off-by: Kevin Wolf --- include/qapi/qmp/qdict.h | 1 + qobject/qdict.c | 68 +--- 2 files changed, 65 insertions(+), 4 deletions(-) diff --git a/include/qapi/qmp/qdict.h b/include/qapi/qmp/qd

Re: [Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entries()

2015-05-08 Thread Eric Blake
On 05/08/2015 02:06 PM, Eric Blake wrote: > On 05/08/2015 11:21 AM, Kevin Wolf wrote: >> Signed-off-by: Kevin Wolf >> --- > > Might want to include mention of what it will be used for in the commit > body. > >> include/qapi/qmp/qdict.h | 1 + >> qobject/qdict.c | 68 >> ++

Re: [Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entries()

2015-05-08 Thread Eric Blake
On 05/08/2015 11:21 AM, Kevin Wolf wrote: > Signed-off-by: Kevin Wolf > --- Might want to include mention of what it will be used for in the commit body. > include/qapi/qmp/qdict.h | 1 + > qobject/qdict.c | 68 > +--- > 2 files changed, 65

[Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entries()

2015-05-08 Thread Kevin Wolf
Signed-off-by: Kevin Wolf --- include/qapi/qmp/qdict.h | 1 + qobject/qdict.c | 68 +--- 2 files changed, 65 insertions(+), 4 deletions(-) diff --git a/include/qapi/qmp/qdict.h b/include/qapi/qmp/qdict.h index d68f4eb..d20db94 100644 --- a/in