RSSyl: Use g_mkstemp() instead of mkstemp()

[claws.git] / src / plugins / rssyl / rssyl_add_item.c

diff --git a/src/plugins/rssyl/rssyl_add_item.c b/src/plugins/rssyl/rssyl_add_item.c
index 551f29dd86ec2f3d905ee924b4ecc14aec17770d..c2258f9e842266e105cb3fb0f1a6a2af09392fea 100644 (file)
--- a/src/plugins/rssyl/rssyl_add_item.c
+++ b/src/plugins/rssyl/rssyl_add_item.c
@@ -53,7 +53,9 @@
 static gint rssyl_cb_feed_compare(const FeedItem *a, const FeedItem *b)
 {
        gboolean date_eq = FALSE, url_eq = FALSE, title_eq = FALSE;
+       gboolean pubdate_eq = FALSE, moddate_eq = FALSE;
        gboolean no_url = FALSE, no_date = FALSE, no_title = FALSE;
+       gboolean no_pubdate = FALSE, no_moddate = FALSE;
        gchar *atit = NULL, *btit = NULL;
 
        g_return_val_if_fail(a != NULL && b != NULL, 1);
@@ -77,6 +79,10 @@ static gint rssyl_cb_feed_compare(const FeedItem *a, const FeedItem *b)
        } else
                no_url = TRUE;
 
+       /* Now we prepare some boolean flags to help us express comparing choices
+        * later on. */
+
+       /* Title */
        if( (a->title != NULL) && (b->title != NULL) ) {
                atit = conv_unmime_header(a->title, CS_UTF_8, FALSE);
                btit = conv_unmime_header(b->title, CS_UTF_8, FALSE);
@@ -87,41 +93,35 @@ static gint rssyl_cb_feed_compare(const FeedItem *a, const FeedItem *b)
        } else
                no_title = TRUE;
 
-       /* If there's no 'published' timestamp for the item, we can only judge
-        * by item url - 'modified' timestamp can have changed if the item was
-        * updated recently. */
-       if( b->date_published <= 0 ) {
-               if( b->date_modified > 0 ) {
-                       /* If the item has 'modified' timestamp, we can only rely on url
-                        * and title at this point. */
-                       if( (url_eq || no_url) && title_eq
-                               && (a->date_modified >= b->date_modified) )
-                               return 0;
-                       else
-                               return 1;
-               } else {
-                       /* No timestamp of any kind, we'll just assume if both title and url
-                        * match, we found the right item. Items in such feeds rarely change,
-                        * and if they do, there's no way we can really */
-                       if( (url_eq || no_url) && title_eq )
-                               return 0;
-                       else
-                               return 1;
-               }
+       /* Published date */
+       if (b->date_published <= 0) {
+               no_pubdate = TRUE;
+       } else {
+               if (a->date_published == b->date_published)
+                       pubdate_eq = TRUE;
        }
 
-       /* Check if 'published' or at least 'modified' timestamps match */
-       if( ((a->date_published > 0) && (b->date_published > 0) &&
-                       (a->date_published == b->date_published))
-                       || ((a->date_modified > 0) && (b->date_modified > 0) &&
-                       (a->date_modified == b->date_modified))) {
-               date_eq = TRUE;
-       } else
+       /* Modified date */
+       if (b->date_modified <= 0) {
+               no_moddate = TRUE;
+       } else {
+               if (a->date_modified == b->date_modified)
+                       moddate_eq = TRUE;
+       }
+
+       if (no_pubdate && no_moddate)
                no_date = TRUE;
 
-       /* If 'published' time and item url match, it is reasonable to assume
+       if (pubdate_eq || (no_pubdate && moddate_eq))
+               date_eq = TRUE;
+
+       /* If timestamp and url match, it is reasonable to assume
         * we found our item. */
-       if( (no_url || url_eq) && date_eq )
+       if (url_eq && date_eq)
+               return 0;
+
+       /* Likewise if timestamp and title match. */
+       if (title_eq && date_eq)
                return 0;
 
        /* There is no timestamp and the url matches (or there is none),
@@ -374,8 +374,8 @@ void rssyl_add_item(RFolderItem *ritem, FeedItem *feed_item)
        dirname = folder_item_get_path(&ritem->item);
        template = g_strconcat(dirname, G_DIR_SEPARATOR_S,
                        RSSYL_TMP_TEMPLATE, NULL);
-       if ((fd = mkstemp(template)) < 0) {
-               g_warning("Couldn't mkstemp('%s'), not adding message!\n", template);
+       if ((fd = g_mkstemp(template)) < 0) {
+               g_warning("Couldn't g_mkstemp('%s'), not adding message!\n", template);
                g_free(template);
                return;
        }