NuKVM/libvirt
3
0
Fork 0
mirror of https://github.com/libvirt/libvirt.git synced 2026-09-03 20:53:04 -05:00
Code Issues Packages Projects Releases Wiki Activity

Revert "error: Add VIR_ERR_DEPRECATED error code"

Browse Source 
This reverts commit 226094fbc4.

A deprecation is a warning to something that use of a feature is
being discouraged. By definition it is not an error condition to
continue to use a deprecated feature.

A VIR_ERR_DEPRECATED constant thus makes no conceptual sense. For
features which are entirely absent we already document that the
VIR_ERR_NO_SUPPORT code will be used. There is no need to distinguish
between a feature which never existed and a feature which previously
existed and was since removed.

Reviewed-by: Ján Tomko <jtomko@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Daniel P. Berrangé
2019-06-27 14:47:10 +01:00
parent 2fbaa28e12
commit bd17012f0c
2 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
include/libvirt/virterror.h
-1
View File
@@ -329,7 +329,6 @@ typedef enum {
VIR_ERR_INVALID_NETWORK_PORT = 105, /* invalid network port object */ VIR_ERR_INVALID_NETWORK_PORT = 105, /* invalid network port object */
VIR_ERR_NETWORK_PORT_EXIST = 106, /* the network port already exist */ VIR_ERR_NETWORK_PORT_EXIST = 106, /* the network port already exist */
VIR_ERR_NO_NETWORK_PORT = 107, /* network port not found */ VIR_ERR_NO_NETWORK_PORT = 107, /* network port not found */
VIR_ERR_DEPRECATED = 108, /* configuration or operation is no longer supported */
# ifdef VIR_ENUM_SENTINELS # ifdef VIR_ENUM_SENTINELS
VIR_ERR_NUMBER_LAST VIR_ERR_NUMBER_LAST
src/util/virerror.c
-4
View File
@@ -1235,10 +1235,6 @@ const virErrorMsgTuple virErrorMsgStrings[VIR_ERR_NUMBER_LAST] = {
[VIR_ERR_NO_NETWORK_PORT] = { [VIR_ERR_NO_NETWORK_PORT] = {
N_("network port not found"), N_("network port not found"),
N_("network port not found: %s") }, N_("network port not found: %s") },
[VIR_ERR_DEPRECATED] = {
N_("operation or configuration no longer supported"),
"%s",
},
}; };