diff mbox series

[v3,2/3] docs/interop/qmp-spec: Update error description for parsing errors

Message ID 20230515162245.3964307-3-peter.maydell@linaro.org
State Superseded
Headers show
Series docs/interop: Convert QMP related txt docs to rst | expand

Commit Message

Peter Maydell May 15, 2023, 4:22 p.m. UTC
The description text for a parsing error has changed since the
spec doc was first written; update the example in the docs.

Suggested-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 docs/interop/qmp-spec.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Eric Blake May 15, 2023, 9:10 p.m. UTC | #1
On Mon, May 15, 2023 at 05:22:44PM +0100, Peter Maydell wrote:
> 
> The description text for a parsing error has changed since the
> spec doc was first written; update the example in the docs.
> 
> Suggested-by: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  docs/interop/qmp-spec.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Eric Blake <eblake@redhat.com>

You mentioned elsewhere in the series about keeping my other R-b
because the changes were minor; I agree with that decision.
Markus Armbruster May 16, 2023, 6:19 a.m. UTC | #2
Peter Maydell <peter.maydell@linaro.org> writes:

> The description text for a parsing error has changed since the
> spec doc was first written; update the example in the docs.
>
> Suggested-by: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Reviewed-by: Markus Armbruster <armbru@redhat.com>
diff mbox series

Patch

diff --git a/docs/interop/qmp-spec.rst b/docs/interop/qmp-spec.rst
index bfad570a160..2609b3ff9b2 100644
--- a/docs/interop/qmp-spec.rst
+++ b/docs/interop/qmp-spec.rst
@@ -307,7 +307,7 @@  This section provides some examples of real QMP usage, in all of them
   .. code-block:: QMP
 
     -> { "execute": }
-    <- { "error": { "class": "GenericError", "desc": "Invalid JSON syntax" } }
+    <- { "error": { "class": "GenericError", "desc": "JSON parse error, expecting value" } }
 
 .. admonition:: Example