p_clear should not be used with sizeof but *countof*

[apps/madmutt.git] / keymap.c

diff --git a/keymap.c b/keymap.c
index 44ccfd3..c7cede6 100644 (file)
--- a/keymap.c
+++ b/keymap.c
@@ -18,18 +18,20 @@
 #include <lib-lib/buffer.h>
 #include <lib-lib/mapping.h>
 
+#include <lib-ui/curses.h>
+
 #include "mutt.h"
 #include "mutt_menu.h"
-#include "mutt_curses.h"
 #include "keymap.h"
-#include "mutt_crypt.h"
+#include <lib-crypt/crypt.h>
 
 
 #include <stdlib.h>
 #include <string.h>
 #include <ctype.h>
 
-#include "functions.h"
+#define MUTT_FUNCTIONS_VALUES
+#include "functions.def"
 
 struct mapping_t Menus[] = {
   {"alias", MENU_ALIAS},
@@ -104,9 +106,9 @@ static struct keymap_t *allocKeys(int len, keycode_t *keys)
     return p;
 }
 
-static int parse_fkey (char *s)
+static int parse_fkey (const char *s)
 {
-  char *t;
+  const char *t;
   int n = 0;
 
   if (s[0] != '<' || ascii_tolower (s[1]) != 'f')
@@ -246,7 +248,7 @@ void km_bindkey (const char *s, int menu, int op)
   km_bind (s, menu, op, NULL, NULL);
 }
 
-static int get_op (struct binding_t *bindings, const char *start, size_t len)
+static int get_op (struct binding_t *bindings, const char *start, ssize_t len)
 {
   int i;
 
@@ -275,7 +277,7 @@ static const char *get_func (struct binding_t *bindings, int op)
 static void push_string(const char *s)
 {
   const char *pp, *p = s + m_strlen(s) - 1;
-  size_t l;
+  ssize_t l;
   int i, op = OP_NULL;
 
   while (p >= s) {
@@ -528,11 +530,8 @@ void km_init (void)
   create_bindings (OpAlias, MENU_ALIAS);
 
 
-  if ((WithCrypto & APPLICATION_PGP))
-    create_bindings (OpPgp, MENU_PGP);
-
-  if ((WithCrypto & APPLICATION_SMIME))
-    create_bindings (OpSmime, MENU_SMIME);
+  create_bindings (OpPgp, MENU_PGP);
+  create_bindings (OpSmime, MENU_SMIME);
 
 #ifdef CRYPT_BACKEND_GPGME
   create_bindings (OpPgp, MENU_KEY_SELECT_PGP);
@@ -654,7 +653,7 @@ void km_error_key (int menu)
   return;
 }
 
-int mutt_parse_push (BUFFER * buf, BUFFER * s, unsigned long data,
+int mutt_parse_push (BUFFER * buf, BUFFER * s, unsigned long data __attribute__ ((unused)),
                      BUFFER * err)
 {
   int r = 0;
@@ -754,7 +753,7 @@ struct binding_t *km_get_table (int menu)
     return OpQuery;
 
   case MENU_PGP:
-    return (WithCrypto & APPLICATION_PGP) ? OpPgp : NULL;
+    return OpPgp;
 
 #ifdef CRYPT_BACKEND_GPGME
   case MENU_KEY_SELECT_PGP:
@@ -773,7 +772,7 @@ struct binding_t *km_get_table (int menu)
 }
 
 /* bind menu-name '<key_sequence>' function-name */
-int mutt_parse_bind (BUFFER * buf, BUFFER * s, unsigned long data,
+int mutt_parse_bind (BUFFER * buf, BUFFER * s, unsigned long data __attribute__ ((unused)),
                      BUFFER * err)
 {
   struct binding_t *bindings = NULL;
@@ -817,7 +816,7 @@ int mutt_parse_bind (BUFFER * buf, BUFFER * s, unsigned long data,
 }
 
 /* macro <menu> <key> <macro> <description> */
-int mutt_parse_macro (BUFFER * buf, BUFFER * s, unsigned long data,
+int mutt_parse_macro (BUFFER * buf, BUFFER * s, unsigned long data __attribute__ ((unused)),
                       BUFFER * err)
 {
   int menu[sizeof (Menus) / sizeof (struct mapping_t) - 1], r =
@@ -864,7 +863,7 @@ int mutt_parse_macro (BUFFER * buf, BUFFER * s, unsigned long data,
 }
 
 /* exec function-name */
-int mutt_parse_exec (BUFFER * buf, BUFFER * s, unsigned long data,
+int mutt_parse_exec (BUFFER * buf, BUFFER * s, unsigned long data __attribute__ ((unused)),
                      BUFFER * err)
 {
   int ops[128];
@@ -896,7 +895,7 @@ int mutt_parse_exec (BUFFER * buf, BUFFER * s, unsigned long data,
     }
     nops++;
   }
-  while (MoreArgs (s) && nops < sizeof (ops) / sizeof (ops[0]));
+  while (MoreArgs (s) && nops < ssizeof (ops) / ssizeof (ops[0]));
 
   while (nops)
     mutt_ungetch (0, ops[--nops]);