Fixed some compilation errors
[modest] / src / modest-text-utils.c
index e1f82cc..db6d708 100644 (file)
@@ -48,6 +48,7 @@
 #define SENT_STRING _("Sent:")
 #define TO_STRING _("To:")
 #define        SUBJECT_STRING _("Subject:")
+#define EMPTY_STRING ""
 
 /*
  * we need these regexps to find URLs in plain text e-mails
@@ -121,7 +122,6 @@ modest_text_utils_quote (const gchar *text,
 
        g_return_val_if_fail (text, NULL);
        g_return_val_if_fail (content_type, NULL);
-       g_return_val_if_fail (from, NULL);
 
        cited = cite (sent_date, from);
        
@@ -148,7 +148,6 @@ modest_text_utils_cite (const gchar *text,
 
        g_return_val_if_fail (text, NULL);
        g_return_val_if_fail (content_type, NULL);
-       g_return_val_if_fail (from, NULL);
 
        tmp = cite (sent_date, from);
        retval = g_strdup_printf ("%s%s\n", tmp, text);
@@ -178,9 +177,7 @@ modest_text_utils_inline (const gchar *text,
 
        g_return_val_if_fail (text, NULL);
        g_return_val_if_fail (content_type, NULL);
-       g_return_val_if_fail (from, NULL);
        g_return_val_if_fail (text, NULL);
-       g_return_val_if_fail (to, NULL);
        
        modest_text_utils_strftime (sent_str, 100, "%c", sent_date);
 
@@ -193,10 +190,10 @@ modest_text_utils_inline (const gchar *text,
 
        return g_strdup_printf (format, 
                                FORWARD_STRING,
-                               FROM_STRING, from,
+                               FROM_STRING, (from) ? from : EMPTY_STRING,
                                SENT_STRING, sent_str,
-                               TO_STRING, to,
-                               SUBJECT_STRING, subject,
+                               TO_STRING, (to) ? to : EMPTY_STRING,
+                               SUBJECT_STRING, (subject) ? subject : EMPTY_STRING,
                                text);
 }
 
@@ -206,17 +203,11 @@ modest_text_utils_inline (const gchar *text,
 gsize
 modest_text_utils_strftime(char *s, gsize max, const char *fmt, time_t timet)
 {
-       /* only since Gtk 2.10
-        *
-        *static GDate date;
-        *g_date_set_time_t (&date, timet);
-        *return g_date_strftime (s, max, fmt, (const GDate*) &date);
-        */
+       static GDate date;
 
-       struct tm *time_tm;
-       time_tm = localtime (&timet);
-       
-       return strftime (s, max, fmt, time_tm);
+       /* FIXME: deprecated in newer glib versions */
+       g_date_set_time (&date, (GTime) timet);
+       return g_date_strftime (s, max, fmt, (const GDate*) &date);
 }
 
 gchar *
@@ -493,7 +484,9 @@ cite (const time_t sent_date, const gchar *from)
 
        /* format sent_date */
        modest_text_utils_strftime (sent_str, 100, "%c", sent_date);
-       return g_strdup_printf (N_("On %s, %s wrote:\n"), sent_str, from);
+       return g_strdup_printf (N_("On %s, %s wrote:\n"), 
+                               sent_str, 
+                               (from) ? from : EMPTY_STRING);
 }
 
 
