2009-08-10 Marcus Brinkmann <marcus@g10code.de>
authorMarcus Brinkmann <mb@g10code.com>
Mon, 10 Aug 2009 15:27:56 +0000 (15:27 +0000)
committerMarcus Brinkmann <mb@g10code.com>
Mon, 10 Aug 2009 15:27:56 +0000 (15:27 +0000)
* pinentry/pinentry.c [!ENABLE_ENHANCED]: Remove option enhanced.
* pinentry/pinentry.h (struct pinentry) [!ENABLE_ENHANCED]: Remove
member enhanced.
* gtk+-2/pinentry-gtk-2.c: New macro.
(button_clicked, create_window) [!ENABLE_ENHANCED]: Disable code.
* gtk/pinentry-gtk.c (button_clicked, create_window): Likewise.
* doc/pinentry.texi (Using pinentry): Comment out enhanced option.

ChangeLog
doc/pinentry.texi
gtk+-2/pinentry-gtk-2.c
gtk/pinentry-gtk.c
pinentry/pinentry.c
pinentry/pinentry.h

index 7dfe649..576915d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2009-08-10  Marcus Brinkmann  <marcus@g10code.de>
+
+       * pinentry/pinentry.c [!ENABLE_ENHANCED]: Remove option enhanced.
+       * pinentry/pinentry.h (struct pinentry) [!ENABLE_ENHANCED]: Remove
+       member enhanced.
+       * gtk+-2/pinentry-gtk-2.c: New macro.
+       (button_clicked, create_window) [!ENABLE_ENHANCED]: Disable code.
+       * gtk/pinentry-gtk.c (button_clicked, create_window): Likewise.
+       * doc/pinentry.texi (Using pinentry): Comment out enhanced option.
+
 2009-07-10  Till Adam  <till@kdab.com>
 
        * qt4/pinentrydialog.cpp (PinEntryDialog): Redesign the dialog to
index 6da07ca..3f00d7a 100644 (file)
@@ -167,12 +167,12 @@ Print a usage message summarizing the most useful command line options.
 Turn on some debugging.  Mostly useful for the maintainers.  Note that
 this may reveal sensitive information like the entered passphrase.
 
-@item --enhanced
-@itemx -e
-@opindex enhanced
-@opindex e
-Ask for timeouts and insurance, too.  Note that this is currently not
-fully supported.
+@c @item --enhanced
+@c @itemx -e
+@c @opindex enhanced
+@c @opindex e
+@c Ask for timeouts and insurance, too.  Note that this is currently not
+@c fully supported.
 
 @item --no-global-grab
 @itemx -g
index 304a06c..602abd3 100644 (file)
@@ -163,12 +163,15 @@ button_clicked (GtkWidget *widget, gpointer data)
       const char *s;
 
       /* Okay button or enter used in text field.  */
-
+#ifdef ENABLE_ENHANCED
+      /* FIXME: This is not compatible with assuan.  We can't just
+        print stuff on stdout.  */
       if (pinentry->enhanced)
        printf ("Options: %s\nTimeout: %d\n\n",
                gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (insure))
                ? "insure" : "",
                gtk_spin_button_get_value_as_int (GTK_SPIN_BUTTON (time_out)));
+#endif
 
       s = gtk_secure_entry_get_text (GTK_SECURE_ENTRY (entry));
       if (!s)
@@ -396,6 +399,7 @@ create_window (int confirm_mode)
                            GTK_EXPAND|GTK_FILL, GTK_EXPAND|GTK_FILL, 0, 0);
        }
 
+#ifdef ENABLE_ENHANCED
       if (pinentry->enhanced)
        {
          GtkWidget *sbox = gtk_hbox_new (FALSE, HIG_SMALL);
@@ -421,6 +425,7 @@ create_window (int confirm_mode)
          gtk_box_pack_start (GTK_BOX (box), insure, FALSE, FALSE, 0);
          gtk_widget_show (insure);
        }
+#endif
     }
 
   bbox = gtk_hbutton_box_new ();
