Changes in functions.c [697221f:47128d9]

File:

• functions.c (modified) (14 diffs)

functions.c

@@ -32 +32 @@
   char *rv;
   rv=owl_function_command(cmdbuff);
-  g_free(rv);
+  if (rv) g_free(rv);
 }
 
@@ -453 +453 @@
 
   if (rv || status) {
-    g_free(cryptmsg);
+    if(cryptmsg) g_free(cryptmsg);
     g_free(old_msg);
     owl_function_error("Error in zcrypt, possibly no key found.  Message not sent.");
@@ -991 +991 @@
   /* execute the commands in shutdown */
   ret = owl_perlconfig_execute("BarnOwl::Hooks::_shutdown();");
-  g_free(ret);
+  if (ret) g_free(ret);
 
   /* signal our child process, if any */
@@ -1817 +1817 @@
           owl_global_get_cols(&g));
 
-  if (has_colors()) {
+  if (owl_global_get_hascolors(&g)) {
     owl_fmtext_append_normal(&fm, "Color: Yes\n");
-    owl_fmtext_appendf_normal(&fm, "Number of color pairs: %i\n", owl_util_get_colorpairs());
+    owl_fmtext_appendf_normal(&fm, "Number of color pairs: %i\n", owl_global_get_colorpairs(&g));
     owl_fmtext_appendf_normal(&fm, "Can change colors: %s\n", can_change_color() ? "yes" : "no");
   } else {
@@ -2226 +2226 @@
     f = fl->data;
     owl_fmtext_append_normal(&fm, "   ");
-    owl_fmtext_append_normal_color(&fm, owl_filter_get_name(f),
-                                   owl_filter_get_fgcolor(f),
-                                   owl_filter_get_bgcolor(f));
+    if (owl_global_get_hascolors(&g)) {
+      owl_fmtext_append_normal_color(&fm, owl_filter_get_name(f), owl_filter_get_fgcolor(f), owl_filter_get_bgcolor(f));
+    } else {
+      owl_fmtext_append_normal(&fm, owl_filter_get_name(f));
+    }
     owl_fmtext_append_normal(&fm, "\n");
   }
@@ -2358 +2360 @@
 done:
   g_free(class);
-  g_free(instance);
+  if (instance) {
+    g_free(instance);
+  }
   return(filtname);
 }
@@ -2519 +2523 @@
 
   filtname = g_strdup_printf("conversation-%s", ccs);
-  g_strdelimit(filtname, " ", '-');
+  owl_text_tr(filtname, ' ', '-');
 
   if (owl_global_get_filter(&g, filtname)) {
@@ -2781 +2785 @@
 void owl_function_zpunt(const char *class, const char *inst, const char *recip, int direction)
 {
-  GPtrArray *argv;
+  GString *buf;
   char *quoted;
 
-  argv = g_ptr_array_new();
+  buf = g_string_new("");
   if (!strcmp(class, "*")) {
-    g_ptr_array_add(argv, g_strdup("class"));
-    g_ptr_array_add(argv, g_strdup(".*"));
+    g_string_append(buf, "class .*");
   } else {
     quoted=owl_text_quote(class, OWL_REGEX_QUOTECHARS, OWL_REGEX_QUOTEWITH);
-    g_ptr_array_add(argv, g_strdup("class"));
-    g_ptr_array_add(argv, g_strdup_printf("^(un)*%s(\\.d)*$", quoted));
+    owl_string_appendf_quoted(buf, "class ^(un)*%q(\\.d)*$", quoted);
     g_free(quoted);
   }
   if (!strcmp(inst, "*")) {
-    g_ptr_array_add(argv, g_strdup("and"));
-    g_ptr_array_add(argv, g_strdup("instance"));
-    g_ptr_array_add(argv, g_strdup(".*"));
+    g_string_append(buf, " and instance .*");
   } else {
     quoted=owl_text_quote(inst, OWL_REGEX_QUOTECHARS, OWL_REGEX_QUOTEWITH);
-    g_ptr_array_add(argv, g_strdup("and"));
-    g_ptr_array_add(argv, g_strdup("instance"));
-    g_ptr_array_add(argv, g_strdup_printf("^(un)*%s(\\.d)*$", quoted));
+    owl_string_appendf_quoted(buf, " and instance ^(un)*%q(\\.d)*$", quoted);
     g_free(quoted);
   }
   if (!strcmp(recip, "*")) {
-    /* nothing */
+    /* g_string_append(buf, ""); */
   } else {
     if(!strcmp(recip, "%me%")) {
@@ -2812 +2810 @@
     }
     quoted=owl_text_quote(recip, OWL_REGEX_QUOTECHARS, OWL_REGEX_QUOTEWITH);
-    g_ptr_array_add(argv, g_strdup("and"));
-    g_ptr_array_add(argv, g_strdup("recipient"));
-    g_ptr_array_add(argv, g_strdup_printf("^%s$", quoted));
+    owl_string_appendf_quoted(buf, " and recipient ^%q$", quoted);
     g_free(quoted);
   }
 
-  owl_function_punt(argv->len, (const char *const*) argv->pdata, direction);
-  g_ptr_array_foreach(argv, (GFunc)g_free, NULL);
-  g_ptr_array_free(argv, true);
-}
-
-void owl_function_punt(int argc, const char *const *argv, int direction)
+  owl_function_punt(buf->str, direction);
+  g_string_free(buf, true);
+}
+
+void owl_function_punt(const char *filter, int direction)
 {
   owl_filter *f;
@@ -2831 +2826 @@
 
   /* first, create the filter */
-  f = owl_filter_new("punt-filter", argc, argv);
+  owl_function_debugmsg("About to filter %s", filter);
+  f = owl_filter_new_fromstring("punt-filter", filter);
   if (f == NULL) {
     owl_function_error("Error creating filter for zpunt");
@@ -2858 +2854 @@
   }
 
-  if (direction == 0) {
-    owl_function_debugmsg("punting");
-    /* If we're punting, add the filter to the global punt list */
+  owl_function_debugmsg("punting");
+  /* If we're punting, add the filter to the global punt list */
+  if (direction==0) {
     owl_list_append_element(fl, f);
-  } else if (direction == 1) {
-    owl_function_makemsg("No matching punt filter");
- }
+  }
 }
 
@@ -2965 +2959 @@
   
   if (viewsize==0) {
-    owl_function_makemsg("No messages present");
+    owl_function_error("No messages present");
     return;
   }
@@ -2979 +2973 @@
   /* bounds check */
   if (start>=viewsize || start<0) {
-    owl_function_makemsg("No further matches found");
+    owl_function_error("No further matches found");
     return;
   }
@@ -3008 +3002 @@
   }
   owl_mainwin_redisplay(owl_global_get_mainwin(&g));
-  owl_function_makemsg("No matches found");
+  owl_function_error("No matches found");
 }