Marc-André Lureau <marcandre.lur...@gmail.com> writes: > Hi > On Thu, Aug 30, 2018 at 6:03 PM Markus Armbruster <arm...@redhat.com> wrote: >> >> Marc-André Lureau <marcandre.lur...@redhat.com> writes: >> >> > Simplify the code around qmp_dispatch(): >> > - rely on qmp_dispatch/check_obj() for message checking >> > - have a single send_response() point >> > - constify send_response() argument >> > >> > It changes a couple of error messages: >> > >> > * When @req isn't a dictionary, from >> > Invalid JSON syntax >> > to >> > QMP input must be a JSON object >> > >> > * When @req lacks member "execute", from >> > this feature or command is not currently supported >> > to >> > QMP input lacks member 'execute' >> > >> > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> >> > Reviewed-by: Michael Roth <mdr...@linux.vnet.ibm.com> >> >> Reviewed-by: Markus Armbruster <arm...@redhat.com> >> > > Can someone take the patch? thanks
Michael?
