On 30.11.2011 09:25, ajia(a)redhat.com wrote:
From: Alex Jia <ajia(a)redhat.com>
The 'true' bool value should be 1, and false is 0.
https://bugzilla.redhat.com/show_bug.cgi?id=758590
Signed-off-by: Alex Jia <ajia(a)redhat.com>
---
tools/virsh.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tools/virsh.c b/tools/virsh.c
index 01da8e5..618b0c1 100644
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -1745,7 +1745,7 @@ cmdDomblklist(vshControl *ctl, const vshCmd *cmd)
VIR_FREE(source);
}
- ret = 0;
+ ret = true;
cleanup:
VIR_FREE(disks);
I'd change commit message:
Fix cmdDomblklist to return true on success instead of 0
But I can live with your's too.
ACK
Michal