Remove display and geometry command line options
authorHugo Villeneuve <hugo@hugovil.com>
Sat, 20 Jul 2013 14:53:06 +0000 (10:53 -0400)
committerHugo Villeneuve <hugo@hugovil.com>
Sat, 27 Jul 2013 03:46:15 +0000 (23:46 -0400)
These options are never used for a dockapp as WindowMaker
position our dockapp automatically in the dock area.

As for the display option, the same can be achieved using
the DISPLAY environment variable.

doc/hvclock.man
src/dockapp.c
src/dockapp.h
src/hvclock.c
src/hvclock.h
src/options.c

index eb28fad..a6a4b6d 100644 (file)
@@ -20,17 +20,6 @@ clock view, single-click on the clock or calendar image.
 \fB\-d\fR
 display debugging messages.
 
-.TP
-.BI \-display " host" : display
-specifies the host and screen to be used by \fBhvclock\fR. By default this
-is obtained from the environment variable
-.SB DISPLAY.
-
-.TP
-.BI \-geometry " geometry"
-.RB ( *geometry )
-specifies the initial geometry of the window.
-
 .TP
 \fB\-h\fR
 display usage and exit
index d7e268e..df61e5e 100644 (file)
@@ -122,7 +122,7 @@ dockapp_set_mask(GtkWidget *iconwin, char **xpm)
  * New window creation and initialization for a Dockable Application
  ******************************************************************************/
 void
-dockapp_init(int argc, char *argv[], char *display_arg, char *geometry_arg)
+dockapp_init(int argc, char *argv[])
 {
        GtkWidget *dockwin;
        GtkWidget *iconwin;
index 5ae2ac5..28dec5e 100644 (file)
@@ -25,7 +25,7 @@
 #include <gdk/gdkx.h>
 
 void
-dockapp_init(int argc, char *argv[], char *display_arg, char *geometry_arg);
+dockapp_init(int argc, char *argv[]);
 
 void
 dockapp_set_mask(GtkWidget *iconwin, char **xpm);
index 435bd6d..d4dfae7 100644 (file)
@@ -51,8 +51,7 @@ int main(int argc, char *argv[])
        ParseCommandLineOptions(argc, argv);
 
        /* Initializing and creating a DockApp window. */
-       dockapp_init(argc, argv, hvclock_infos.display_arg,
-                    hvclock_infos.geometry_arg);
+       dockapp_init(argc, argv);
 
        gtk_main();
 
index 22ca15a..074bd58 100644 (file)
@@ -25,8 +25,6 @@ struct hvclock_t {
        int debug;
        int show_seconds;
        int dockapp_mode;
-       char *display_arg;
-       char *geometry_arg;
 };
 
 /* Exported variables */
index bcf1d52..0b305e7 100644 (file)
@@ -44,9 +44,7 @@ static void DisplayUsage(void)
        printf("Usage: %s [OPTIONS]...\n", PACKAGE);
        printf("Dockable analog clock and calendar.\n\n");
        printf
-           ("  -d                        Display debugging messages.\n");
-       printf("  -display <host:display>   X display name\n");
-       printf("  -geometry +XPOS+YPOS      initial window position\n");
+           ("  -d                        display debugging messages.\n");
        printf("  -h                        display this help and exit\n");
        printf("  -n                        disable dockapp mode\n");
        printf("  -s                        display seconds hand\n");
@@ -84,8 +82,6 @@ void ParseCommandLineOptions(int argc, char *argv[])
 {
        int i;
        char *token;
-       int display_on = false;
-       int geometry_on = false;
 
        /* Default values. */
        hvclock_infos.debug = false;
@@ -99,14 +95,7 @@ void ParseCommandLineOptions(int argc, char *argv[])
                        /* Processing options names */
                        switch (token[1]) {
                        case 'd':
-                               if (STREQ("display", &token[1]))
-                                       display_on = true;
-                               else if (strlen(&token[1]) == 1)
-                                       hvclock_infos.debug = true;
-                               break;
-                       case 'g':
-                               if (STREQ("geometry", &token[1]))
-                                       geometry_on = true;
+                               hvclock_infos.debug = true;
                                break;
                        case 'h':
                                DisplayUsage();
@@ -131,21 +120,8 @@ void ParseCommandLineOptions(int argc, char *argv[])
                        break;
                default:
                        /* Processing options arguments */
-                       if (display_on != false) {
-                               display_on = false;
-                               hvclock_infos.display_arg = token;
-                       } else if (geometry_on != false) {
-                               geometry_on = false;
-                               hvclock_infos.geometry_arg = token;
-                       } else {
-                               InvalidOption("invalid option", token);
-                       }
+                       InvalidOption("invalid option", token);
                        break;
                }               /* end switch( token[0] ) */
        }                       /* end for */
-
-       if (display_on != false)
-               InvalidOption("missing display parameter", NULL);
-       else if (geometry_on != false)
-               InvalidOption("missing geometry parameter", NULL);
 }