Fixed some compilation errors
[modest] / src / modest-text-utils.c
index 9d26aed..db6d708 100644 (file)
@@ -33,7 +33,9 @@
 #include <stdlib.h>
 #include <glib/gi18n.h>
 #include <regex.h>
-#include "modest-text-utils.h"
+#include <modest-tny-platform-factory.h>
+#include <modest-text-utils.h>
+#include <modest-runtime.h>
 
 
 #ifdef HAVE_CONFIG_H
@@ -46,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
@@ -119,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);
        
@@ -144,6 +146,9 @@ modest_text_utils_cite (const gchar *text,
 {
        gchar *tmp, *retval;
 
+       g_return_val_if_fail (text, NULL);
+       g_return_val_if_fail (content_type, NULL);
+
        tmp = cite (sent_date, from);
        retval = g_strdup_printf ("%s%s\n", tmp, text);
        g_free (tmp);
@@ -172,12 +177,9 @@ 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);
-       g_return_val_if_fail (subject, NULL);
        
-       modest_text_utils_strftime (sent_str, 100, "%c", localtime (&sent_date));
+       modest_text_utils_strftime (sent_str, 100, "%c", sent_date);
 
        if (!strcmp (content_type, "text/html"))
                /* TODO: extract the <body> of the HTML and pass it to
@@ -188,20 +190,24 @@ 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);
 }
 
 /* just to prevent warnings:
  * warning: `%x' yields only last 2 digits of year in some locales
  */
-size_t
-modest_text_utils_strftime(char *s, size_t max, const char  *fmt, const  struct tm *tm)
+gsize
+modest_text_utils_strftime(char *s, gsize max, const char *fmt, time_t timet)
 {
-       return strftime(s, max, fmt, tm);
+       static GDate date;
+
+       /* FIXME: deprecated in newer glib versions */
+       g_date_set_time (&date, (GTime) timet);
+       return g_date_strftime (s, max, fmt, (const GDate*) &date);
 }
 
 gchar *
@@ -477,8 +483,10 @@ cite (const time_t sent_date, const gchar *from)
        gchar sent_str[101];
 
        /* format sent_date */
-       modest_text_utils_strftime (sent_str, 100, "%c", localtime (&sent_date));
-       return g_strdup_printf (N_("On %s, %s wrote:\n"), sent_str, from);
+       modest_text_utils_strftime (sent_str, 100, "%c", sent_date);
+       return g_strdup_printf (N_("On %s, %s wrote:\n"), 
+                               sent_str, 
+                               (from) ? from : EMPTY_STRING);
 }
 
 
@@ -491,7 +499,6 @@ modest_text_utils_quote_plain_text (const gchar *text,
        gint indent, breakpoint, rem_indent = 0;
        GString *q, *l, *remaining;
        gsize len;
-       gchar *tmp;
 
        /* remaining will store the rest of the line if we have to break it */
        q = g_string_new (cite);
@@ -512,7 +519,7 @@ modest_text_utils_quote_plain_text (const gchar *text,
                        } else {
                                do {
                                        breakpoint =
-                                               get_breakpoint (remaining->     str,
+                                               get_breakpoint (remaining->str,
                                                                rem_indent,
                                                                limit);
                                        append_quoted (q, rem_indent,
@@ -586,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) {
@@ -596,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)
@@ -610,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;
@@ -622,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.
@@ -651,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 */
@@ -661,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);
@@ -737,17 +747,13 @@ modest_text_utils_utf8_strcmp (const gchar* s1, const gchar *s2, gboolean insens
        /* work even when s1 and/or s2 == NULL */
        if (G_UNLIKELY(s1 == s2))
                return 0;
-       if (G_UNLIKELY(!s1))
-               return -1;
-       if (G_UNLIKELY(!s2))
-               return 1;       
 
        /* if it's not case sensitive */
        if (!insensitive)
-               return strcmp (s1, s2);
+               return strcmp (s1 ? s1 : "", s2 ? s2 : "");
        
-       n1 = g_utf8_collate_key (s1, -1);
-       n2 = g_utf8_collate_key (s2, -1);
+       n1 = g_utf8_collate_key (s1 ? s1 : "", -1);
+       n2 = g_utf8_collate_key (s2 ? s2 : "", -1);
        
        result = strcmp (n1, n2);
 
@@ -758,31 +764,24 @@ modest_text_utils_utf8_strcmp (const gchar* s1, const gchar *s2, gboolean insens
 }
 
 
-const gchar*
+gchar*
 modest_text_utils_get_display_date (time_t date)
 {
-       struct tm date_tm, now_tm; 
        time_t now;
-
-       const gint buf_size = 64; 
-       static gchar date_buf[64]; /* buf_size is not ... */
-       static gchar now_buf[64];  /* ...const enough... */
+       const guint BUF_SIZE = 64; 
+       gchar date_buf[BUF_SIZE];  
+       gchar now_buf [BUF_SIZE];  
        
        now = time (NULL);
-       
-       localtime_r(&now, &now_tm);
-       localtime_r(&date, &date_tm);
-
-       /* get today's date */
-       modest_text_utils_strftime (date_buf, buf_size, "%x", &date_tm);
-       modest_text_utils_strftime (now_buf,  buf_size, "%x",  &now_tm);
-       /* 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)
-               strftime (date_buf, buf_size, _("%X"), &date_tm); 
-               
-       return date_buf;
+               modest_text_utils_strftime (date_buf, BUF_SIZE, _("%X"), date);
+       
+       return g_strdup(date_buf);
 }
 
 gboolean 
@@ -841,3 +840,26 @@ modest_text_utils_validate_email_address (const gchar *email_address)
 
        return (count >= 1) ? TRUE : FALSE;
 }
+
+
+
+
+gchar *
+modest_text_utils_get_display_size (guint size)
+{
+       const guint KB=1024;
+       const guint MB=1024 * KB;
+       const guint GB=1024 * MB;
+       const guint TB=1024 * GB;
+
+       if (size < KB)
+               return g_strdup_printf (_("%0.1f Kb"), (double)size / KB);
+       else if (size < MB)
+               return g_strdup_printf (_("%d Kb"), size / KB);
+       else if (size < GB)
+               return g_strdup_printf (_("%d Mb"), size / MB);
+       else if (size < TB)
+               return g_strdup_printf (_("%d Gb"), size/ GB);
+       else
+               return g_strdup_printf (_("Very big"));
+}