diff options
author | Richard Jones <rjones@redhat.com> | 2010-02-12 14:06:25 +0000 |
---|---|---|
committer | Richard Jones <rjones@redhat.com> | 2010-02-12 16:49:00 +0000 |
commit | e9c37113104c1cfb234535adc9b52ad3880a41ce (patch) | |
tree | a3367b5e700b9b455340243d7c66c45a5adf6067 /daemon/swap.c | |
parent | 9b3ef85a3d1b43faa98bb5dc922f8a273a5e1dde (diff) | |
download | libguestfs-e9c37113104c1cfb234535adc9b52ad3880a41ce.tar.gz libguestfs-e9c37113104c1cfb234535adc9b52ad3880a41ce.tar.xz libguestfs-e9c37113104c1cfb234535adc9b52ad3880a41ce.zip |
daemon: Don't need to prefix error messages with the command name.
The RPC stubs already prefix the command name to error messages.
The daemon doesn't have to do this. As a (small) benefit this also
makes the daemon slightly smaller.
Code in the daemon such as:
if (argv[0] == NULL) {
reply_with_error ("passed an empty list");
return NULL;
}
now results in error messages like this:
><fs> command ""
libguestfs: error: command: passed an empty list
(whereas previously you would have seen ..command: command:..)
Diffstat (limited to 'daemon/swap.c')
-rw-r--r-- | daemon/swap.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/daemon/swap.c b/daemon/swap.c index 2d3d9ff4..fbeb895f 100644 --- a/daemon/swap.c +++ b/daemon/swap.c @@ -58,7 +58,7 @@ mkswap (const char *device, const char *flag, const char *value) r = command (NULL, &err, "/sbin/mkswap", "-f", flag, value, device, NULL); if (r == -1) { - reply_with_error ("mkswap: %s", err); + reply_with_error ("%s", err); free (err); return -1; } @@ -115,7 +115,7 @@ swaponoff (const char *cmd, const char *flag, const char *value) r = command (NULL, &err, cmd, flag, value, NULL); if (r == -1) { - reply_with_error ("%s: %s: %s", cmd, value, err); + reply_with_error ("%s: %s", value, err); free (err); return -1; } |