diff options
author | Christophe Fergeau <cfergeau@redhat.com> | 2011-07-19 13:26:05 +0200 |
---|---|---|
committer | Christophe Fergeau <cfergeau@redhat.com> | 2011-07-19 13:31:10 +0200 |
commit | b8a1de32749fe536b331ca69c04e4d9b85e7e30d (patch) | |
tree | 770f0cd547f1fddb772fa1220d723b7d697c8ab2 | |
parent | c4d6f9791a7c5664843ad08bbb674b96d6dee594 (diff) | |
download | spice-b8a1de32749fe536b331ca69c04e4d9b85e7e30d.tar.gz spice-b8a1de32749fe536b331ca69c04e4d9b85e7e30d.tar.xz spice-b8a1de32749fe536b331ca69c04e4d9b85e7e30d.zip |
client: add missing "," in option list
In commit 44073d1b38e2 - client: improve WAN option description
one "," was missing at the end of the line. Since the next argument
was a string too, gcc silently concatenated them, and thanks to C++
polymorphic functions, the compiler didn't complain about the
missing argument, so it went unnoticed.
The effects are pretty bad though, since it prevents spicec from
running because it thinks command line parsing fails.
-rw-r--r-- | client/application.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/client/application.cpp b/client/application.cpp index 18101a59..5ebcce7f 100644 --- a/client/application.cpp +++ b/client/application.cpp @@ -2276,7 +2276,7 @@ bool Application::process_cmd_line(int argc, char** argv, bool &full_screen) parser.add(SPICE_OPT_CANVAS_TYPE, "canvas-type", "set rendering canvas", "canvas_type", true); parser.set_multi(SPICE_OPT_CANVAS_TYPE, ','); - parser.add(SPICE_OPT_DISPLAY_COLOR_DEPTH, "color-depth", "guest display color depth (if supported by the guest vdagent)" + parser.add(SPICE_OPT_DISPLAY_COLOR_DEPTH, "color-depth", "guest display color depth (if supported by the guest vdagent)", "16/32", true); parser.add(SPICE_OPT_DISABLE_DISPLAY_EFFECTS, "disable-effects", |