Hi
----- Original Message -----
> marcandre.lur...@redhat.com writes:
>
> > From: Marc-André Lureau <marcandre.lur...@redhat.com>
> >
> > Once the middle mode is removed, the generated marshal functions will no
> > longer be exported.
> >
> > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
> > Reviewed-by: Eric Blake <ebl...@redhat.com>
> > ---
> > monitor.c | 10 +++++-----
> > 1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/monitor.c b/monitor.c
> > index c9191e5..3a28b43 100644
> > --- a/monitor.c
> > +++ b/monitor.c
> > @@ -3659,21 +3659,21 @@ static int monitor_can_read(void *opaque)
> > return (mon->suspend_cnt == 0) ? 1 : 0;
> > }
> >
> > -static bool invalid_qmp_mode(const Monitor *mon, const mon_cmd_t *cmd,
> > +static bool invalid_qmp_mode(const Monitor *mon, const char *cmd,
> > Error **errp)
> > {
> > - bool is_cap = cmd->mhandler.cmd_new == qmp_marshal_qmp_capabilities;
> > + bool is_cap = !g_strcmp0(cmd, "qmp_capabilities");
>
> What's wrong with strcmp()?
>
g_strcmp0() is a slightly safer version, it's just an habit, I guess it doesn't
matter here, would you like to change it?
> >
> > if (is_cap && mon->qmp.in_command_mode) {
> > error_set(errp, ERROR_CLASS_COMMAND_NOT_FOUND,
> > "Capabilities negotiation is already complete, command "
> > - "'%s' ignored", cmd->name);
> > + "'%s' ignored", cmd);
> > return true;
> > }
> > if (!is_cap && !mon->qmp.in_command_mode) {
> > error_set(errp, ERROR_CLASS_COMMAND_NOT_FOUND,
> > "Expecting capabilities negotiation with "
> > - "'qmp_capabilities' before command '%s'", cmd->name);
> > + "'qmp_capabilities' before command '%s'", cmd);
> > return true;
> > }
> > return false;
> > @@ -3964,7 +3964,7 @@ static void handle_qmp_command(JSONMessageParser
> > *parser, GQueue *tokens)
> > "The command %s has not been found", cmd_name);
> > goto err_out;
> > }
> > - if (invalid_qmp_mode(mon, cmd, &local_err)) {
> > + if (invalid_qmp_mode(mon, cmd_name, &local_err)) {
> > goto err_out;
> > }
>