index 80ee984..00bd531 100644 (file)
@@ -136,6 +136,7 @@ button_clicked (GtkWidget *widget, gpointer data)
       char *s_utf8;
       char *s_buffer;
 
+#ifdef ENABLE_ENHANCED
       if (pinentry->enhanced)
         {
           printf("Options: %s\nTimeout: %d\n\n",
@@ -144,6 +145,7 @@ button_clicked (GtkWidget *widget, gpointer data)
                  : "",
                  gtk_spin_button_get_value_as_int(GTK_SPIN_BUTTON(time_out)));
         }
+#endif
 
       pinentry->locale_err = 1;
       s = gtk_secure_entry_get_text (GTK_SECURE_ENTRY(entry));
@@ -293,6 +295,7 @@ create_window (int confirm_mode)
       gtk_widget_grab_focus (entry);
       gtk_widget_show (entry);
 
+#ifdef ENABLE_ENHANCED
       if (pinentry->enhanced)
         {
           sbox = gtk_hbox_new(FALSE, 5);
@@ -318,6 +321,7 @@ create_window (int confirm_mode)
           gtk_box_pack_start (GTK_BOX(box), insure, FALSE, FALSE, 0);
           gtk_widget_show (insure);
         }
+#endif
     }
 
 
index 6fedf91..9125c35 100644 (file)
@@ -63,7 +63,9 @@ struct pinentry pinentry =
     0,         /* TTY LC_CTYPE.  */
     0,         /* TTY LC_MESSAGES.  */
     0,         /* Debug mode.  */
+#ifdef ENABLE_ENHANCED
     0,         /* Enhanced mode.  */
+#endif
     1,         /* Global grab.  */
     0,         /* Parent Window ID.  */
     NULL,       /* Touch file.  */
@@ -396,7 +398,9 @@ usage (void)
 "      --ttytype NAME    Set the tty terminal type\n"
 "      --lc-ctype        Set the tty LC_CTYPE value\n"
 "      --lc-messages     Set the tty LC_MESSAGES value\n"
+#ifdef ENABLE_ENHANCED
 "  -e, --enhanced        Ask for timeout and insurance, too\n"
+#endif
 "  -g, --no-global-grab  Grab keyboard only while window is focused\n"
 "      --parent-wid     Parent window ID (for positioning)\n"
 "  -d, --debug           Turn on debugging output\n"
@@ -471,7 +475,9 @@ pinentry_parse_opts (int argc, char *argv[])
      { "ttytype", required_argument,     0, 'N' },
      { "lc-ctype", required_argument,    0, 'C' },
      { "lc-messages", required_argument, 0, 'M' },
+#ifdef ENABLE_ENHANCED
      { "enhanced", no_argument,          0, 'e' },
+#endif
      { "no-global-grab", no_argument,    0, 'g' },
      { "parent-wid", required_argument,  0, 'W' },
      { "colors", required_argument,     0, 'c' },
@@ -489,9 +495,11 @@ pinentry_parse_opts (int argc, char *argv[])
         case 'd':
           pinentry.debug = 1;
           break;
+#ifdef ENABLE_ENHANCED
         case 'e':
           pinentry.enhanced = 1;
           break;
+#endif
         case 'g':
           pinentry.grab = 0;
           break;
index cc9ee28..a0b243c 100644 (file)
@@ -28,6 +28,8 @@ extern "C" {
 #endif
 #endif
 
+#undef ENABLE_ENHANCED
+
 typedef enum {
   PINENTRY_COLOR_NONE, PINENTRY_COLOR_DEFAULT,
   PINENTRY_COLOR_BLACK, PINENTRY_COLOR_RED,
@@ -70,8 +72,12 @@ struct pinentry
 
   /* True if debug mode is requested.  */
   int debug;
+
+#ifdef ENABLE_ENHANCED
   /* True if enhanced mode is requested.  */
   int enhanced;
+#endif
+
   /* True if caller should grab the keyboard.  */
   int grab;
   /* The window ID of the parent window over which the pinentry window