Author: cazfi
Date: Sat Dec 13 23:42:45 2014
New Revision: 27287

URL: http://svn.gna.org/viewcvs/freeciv?rev=27287&view=rev
Log:
Sdl-, Xaw-, and Stub-client give an error about unknown gui-specific 
commandline option.
Gtk- and Qt-client cannot do that, so only a comment explaining the situation 
added.

See bug #23035

Modified:
    branches/S2_5/client/gui-gtk-2.0/gui_main.c
    branches/S2_5/client/gui-gtk-3.0/gui_main.c
    branches/S2_5/client/gui-qt/gui_main.cpp
    branches/S2_5/client/gui-sdl/gui_main.c
    branches/S2_5/client/gui-stub/gui_main.c
    branches/S2_5/client/gui-xaw/gui_main.c

Modified: branches/S2_5/client/gui-gtk-2.0/gui_main.c
URL: 
http://svn.gna.org/viewcvs/freeciv/branches/S2_5/client/gui-gtk-2.0/gui_main.c?rev=27287&r1=27286&r2=27287&view=diff
==============================================================================
--- branches/S2_5/client/gui-gtk-2.0/gui_main.c (original)
+++ branches/S2_5/client/gui-gtk-2.0/gui_main.c Sat Dec 13 23:42:45 2014
@@ -300,6 +300,8 @@
       print_usage(argv[0]);
       exit(EXIT_SUCCESS);
     }
+    /* Can't check against unknown options, as those might be gtk options */
+
     i++;
   }
 }

Modified: branches/S2_5/client/gui-gtk-3.0/gui_main.c
URL: 
http://svn.gna.org/viewcvs/freeciv/branches/S2_5/client/gui-gtk-3.0/gui_main.c?rev=27287&r1=27286&r2=27287&view=diff
==============================================================================
--- branches/S2_5/client/gui-gtk-3.0/gui_main.c (original)
+++ branches/S2_5/client/gui-gtk-3.0/gui_main.c Sat Dec 13 23:42:45 2014
@@ -285,6 +285,8 @@
       print_usage(argv[0]);
       exit(EXIT_SUCCESS);
     }
+    /* Can't check against unknown options, as those might be gtk options */
+
     i++;
   }
 }

Modified: branches/S2_5/client/gui-qt/gui_main.cpp
URL: 
http://svn.gna.org/viewcvs/freeciv/branches/S2_5/client/gui-qt/gui_main.cpp?rev=27287&r1=27286&r2=27287&view=diff
==============================================================================
--- branches/S2_5/client/gui-qt/gui_main.cpp    (original)
+++ branches/S2_5/client/gui-qt/gui_main.cpp    Sat Dec 13 23:42:45 2014
@@ -125,6 +125,8 @@
       print_usage();
       return false;
     }
+    // Can't check against unknown options, as those might be Qt options
+
     i++;
   }
 

Modified: branches/S2_5/client/gui-sdl/gui_main.c
URL: 
http://svn.gna.org/viewcvs/freeciv/branches/S2_5/client/gui-sdl/gui_main.c?rev=27287&r1=27286&r2=27287&view=diff
==============================================================================
--- branches/S2_5/client/gui-sdl/gui_main.c     (original)
+++ branches/S2_5/client/gui-sdl/gui_main.c     Sat Dec 13 23:42:45 2014
@@ -192,17 +192,20 @@
     if (is_option("--help", argv[i])) {
       print_usage(argv[0]);
       exit(EXIT_SUCCESS);
-    } else if (is_option("--fullscreen",argv[i])) {
+    } else if (is_option("--fullscreen", argv[i])) {
       gui_sdl_fullscreen = TRUE;
-    } else if (is_option("--eventthread",argv[i])) {
+    } else if (is_option("--eventthread", argv[i])) {
       /* init events in other thread ( only linux and BeOS ) */  
       SDL_InitSubSystem(SDL_INIT_EVENTTHREAD);
     } else if ((option = get_option_malloc("--theme", argv, &i, argc))) {
       sz_strlcpy(gui_sdl_default_theme_name, option);
-    }
+    } else {
+      fc_fprintf(stderr, _("Unrecognized option: \"%s\"\n"), argv[i]);
+      exit(EXIT_FAILURE);
+    }
+
     i++;
   }
-  
 }
 
 /**************************************************************************

Modified: branches/S2_5/client/gui-stub/gui_main.c
URL: 
http://svn.gna.org/viewcvs/freeciv/branches/S2_5/client/gui-stub/gui_main.c?rev=27287&r1=27286&r2=27287&view=diff
==============================================================================
--- branches/S2_5/client/gui-stub/gui_main.c    (original)
+++ branches/S2_5/client/gui-stub/gui_main.c    Sat Dec 13 23:42:45 2014
@@ -86,18 +86,18 @@
 **************************************************************************/
 static void parse_options(int argc, char **argv)
 {
-  int i;
-
-  i = 1;
-  while (i < argc)
-  {
-    if (is_option("--help", argv[i]))
-    {
+  int i = 1;
+
+  while (i < argc) {
+    if (is_option("--help", argv[i])) {
       print_usage(argv[0]);
       exit(EXIT_SUCCESS);
+    } else {
+      fc_fprintf(stderr, _("Unrecognized option: \"%s\"\n"), argv[i]);
+      exit(EXIT_FAILURE);
     }
-    /* PORTME */
-    i += 1;
+
+    i++;
   }
 }
 

Modified: branches/S2_5/client/gui-xaw/gui_main.c
URL: 
http://svn.gna.org/viewcvs/freeciv/branches/S2_5/client/gui-xaw/gui_main.c?rev=27287&r1=27286&r2=27287&view=diff
==============================================================================
--- branches/S2_5/client/gui-xaw/gui_main.c     (original)
+++ branches/S2_5/client/gui-xaw/gui_main.c     Sat Dec 13 23:42:45 2014
@@ -260,21 +260,22 @@
 }
 
 /**************************************************************************
-...
+  Parse and enact any client-specific options.
 **************************************************************************/
 static void parse_options(int argc, char **argv)
 {
-  int i;
-
-  i = 1;
-  while (i < argc)
-  {
-    if (is_option("--help", argv[i]))
-    {
+  int i = 1;
+
+  while (i < argc) {
+    if (is_option("--help", argv[i])) {
       print_usage(argv[0]);
       exit(EXIT_SUCCESS);
+    } else {
+      fc_fprintf(stderr, _("Unrecognized option: \"%s\"\n"), argv[i]);
+      exit(EXIT_FAILURE);
     }
-    i += 1;
+
+    i++;
   }
 }
 


_______________________________________________
Freeciv-commits mailing list
Freeciv-commits@gna.org
https://mail.gna.org/listinfo/freeciv-commits

Reply via email to