Changes in message.c [2354e9a:259e60a8]

File:

• message.c (modified) (21 diffs)

message.c

@@ +1 @@
+#include <stdlib.h>
+#include <unistd.h>
+#include <string.h>
+#include <sys/socket.h>
+#include <netdb.h>
+#include <sys/types.h>
+#include <sys/socket.h>
+#include <netinet/in.h>
+#include <arpa/inet.h>
+#include <time.h>
 #include "owl.h"
 #include "filterproc.h"
-#include <sys/socket.h>
-#include <arpa/inet.h>
 
 static owl_fmtext_cache fmtext_cache[OWL_FMTEXT_CACHE_SIZE];
@@ -34 +42 @@
   m->delete=0;
 
-#ifdef HAVE_LIBZEPHYR
-  m->has_notice = false;
-#endif
-
   owl_message_set_hostname(m, "");
-  m->attributes = g_ptr_array_new();
+  owl_list_create(&(m->attributes));
   
   /* save the time */
-  m->time = time(NULL);
-  m->timestr = g_strdup(ctime(&m->time));
-  m->timestr[strlen(m->timestr)-1] = '\0';
+  m->time=time(NULL);
+  m->timestr=g_strdup(ctime(&(m->time)));
+  m->timestr[strlen(m->timestr)-1]='\0';
 
   m->fmtext = NULL;
