Re: [PATCH 8/8] vbox: Stop reporting RC in error messages

As shown in the commit that introduced vboxReportError(), we are appending the retval of a failed VirtualBox API onto our error messages. Well, this is no longer needed because vboxReportError() already appends the VirtualBox error in plain text. Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com&gt; --- src/vbox/vbox_common.c | 301 ++++++++++++++++++----------------------- 1 file changed, 132 insertions(+), 169 deletions(-) diff --git a/src/vbox/vbox_common.c b/src/vbox/vbox_common.c index cf404ea9d7..0a83cc5a61 100644 --- a/src/vbox/vbox_common.c +++ b/src/vbox/vbox_common.c @@ -570,8 +570,8 @@ vboxSetStorageController(virDomainControllerDef *controller, if (NS_FAILED(rc)) { vboxReportError(VIR_ERR_INTERNAL_ERROR, - _("Failed to add storage controller (name: %s, busType: %d), rc=%08x"), - debugName, vboxBusType, rc); + _("Failed to add storage controller (name: %s, busType: %d)"), + debugName, vboxBusType); goto cleanup; } @@ -581,7 +581,7 @@ vboxSetStorageController(virDomainControllerDef *controller, vboxModel); if (NS_FAILED(rc)) { vboxReportError(VIR_ERR_INTERNAL_ERROR,