@@ -600,9 +593,11 @@ get_url_matches (GString *txt)
 
        /* initalize the regexps */
        for (i = 0; i != pattern_num; ++i) {
-               patterns[i].preg = g_new0 (regex_t,1);
-               g_assert(regcomp (patterns[i].preg, patterns[i].regex,
-                                 REG_ICASE|REG_EXTENDED|REG_NEWLINE) == 0);
+               patterns[i].preg = g_slice_new0 (regex_t);
+
+               /* this should not happen */
+               g_return_val_if_fail (regcomp (patterns[i].preg, patterns[i].regex,
+                                              REG_ICASE|REG_EXTENDED|REG_NEWLINE) == 0, NULL);
        }
         /* find all the matches */
        for (i = 0; i != pattern_num; ++i) {
@@ -610,7 +605,7 @@ get_url_matches (GString *txt)
                while (1) {
                        int test_offset;
                        if ((rv = regexec (patterns[i].preg, txt->str + offset, 1, &rm, 0)) != 0) {
-                               g_assert (rv == REG_NOMATCH); /* this should not happen */
+                               g_return_val_if_fail (rv == REG_NOMATCH, NULL); /* this should not happen */
                                break; /* try next regexp */ 
                        }
                        if (rm.rm_so == -1)
@@ -624,7 +619,7 @@ get_url_matches (GString *txt)
                        
                        /* make a list of our matches (<offset, len, prefix> tupels)*/
                        if (test_offset != -1) {
-                               url_match_t *match = g_new (url_match_t,1);
+                               url_match_t *match = g_slice_new (url_match_t);
                                match->offset = offset + rm.rm_so;
                                match->len    = rm.rm_eo - rm.rm_so;
                                match->prefix = patterns[i].prefix;
@@ -636,7 +631,7 @@ get_url_matches (GString *txt)
 
        for (i = 0; i != pattern_num; ++i) {
                regfree (patterns[i].preg);
-               g_free  (patterns[i].preg);
+               g_slice_free  (regex_t, patterns[i].preg);
        } /* don't free patterns itself -- it's static */
        
        /* now sort the list, so the matches are in reverse order of occurence.
@@ -665,7 +660,8 @@ hyperlinkify_plain_text (GString *txt)
 
                /* the prefix is NULL: use the one that is already there */
                repl = g_strdup_printf ("<a href=\"%s%s\">%s</a>",
-                                       match->prefix ? match->prefix : "", url, url);
+                                       match->prefix ? match->prefix : EMPTY_STRING, 
+                                       url, url);
 
                /* replace the old thing with our hyperlink
                 * replacement thing */
@@ -675,7 +671,7 @@ hyperlinkify_plain_text (GString *txt)
                g_free (url);
                g_free (repl);
 
-               g_free (cursor->data);  
+               g_slice_free (url_match_t, match);      
        }
        
        g_slist_free (match_list);
@@ -767,48 +763,25 @@ modest_text_utils_utf8_strcmp (const gchar* s1, const gchar *s2, gboolean insens
        return result;
 }
 
-static GHashTable*
-get_display_date_cache (void)
-{
-       return modest_cache_mgr_get_cache (modest_runtime_get_cache_mgr (),
-                                          MODEST_CACHE_MGR_CACHE_TYPE_DATE_STRING);
-}
-
 
-
-const gchar*
+gchar*
 modest_text_utils_get_display_date (time_t date)
 {
-       static GHashTable *date_cache = NULL;
-
        time_t now;
        const guint BUF_SIZE = 64; 
        gchar date_buf[BUF_SIZE];  
        gchar now_buf [BUF_SIZE];  
-       gchar* cached_val;
-       
-       if (G_UNLIKELY(!date_cache))
-               date_cache = get_display_date_cache ();
        
-       cached_val = g_hash_table_lookup (date_cache, &date);
-       if (cached_val)
-               return cached_val;
-                                                   
        now = time (NULL);
-       
-       /* get today's date */
-       modest_text_utils_strftime (date_buf, BUF_SIZE, "%x", date);
-       modest_text_utils_strftime (now_buf,  BUF_SIZE, "%x",  now);
-       /* today */
 
+       modest_text_utils_strftime (date_buf, BUF_SIZE, "%x", date);
+       modest_text_utils_strftime (now_buf,  BUF_SIZE, "%x",  now); /* today */
+       
        /* if this is today, get the time instead of the date */
        if (strcmp (date_buf, now_buf) == 0)
                modest_text_utils_strftime (date_buf, BUF_SIZE, _("%X"), date);
-
-       cached_val = g_strdup(date_buf);
-       g_hash_table_insert (date_cache, (gpointer)&date, (gpointer)cached_val);
        
-       return cached_val;
+       return g_strdup(date_buf);
 }
 
 gboolean