@@ -54 +58 @@
 void owl_message_set_attribute(owl_message *m, const char *attrname, const char *attrvalue)
 {
-  int i;
+  int i, j;
   owl_pair *p = NULL, *pair = NULL;
 
@@ -60 +64 @@
 
   /* look for an existing pair with this key, */
-  for (i = 0; i < m->attributes->len; i++) {
-    p = m->attributes->pdata[i];
+  j=owl_list_get_size(&(m->attributes));
+  for (i=0; i<j; i++) {
+    p=owl_list_get_element(&(m->attributes), i);
     if (owl_pair_get_key(p) == attrname) {
       g_free(owl_pair_get_value(p));
@@ -72 +77 @@
     pair = g_new(owl_pair, 1);
     owl_pair_create(pair, attrname, NULL);
-    g_ptr_array_add(m->attributes, pair);
+    owl_list_append_element(&(m->attributes), pair);
   }
   owl_pair_set_value(pair, owl_validate_or_convert(attrvalue));
@@ -82 +87 @@
 const char *owl_message_get_attribute_value(const owl_message *m, const char *attrname)
 {
-  int i;
+  int i, j;
   owl_pair *p;
   GQuark quark;
@@ -92 +97 @@
   attrname = g_quark_to_string(quark);
 
-  for (i = 0; i < m->attributes->len; i++) {
-    p = m->attributes->pdata[i];
+  j=owl_list_get_size(&(m->attributes));
+  for (i=0; i<j; i++) {
+    p=owl_list_get_element(&(m->attributes), i);
     if (owl_pair_get_key(p) == attrname) {
       return(owl_pair_get_value(p));
@@ -112 +118 @@
  */
 void owl_message_attributes_tofmtext(const owl_message *m, owl_fmtext *fm) {
-  int i;
+  int i, j;
   owl_pair *p;
   char *buff, *tmpbuff;
@@ -118 +124 @@
   owl_fmtext_init_null(fm);
 
-  for (i = 0; i < m->attributes->len; i++) {
-    p = m->attributes->pdata[i];
-
-    buff = g_strdup(owl_pair_get_value(p));
-    if (buff) {
-      tmpbuff = owl_text_indent(buff, 19, false);
-      g_free(buff);
-      buff = g_strdup_printf("  %-15.15s: %s\n", owl_pair_get_key(p), tmpbuff);
-      g_free(tmpbuff);
-    }
+  j=owl_list_get_size(&(m->attributes));
+  for (i=0; i<j; i++) {
+    p=owl_list_get_element(&(m->attributes), i);
+
+    tmpbuff = g_strdup(owl_pair_get_value(p));
+    g_strdelimit(tmpbuff, "\n", '~');
+    g_strdelimit(tmpbuff, "\r", '!');
+    buff = g_strdup_printf("  %-15.15s: %s\n", owl_pair_get_key(p), tmpbuff);
+    g_free(tmpbuff);
 
     if(buff == NULL) {
@@ -347 +352 @@
 }
 
-CALLER_OWN char *owl_message_format_time(const owl_message *m)
-{
-  return owl_util_format_time(localtime(&m->time));
-}
-
 void owl_message_set_type_admin(owl_message *m)
 {
@@ -500 +500 @@
 const ZNotice_t *owl_message_get_notice(const owl_message *m)
 {
-  return m->has_notice ? &m->notice : NULL;
+  return(&(m->notice));
 }
 #else
@@ -580 +580 @@
 
 /* caller must free return value. */
-CALLER_OWN char *owl_message_get_cc(const owl_message *m)
+char *owl_message_get_cc(const owl_message *m)
 {
   const char *cur;
@@ -597 +597 @@
 
 /* caller must free return value */
-CALLER_OWN GList *owl_message_get_cc_without_recipient(const owl_message *m)
+GList *owl_message_get_cc_without_recipient(const owl_message *m)
 {
   char *cc, *shortuser, *recip;
@@ -791 +791 @@
   /* first save the full notice */
   m->notice = *n;
-  m->has_notice = true;
 
   /* a little gross, we'll replace \r's with ' ' for now */
@@ -798 +797 @@
   /* save the time, we need to nuke the string saved by message_init */
   if (m->timestr) g_free(m->timestr);
-  m->time = n->z_time.tv_sec;
-  m->timestr = g_strdup(ctime(&m->time));
-  m->timestr[strlen(m->timestr)-1] = '\0';
+  m->time=n->z_time.tv_sec;
+  m->timestr=g_strdup(ctime(&(m->time)));
+  m->timestr[strlen(m->timestr)-1]='\0';
 
   /* set other info */
@@ -836 +835 @@
 
   
-  /* set the "isprivate" attribute if it's a private zephyr. */
-  if (owl_zwrite_recip_is_personal(n->z_recipient)) {
+  /* set the "isprivate" attribute if it's a private zephyr.
+   ``private'' means recipient is non-empty and doesn't start wit 
+   `@' */
+  if (*n->z_recipient && *n->z_recipient != '@') {
     owl_message_set_isprivate(m);
   }
@@ -875 +876 @@
   if (owl_global_is_zcrypt(&g) && !strcasecmp(n->z_opcode, "crypt")) {
     const char *argv[] = {
-      NULL,
+      "zcrypt",
       "-D",
       "-c", owl_message_get_class(m),
@@ -881 +882 @@
       NULL
     };
-    char *out = NULL;
+    char *out;
     int rv;
     int status;
     char *zcrypt;
 
-    zcrypt = g_build_filename(owl_get_bindir(), "zcrypt", NULL);
-    argv[0] = zcrypt;
-
-    rv = call_filter(argv, owl_message_get_body(m), &out, &status);
+    zcrypt = g_strdup_printf("%s/zcrypt", owl_get_bindir());
+
+    rv = call_filter(zcrypt, argv, owl_message_get_body(m), &out, &status);
     g_free(zcrypt);
 
@@ -898 +898 @@
       }
       owl_message_set_body(m, out);
-    } else {
-      /* Replace the opcode. Otherwise the UI and other bits of code think the
-       * message was encrypted. */
-      owl_message_set_opcode(m, "failed-decrypt");
+      g_free(out);
+    } else if(out) {
+      g_free(out);
     }
-    g_free(out);
   }
 
@@ -1006 +1004 @@
 void owl_message_cleanup(owl_message *m)
 {
-  int i;
+  int i, j;
   owl_pair *p;
 #ifdef HAVE_LIBZEPHYR    
-  if (m->has_notice) {
+  if (owl_message_is_type_zephyr(m) && owl_message_is_direction_in(m)) {
     ZFreeNotice(&(m->notice));
   }
@@ -1016 +1014 @@
 
   /* free all the attributes */
-  for (i = 0; i < m->attributes->len; i++) {
-    p = m->attributes->pdata[i];
+  j=owl_list_get_size(&(m->attributes));
+  for (i=0; i<j; i++) {
+    p=owl_list_get_element(&(m->attributes), i);
     g_free(owl_pair_get_value(p));
     g_free(p);
   }
 
-  g_ptr_array_free(m->attributes, true);
+  owl_list_cleanup(&(m->attributes), NULL);
  
   owl_message_invalidate_format(m);