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 /src | |
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 'src')
-rwxr-xr-x | src/generator.ml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/generator.ml b/src/generator.ml index 9cd588be..6f071add 100755 --- a/src/generator.ml +++ b/src/generator.ml @@ -5810,7 +5810,7 @@ and generate_daemon_actions () = pr " memset (&args, 0, sizeof args);\n"; pr "\n"; pr " if (!xdr_guestfs_%s_args (xdr_in, &args)) {\n" name; - pr " reply_with_error (\"%%s: daemon failed to decode procedure arguments\", \"%s\");\n" name; + pr " reply_with_error (\"daemon failed to decode procedure arguments\");\n"; pr " return;\n"; pr " }\n"; let pr_args n = |