[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 10/10] qapi: Inline and remove QERR_PROPERTY_VALUE_BAD definition
From: |
Markus Armbruster |
Subject: |
[PATCH 10/10] qapi: Inline and remove QERR_PROPERTY_VALUE_BAD definition |
Date: |
Tue, 12 Mar 2024 15:13:43 +0100 |
From: Philippe Mathieu-Daudé <philmd@linaro.org>
Address the comment added in commit 4629ed1e98
("qerror: Finally unused, clean up"), from 2015:
/*
* These macros will go away, please don't use
* in new code, and do not add new ones!
*/
Manual change. Remove the definition in
include/qapi/qmp/qerror.h.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
include/qapi/qmp/qerror.h | 3 ---
hw/core/qdev-properties.c | 3 +--
2 files changed, 1 insertion(+), 5 deletions(-)
diff --git a/include/qapi/qmp/qerror.h b/include/qapi/qmp/qerror.h
index 385a4876d6..00b18e9082 100644
--- a/include/qapi/qmp/qerror.h
+++ b/include/qapi/qmp/qerror.h
@@ -26,9 +26,6 @@
#define QERR_MISSING_PARAMETER \
"Parameter '%s' is missing"
-#define QERR_PROPERTY_VALUE_BAD \
- "Property '%s.%s' doesn't take value '%s'"
-
#define QERR_PROPERTY_VALUE_OUT_OF_RANGE \
"Property %s.%s doesn't take value %" PRId64 " (minimum: %" PRId64 ",
maximum: %" PRId64 ")"
diff --git a/hw/core/qdev-properties.c b/hw/core/qdev-properties.c
index 7d6fa726fd..86a583574d 100644
--- a/hw/core/qdev-properties.c
+++ b/hw/core/qdev-properties.c
@@ -2,7 +2,6 @@
#include "hw/qdev-properties.h"
#include "qapi/error.h"
#include "qapi/qapi-types-misc.h"
-#include "qapi/qmp/qerror.h"
#include "qapi/qmp/qlist.h"
#include "qemu/ctype.h"
#include "qemu/error-report.h"
@@ -792,7 +791,7 @@ void error_set_from_qdev_prop_error(Error **errp, int ret,
Object *obj,
break;
default:
case -EINVAL:
- error_setg(errp, QERR_PROPERTY_VALUE_BAD,
+ error_setg(errp, "Property '%s.%s' doesn't take value '%s'",
object_get_typename(obj), name, value);
break;
case -ENOENT:
--
2.44.0
- [PATCH 01/10] error: Drop superfluous #include "qapi/qmp/qerror.h", (continued)
- [PATCH 01/10] error: Drop superfluous #include "qapi/qmp/qerror.h", Markus Armbruster, 2024/03/12
- [PATCH 06/10] qapi: Inline QERR_INVALID_PARAMETER_TYPE definition (constant value), Markus Armbruster, 2024/03/12
- [PATCH 02/10] qapi: Inline and remove QERR_BUS_NO_HOTPLUG definition, Markus Armbruster, 2024/03/12
- [PATCH 04/10] qapi: Inline and remove QERR_DEVICE_NO_HOTPLUG definition, Markus Armbruster, 2024/03/12
- [PATCH 07/10] qapi: Inline and remove QERR_INVALID_PARAMETER_TYPE definition, Markus Armbruster, 2024/03/12
- [PATCH 10/10] qapi: Inline and remove QERR_PROPERTY_VALUE_BAD definition,
Markus Armbruster <=
- [PATCH 09/10] qapi: Inline and remove QERR_MIGRATION_ACTIVE definition, Markus Armbruster, 2024/03/12
- [PATCH 08/10] qapi: Correct error message for 'vcpu_dirty_limit' parameter, Markus Armbruster, 2024/03/12
[PATCH 05/10] qapi: Inline and remove QERR_INVALID_PARAMETER definition, Markus Armbruster, 2024/03/12