talons

commit f6ff237a84a3cd6cd12430ceed96294b28895137
parent a35fa2bcb535d69a2517314f8f3c3231ca4b4038
Author: Ricardo Mones <ricardo@mones.org>
Date:   Sun, 17 Nov 2013 19:15:21 +0100

Clean mailmbox plugin warnings

Cleans:
 • plugin_gtk.c:312:9: warning: variable 'old_path' set but not used
 • plugin_gtk.c:397:9: warning: variable 'old_path' set but not used
 • mailimf_write.c:636:16: warning: variable 'next_word' set but not used
 • mailimf_write.c:635:16: warning: variable 'word_end' set but not used
 • mailmbox.c:519:10: warning: variable 'fixed_size' set but not used
 • mailmbox.c:610:10: warning: variable 'maxuid' set but not used
 • mailmbox_folder.c:162:36: warning: assignment from incompatible pointer type
 • mailmbox_folder.c:872:5: warning: 'r' may be used uninitialized in this function

Diffstat:
M
src/plugins/mailmbox/mailimf_write.c
 | 
4
----
M
src/plugins/mailmbox/mailmbox.c
 | 
6
------
M
src/plugins/mailmbox/mailmbox_folder.c
 | 
6
+++---
M
src/plugins/mailmbox/plugin_gtk.c
 | 
5
-----

4 files changed, 3 insertions(+), 18 deletions(-)
diff --git a/src/plugins/mailmbox/mailimf_write.c b/src/plugins/mailmbox/mailimf_write.c
@@ -632,16 +632,12 @@ int mailimf_header_string_write(FILE * f, int * col,
   int state;
   const char * p;
   const char * word_begin;
-  const char * word_end;
-  const char * next_word;
   int first;
   
   state = STATE_BEGIN;
   
   p = str;
   word_begin = p;
-  word_end = p;
-  next_word = p;
   first = 1;
   
   while (length > 0) {
diff --git a/src/plugins/mailmbox/mailmbox.c b/src/plugins/mailmbox/mailmbox.c
@@ -516,7 +516,6 @@ static char * write_fixed_message(char * str,
 				  const char * message, size_t size,
 				  uint32_t uid, int force_no_uid)
 {
-  size_t fixed_size;
   size_t cur_token;
   size_t left;
   int end;
@@ -526,8 +525,6 @@ static char * write_fixed_message(char * str,
 
   cur_token = 0;
 
-  fixed_size = 0;
-
   /* headers */
 
   end = FALSE;
@@ -607,7 +604,6 @@ claws_mailmbox_append_message_list_no_lock(struct claws_mailmbox_folder * folder
   char * str;
   unsigned int i;
   size_t from_size;
-  size_t maxuid;
   size_t left;
   size_t crlf_count;
 
@@ -621,8 +617,6 @@ claws_mailmbox_append_message_list_no_lock(struct claws_mailmbox_folder * folder
   if (localtime_r(&date, &time_info) != NULL)
     from_size = strftime(from_line, MAX_FROM_LINE_SIZE, "From - %c\n", &time_info);
 
-  maxuid = /* */ folder->mb_max_uid;
-
   extra_size = 0;
   for(i = 0 ; i < carray_count(append_tab) ; i ++) {
     struct claws_mailmbox_append_info * info;
diff --git a/src/plugins/mailmbox/mailmbox_folder.c b/src/plugins/mailmbox/mailmbox_folder.c
@@ -94,7 +94,7 @@ static gint claws_mailmbox_copy_msgs(Folder *folder, FolderItem *dest,
     MsgInfoList *msglist, GHashTable *relation);
 
 static gint claws_mailmbox_remove_msg(Folder *folder, FolderItem *item, gint num);
-static gint claws_mailmbox_remove_msgs( Folder *folder, FolderItem *item, MsgInfoList *msglist, GRelation *relation );
+static gint claws_mailmbox_remove_msgs( Folder *folder, FolderItem *item, MsgInfoList *msglist, GHashTable *relation );
 static gint claws_mailmbox_remove_all_msg(Folder *folder, FolderItem *item);
 
 static FolderItem *claws_mailmbox_create_folder(Folder *folder, FolderItem *parent,
@@ -824,7 +824,7 @@ static gint claws_mailmbox_remove_msg(Folder *folder, FolderItem *item, gint num
 
 static gint
 claws_mailmbox_remove_msgs( Folder *folder, FolderItem *item,
-                            MsgInfoList *msglist, GRelation *relation )
+                            MsgInfoList *msglist, GHashTable *relation )
 {
     struct claws_mailmbox_folder *mbox;
     int r;
@@ -863,7 +863,7 @@ claws_mailmbox_remove_msgs( Folder *folder, FolderItem *item,
 
     /* Fix for bug 1434
      */
-    claws_mailmbox_expunge(mbox);
+    r = claws_mailmbox_expunge(mbox);
     if (total > 100) {
 		statusbar_progress_all(0,0,0);
 		statusbar_pop_all();
diff --git a/src/plugins/mailmbox/plugin_gtk.c b/src/plugins/mailmbox/plugin_gtk.c
@@ -309,7 +309,6 @@ static void delete_folder_cb(GtkAction *action, gpointer data)
 	FolderItem *item;
 	gchar *message, *name;
 	AlertValue avalue;
-	gchar *old_path;
 	gchar *old_id;
 
 	item = folderview_get_selected_item(folderview);
@@ -328,7 +327,6 @@ static void delete_folder_cb(GtkAction *action, gpointer data)
 	g_free(message);
 	if (avalue != G_ALERTALTERNATE) return;
 
-	Xstrdup_a(old_path, item->path, return);
 	old_id = folder_item_get_identifier(item);
 
 	if (folderview->opened == folderview->selected ||
@@ -394,7 +392,6 @@ static void rename_folder_cb(GtkAction *action, gpointer data)
 	gchar *new_folder;
 	gchar *name;
 	gchar *message;
-	gchar *old_path;
 	gchar *old_id;
 	gchar *new_id;
 	gchar *p;
@@ -430,8 +427,6 @@ static void rename_folder_cb(GtkAction *action, gpointer data)
 		return;
 	}
 
-	Xstrdup_a(old_path, item->path, {g_free(new_folder); return;});
-
 	old_id = folder_item_get_identifier(item);
 
 	if (folder_item_rename(item, new_folder) < 0) {