Pārlūkot izejas kodu

qmp hmp: Improve error messages when SPICE is not in use

Commit 7572150 adopted QERR_DEVICE_NOT_ACTIVE for the purpose,
probably because adding another error seemed cumbersome overkill.
Produces "No spice device has been activated", which is awkward.

We've since abandoned our quest for "rich" error objects.  Time to
undo the damage to this error message.  Replace it by "SPICE is not in
use".

Keep the stupid DeviceNotActive ErrorClass for compatibility, even
though Libvirt doesn't use it.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Gerd Hoffmann <kraxel@redhat.com>
Markus Armbruster 10 gadi atpakaļ
vecāks
revīzija
cfa9bb2369
1 mainītis faili ar 2 papildinājumiem un 2 dzēšanām
  1. 2 2
      include/ui/qemu-spice.h

+ 2 - 2
include/ui/qemu-spice.h

@@ -91,8 +91,8 @@ static inline int qemu_spice_display_add_client(int csock, int skipauth,
 static inline bool qemu_using_spice(Error **errp)
 static inline bool qemu_using_spice(Error **errp)
 {
 {
     if (!using_spice) {
     if (!using_spice) {
-        /* correct one? spice isn't a device ,,, */
-        error_set(errp, QERR_DEVICE_NOT_ACTIVE, "spice");
+        error_set(errp, ERROR_CLASS_DEVICE_NOT_ACTIVE,
+                  "SPICE is not in use");
         return false;
         return false;
     }
     }
     return true;
     return true;