X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=hook.c;h=d665cb38e9fedac95a54db37b2d56ef669ab74fa;hp=0a7ce546b68ee01cc7aa920b83b44db418430961;hb=8e037c67a88cb4680c4391134c578e3b55a80f8a;hpb=c565d2f14f1655a6483171e1636e5803533b6ebe

diff --git a/hook.c b/hook.c
index 0a7ce54..d665cb3 100644
--- a/hook.c
+++ b/hook.c
@@ -11,6 +11,8 @@
 # include "config.h"
 #endif
 
+#include <lib-lib/mem.h>
+
 #include "mutt.h"
 #include "buffer.h"
 #include "mx.h"
@@ -89,7 +91,7 @@ int mutt_parse_hook (BUFFER * buf, BUFFER * s, unsigned long data,
   if (data & (M_FOLDERHOOK | M_MBOXHOOK)) {
     strfcpy (path, pattern.data, sizeof (path));
     _mutt_expand_path (path, sizeof (path), 1);
-    mem_free (&pattern.data);
+    p_delete(&pattern.data);
     memset (&pattern, 0, sizeof (pattern));
     pattern.data = str_dup (path);
   }
@@ -108,7 +110,7 @@ int mutt_parse_hook (BUFFER * buf, BUFFER * s, unsigned long data,
 
     strfcpy (tmp, pattern.data, sizeof (tmp));
     mutt_check_simple (tmp, sizeof (tmp), DefaultHook);
-    mem_free (&pattern.data);
+    p_delete(&pattern.data);
     memset (&pattern, 0, sizeof (pattern));
     pattern.data = str_dup (tmp);
   }
@@ -116,7 +118,7 @@ int mutt_parse_hook (BUFFER * buf, BUFFER * s, unsigned long data,
   if (data & (M_MBOXHOOK | M_SAVEHOOK | M_FCCHOOK)) {
     strfcpy (path, command.data, sizeof (path));
     mutt_expand_path (path, sizeof (path));
-    mem_free (&command.data);
+    p_delete(&command.data);
     memset (&command, 0, sizeof (command));
     command.data = str_dup (path);
   }
@@ -132,8 +134,8 @@ int mutt_parse_hook (BUFFER * buf, BUFFER * s, unsigned long data,
          * pattern, so if we've already seen this pattern/command pair, just
          * ignore it instead of creating a duplicate */
         if (!str_cmp (ptr->command, command.data)) {
-          mem_free (&command.data);
-          mem_free (&pattern.data);
+          p_delete(&command.data);
+          p_delete(&pattern.data);
           return 0;
         }
       }
@@ -143,9 +145,9 @@ int mutt_parse_hook (BUFFER * buf, BUFFER * s, unsigned long data,
          * order of execution of the hooks, which i think is desirable since
          * a common action to perform is to change the default (.) entry
          * based upon some other information. */
-        mem_free (&ptr->command);
+        p_delete(&ptr->command);
         ptr->command = command.data;
-        mem_free (&pattern.data);
+        p_delete(&pattern.data);
         return 0;
       }
     }
@@ -163,7 +165,7 @@ int mutt_parse_hook (BUFFER * buf, BUFFER * s, unsigned long data,
       goto error;
   }
   else {
-    rx = mem_malloc (sizeof (regex_t));
+    rx = p_new(regex_t, 1);
 #ifdef M_CRYPTHOOK
     if ((rc =
          REGCOMP (rx, NONULL (pattern.data),
@@ -178,7 +180,7 @@ int mutt_parse_hook (BUFFER * buf, BUFFER * s, unsigned long data,
     {
       regerror (rc, rx, err->data, err->dsize);
       regfree (rx);
-      mem_free (&rx);
+      p_delete(&rx);
       goto error;
     }
   }
@@ -198,20 +200,20 @@ int mutt_parse_hook (BUFFER * buf, BUFFER * s, unsigned long data,
   return 0;
 
 error:
-  mem_free (&pattern.data);
-  mem_free (&command.data);
+  p_delete(&pattern.data);
+  p_delete(&command.data);
   return (-1);
 }
 
 static void delete_hook (HOOK * h)
 {
-  mem_free (&h->command);
-  mem_free (&h->rx.pattern);
+  p_delete(&h->command);
+  p_delete(&h->rx.pattern);
   if (h->rx.rx) {
     regfree (h->rx.rx);
   }
   mutt_pattern_free (&h->pattern);
-  mem_free (&h);
+  p_delete(&h);
 }
 
 /* Deletes all hooks of type ``type'', or all defined hooks if ``type'' is 0 */
@@ -292,7 +294,7 @@ void mutt_folder_hook (char *path)
           mutt_error ("%s", err.data);
           mutt_sleep (1);       /* pause a moment to let the user see the error */
           if (ERROR_STOP) {
-            mem_free (&token.data);
+            p_delete(&token.data);
             current_hook_type = 0;
             return;
           }
@@ -300,7 +302,7 @@ void mutt_folder_hook (char *path)
       }
     }
   }
-  mem_free (&token.data);
+  p_delete(&token.data);
 
   current_hook_type = 0;
 }
@@ -338,13 +340,13 @@ void mutt_message_hook (CONTEXT * ctx, HEADER * hdr, int type)
           mutt_error ("%s", err.data);
           mutt_sleep (1);
           if (ERROR_STOP) {
-            mem_free (&token.data);
+            p_delete(&token.data);
             current_hook_type = 0;
             return;
           }
         }
   }
-  mem_free (&token.data);
+  p_delete(&token.data);
   current_hook_type = 0;
 }
 
@@ -465,7 +467,7 @@ void mutt_account_hook (const char *url)
         mutt_error ("%s", err.data);
         mutt_sleep (1);
         if (ERROR_STOP) {
-          mem_free (&token.data);
+          p_delete(&token.data);
           current_hook_type = 0;
           return;
         }
@@ -473,6 +475,6 @@ void mutt_account_hook (const char *url)
     }
   }
 
-  mem_free (&token.data);
+  p_delete(&token.data);
 }
 #endif