2007-12-06 [colin] 3.1.0cvs62
[claws.git] / src / imap.c
index edacb29a0d0ad1acb42a9d4054f34f02dafa9e77..b57e5ca1dab85ccce3ce379fbd962c54527265e2 100644 (file)
@@ -1,10 +1,10 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2006 Hiroyuki Yamamoto and the Sylpheed-Claws team
+ * Copyright (C) 1999-2007 Hiroyuki Yamamoto and the Claws Mail team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
@@ -13,8 +13,8 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * 
  */
 
 #ifdef HAVE_CONFIG_H
@@ -31,6 +31,7 @@
 #include "imap_gtk.h"
 #include "inc.h"
 #include "xml.h"
+#include "alertpanel.h"
 
 #ifdef HAVE_LIBETPAN
 
@@ -44,7 +45,7 @@
 #  include <iconv.h>
 #endif
 
-#if USE_OPENSSL
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
 #  include "ssl.h"
 #endif
 
 #include "inputdialog.h"
 #include "log.h"
 #include "remotefolder.h"
-#include "alertpanel.h"
-#include "sylpheed.h"
+#include "claws.h"
 #include "statusbar.h"
 #include "msgcache.h"
 #include "imap-thread.h"
 #include "account.h"
+#include "tags.h"
 
 typedef struct _IMAPFolder     IMAPFolder;
 typedef struct _IMAPSession    IMAPSession;
@@ -91,6 +92,7 @@ struct _IMAPFolder
        GList *ns_shared;
        gchar last_seen_separator;
        guint refcnt;
+       guint max_set_size;
 };
 
 struct _IMAPSession
@@ -108,8 +110,16 @@ struct _IMAPSession
        /* CLAWS */
        gboolean folder_content_changed;
        guint exists;
+       guint recent;
+       guint expunge;
+       guint unseen;
+       guint uid_validity;
+       guint uid_next;
+
        Folder * folder;
        gboolean busy;
+       gboolean cancelled;
+       gboolean sens_update_block;
 };
 
 struct _IMAPNameSpace
@@ -118,14 +128,6 @@ struct _IMAPNameSpace
        gchar separator;
 };
 
-#define IMAP_SUCCESS   0
-#define IMAP_SOCKET    2
-#define IMAP_AUTHFAIL  3
-#define IMAP_PROTOCOL  4
-#define IMAP_SYNTAX    5
-#define IMAP_IOERR     6
-#define IMAP_ERROR     7
-
 #define IMAPBUFSIZE    8192
 
 typedef enum
@@ -145,12 +147,18 @@ typedef enum
 
 
 #define IMAP4_PORT     143
-#if USE_OPENSSL
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
 #define IMAPS_PORT     993
 #endif
 
 #define IMAP_CMD_LIMIT 1000
 
+enum {
+       ITEM_CAN_CREATE_FLAGS_UNKNOWN = 0,
+       ITEM_CAN_CREATE_FLAGS,
+       ITEM_CANNOT_CREATE_FLAGS
+};
+
 struct _IMAPFolderItem
 {
        FolderItem item;
@@ -160,14 +168,17 @@ struct _IMAPFolderItem
        GSList *uid_list;
        gboolean batching;
 
-       time_t use_cache;
-       gint c_messages;
-       guint32 c_uid_next;
-       guint32 c_uid_validity;
-       gint c_unseen;
-
        GHashTable *flags_set_table;
        GHashTable *flags_unset_table;
+       guint32 last_change;
+       guint32 last_sync;
+       gboolean should_update;
+       gboolean should_trash_cache;
+       gint can_create_flags;
+
+       GHashTable *tags_set_table;
+       GHashTable *tags_unset_table;
+
 };
 
 static XMLTag *imap_item_get_xml(Folder *folder, FolderItem *item);
@@ -195,6 +206,9 @@ static gchar   *imap_fetch_msg_full (Folder         *folder,
                                         gint            uid,
                                         gboolean        headers,
                                         gboolean        body);
+static void    imap_remove_cached_msg  (Folder         *folder, 
+                                        FolderItem     *item, 
+                                        MsgInfo        *msginfo);
 static gint    imap_add_msg            (Folder         *folder,
                                         FolderItem     *dest,
                                         const gchar    *file, 
@@ -254,7 +268,8 @@ static gint imap_auth                       (IMAPSession    *session,
                                         IMAPAuthType    type);
 
 static gint imap_scan_tree_recursive   (IMAPSession    *session,
-                                        FolderItem     *item);
+                                        FolderItem     *item,
+                                        gboolean        subs_only);
 
 static void imap_create_missing_folders        (Folder         *folder);
 static FolderItem *imap_create_special_folder
@@ -274,14 +289,16 @@ static void imap_set_batch                (Folder         *folder,
 static gint imap_set_message_flags     (IMAPSession    *session,
                                         MsgNumberList  *numlist,
                                         IMAPFlags       flags,
+                                        GSList         *tags,
                                         gboolean        is_set);
 static gint imap_select                        (IMAPSession    *session,
                                         IMAPFolder     *folder,
-                                        const gchar    *path,
+                                        FolderItem     *item,
                                         gint           *exists,
                                         gint           *recent,
                                         gint           *unseen,
                                         guint32        *uid_validity,
+                                        gint           *can_create_flags,
                                         gboolean        block);
 static gint imap_status                        (IMAPSession    *session,
                                         IMAPFolder     *folder,
@@ -292,14 +309,21 @@ static gint imap_status                   (IMAPSession    *session,
                                         guint32        *uid_validity,
                                         gint           *unseen,
                                         gboolean        block);
+static void    imap_commit_tags        (FolderItem     *item, 
+                                        MsgInfo        *msginfo,
+                                        GSList         *set_tags,
+                                        GSList         *unset_tags);
 
-static IMAPNameSpace *imap_find_namespace      (IMAPFolder     *folder,
-                                                const gchar    *path);
-static gchar imap_get_path_separator           (IMAPFolder     *folder,
+static gchar imap_get_path_separator           (IMAPSession    *session,
+                                                IMAPFolder     *folder,
                                                 const gchar    *path);
-static gchar *imap_get_real_path               (IMAPFolder     *folder,
+static gchar *imap_get_real_path               (IMAPSession    *session,
+                                                IMAPFolder     *folder,
                                                 const gchar    *path);
-static void imap_synchronise           (FolderItem     *item);
+#ifdef HAVE_LIBETPAN
+static void imap_synchronise           (FolderItem     *item, gint days);
+#endif
+static gboolean imap_is_busy           (Folder *folder);
 
 static void imap_free_capabilities     (IMAPSession    *session);
 
@@ -309,7 +333,7 @@ static gint imap_cmd_login  (IMAPSession    *session,
                                 const gchar    *pass,
                                 const gchar    *type);
 static gint imap_cmd_noop      (IMAPSession    *session);
-#if USE_OPENSSL
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
 static gint imap_cmd_starttls  (IMAPSession    *session);
 #endif
 static gint imap_cmd_select    (IMAPSession    *session,
@@ -318,7 +342,9 @@ static gint imap_cmd_select (IMAPSession    *session,
                                 gint           *recent,
                                 gint           *unseen,
                                 guint32        *uid_validity,
+                                gint           *can_create_flags,
                                 gboolean        block);
+static gint imap_cmd_close     (IMAPSession    *session);
 static gint imap_cmd_examine   (IMAPSession    *session,
                                 const gchar    *folder,
                                 gint           *exists,
@@ -352,15 +378,13 @@ static gint imap_cmd_copy       (IMAPSession *session,
 static gint imap_cmd_store     (IMAPSession    *session,
                                 struct mailimap_set * set,
                                 IMAPFlags flags,
+                                GSList *tags,
                                 int do_add);
 static gint imap_cmd_expunge   (IMAPSession    *session);
 
 static void imap_path_separator_subst          (gchar          *str,
                                                 gchar           separator);
 
-static gchar *imap_utf8_to_modified_utf7       (const gchar    *from);
-static gchar *imap_modified_utf7_to_utf8       (const gchar    *mutf7_str);
-
 static gboolean imap_rename_folder_func                (GNode          *node,
                                                 gpointer        data);
 static gint imap_get_num_list                  (Folder         *folder,
@@ -389,20 +413,22 @@ static gchar *imap_item_get_path          (Folder         *folder,
 static MsgInfo *imap_parse_msg(const gchar *file, FolderItem *item);
 
 
-/* data types conversion libetpan <-> sylpheed */
+/* data types conversion libetpan <-> claws */
 static GSList * imap_list_from_lep(IMAPFolder * folder,
                                   clist * list, const gchar * real_path, gboolean all);
-static GSList * imap_get_lep_set_from_numlist(MsgNumberList *numlist);
-static GSList * imap_get_lep_set_from_msglist(MsgInfoList *msglist);
+static GSList * imap_get_lep_set_from_numlist(IMAPFolder *folder, MsgNumberList *numlist);
+static GSList * imap_get_lep_set_from_msglist(IMAPFolder *folder, MsgInfoList *msglist);
 static GSList * imap_uid_list_from_lep(clist * list);
 static GSList * imap_uid_list_from_lep_tab(carray * list);
+static void imap_flags_hash_from_lep_uid_flags_tab(carray * list,
+                                                  GHashTable * hash,
+                                                  GHashTable *tags_hash);
 static MsgInfo *imap_envelope_from_lep(struct imap_fetch_env_info * info,
                                       FolderItem *item);
 static void imap_lep_set_free(GSList *seq_list);
-static struct mailimap_flag_list * imap_flag_to_lep(IMAPFlags flags);
+static struct mailimap_flag_list * imap_flag_to_lep(IMAPFlags flags, GSList *tags);
 
 typedef struct _hashtable_data {
-       IMAPSession *session;
        GSList *msglist;
        IMAPFolderItem *item;
 } hashtable_data;
@@ -414,7 +440,7 @@ typedef struct _thread_data {
        gushort port;
        gboolean done;
        SockInfo *sock;
-#ifdef USE_OPENSSL
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
        SSLType ssl_type;
 #endif
 } thread_data;
@@ -464,6 +490,8 @@ FolderClass *imap_get_class(void)
                imap_class.get_flags = imap_get_flags;
                imap_class.set_batch = imap_set_batch;
                imap_class.synchronise = imap_synchronise;
+               imap_class.remove_cached_msg = imap_remove_cached_msg;
+               imap_class.commit_tags = imap_commit_tags;
 #ifdef USE_PTREAD
                pthread_mutex_init(&imap_mutex, NULL);
 #endif
@@ -472,6 +500,220 @@ FolderClass *imap_get_class(void)
        return &imap_class;
 }
 
+static void imap_refresh_sensitivity (IMAPSession *session)
+{
+        MainWindow *mainwin;
+
+       if (session->sens_update_block)
+               return;
+       mainwin = mainwindow_get_mainwindow();
+       if (mainwin) {
+               toolbar_main_set_sensitive(mainwin);
+               main_window_set_menu_sensitive(mainwin);
+       }
+}
+
+static void lock_session(IMAPSession *session)
+{
+       if (session) {
+               debug_print("locking session %p (%d)\n", session, session->busy);
+               if (session->busy)
+                       debug_print("         SESSION WAS LOCKED !!      \n");
+                session->busy = TRUE;
+               imap_refresh_sensitivity(session);
+       } else {
+               debug_print("can't lock null session\n");
+       }
+}
+
+static void unlock_session(IMAPSession *session)
+{
+       if (session) {
+               debug_print("unlocking session %p\n", session);
+               session->busy = FALSE;
+               imap_refresh_sensitivity(session);
+       } else {
+               debug_print("can't unlock null session\n");
+       }
+}
+
+static void imap_disc_session_destroy(Folder *folder)
+{
+       RemoteFolder *rfolder = REMOTE_FOLDER(folder);
+       IMAPSession *session = NULL;
+       
+       if (!rfolder)
+               return;
+       session = IMAP_SESSION(rfolder->session);
+       if (!session)
+               return;
+       rfolder->session = NULL;
+       log_warning(LOG_PROTOCOL, _("IMAP4 connection broken\n"));
+       SESSION(session)->state = SESSION_DISCONNECTED;
+       session_destroy(SESSION(session));
+}
+
+static gboolean is_fatal(int libetpan_errcode)
+{
+       switch(libetpan_errcode) {
+       case MAILIMAP_ERROR_STREAM:
+       case MAILIMAP_ERROR_PROTOCOL:
+       case MAILIMAP_ERROR_PARSE:
+       case MAILIMAP_ERROR_BAD_STATE:
+               return TRUE;
+       default:
+               return FALSE;
+       }
+}
+
+static void imap_handle_error(Session *session, int libetpan_errcode)
+{
+       switch(libetpan_errcode) {
+       case MAILIMAP_NO_ERROR:
+               return;
+       case MAILIMAP_NO_ERROR_AUTHENTICATED:
+               log_warning(LOG_PROTOCOL, _("IMAP error: authenticated\n"));
+               break;
+       case MAILIMAP_NO_ERROR_NON_AUTHENTICATED:
+               log_warning(LOG_PROTOCOL, _("IMAP error: not authenticated\n"));
+               break;
+       case MAILIMAP_ERROR_BAD_STATE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: bad state\n"));
+               break;
+       case MAILIMAP_ERROR_STREAM:
+               log_warning(LOG_PROTOCOL, _("IMAP error: stream error\n"));
+               break;
+       case MAILIMAP_ERROR_PARSE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: parse error "
+                                           "(very probably non-RFC compliance from the server)\n"));
+               break;
+       case MAILIMAP_ERROR_CONNECTION_REFUSED:
+               log_warning(LOG_PROTOCOL, _("IMAP error: connection refused\n"));
+               break;
+       case MAILIMAP_ERROR_MEMORY:
+               log_warning(LOG_PROTOCOL, _("IMAP error: memory error\n"));
+               break;
+       case MAILIMAP_ERROR_FATAL:
+               log_warning(LOG_PROTOCOL, _("IMAP error: fatal error\n"));
+               break;
+       case MAILIMAP_ERROR_PROTOCOL:
+               log_warning(LOG_PROTOCOL, _("IMAP error: protocol error"
+                                           "(very probably non-RFC compliance from the server)\n"));
+               break;
+       case MAILIMAP_ERROR_DONT_ACCEPT_CONNECTION:
+               log_warning(LOG_PROTOCOL, _("IMAP error: connection not accepted\n"));
+               break;
+       case MAILIMAP_ERROR_APPEND:
+               log_warning(LOG_PROTOCOL, _("IMAP error: APPEND error\n"));
+               break;
+       case MAILIMAP_ERROR_NOOP:
+               log_warning(LOG_PROTOCOL, _("IMAP error: NOOP error\n"));
+               break;
+       case MAILIMAP_ERROR_LOGOUT:
+               log_warning(LOG_PROTOCOL, _("IMAP error: LOGOUT error\n"));
+               break;
+       case MAILIMAP_ERROR_CAPABILITY:
+               log_warning(LOG_PROTOCOL, _("IMAP error: CAPABILITY error\n"));
+               break;
+       case MAILIMAP_ERROR_CHECK:
+               log_warning(LOG_PROTOCOL, _("IMAP error: CHECK error\n"));
+               break;
+       case MAILIMAP_ERROR_CLOSE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: CLOSE error\n"));
+               break;
+       case MAILIMAP_ERROR_EXPUNGE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: EXPUNGE error\n"));
+               break;
+       case MAILIMAP_ERROR_COPY:
+               log_warning(LOG_PROTOCOL, _("IMAP error: COPY error\n"));
+               break;
+       case MAILIMAP_ERROR_UID_COPY:
+               log_warning(LOG_PROTOCOL, _("IMAP error: UID COPY error\n"));
+               break;
+       case MAILIMAP_ERROR_CREATE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: CREATE error\n"));
+               break;
+       case MAILIMAP_ERROR_DELETE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: DELETE error\n"));
+               break;
+       case MAILIMAP_ERROR_EXAMINE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: EXAMINE error\n"));
+               break;
+       case MAILIMAP_ERROR_FETCH:
+               log_warning(LOG_PROTOCOL, _("IMAP error: FETCH error\n"));
+               break;
+       case MAILIMAP_ERROR_UID_FETCH:
+               log_warning(LOG_PROTOCOL, _("IMAP error: UID FETCH error\n"));
+               break;
+       case MAILIMAP_ERROR_LIST:
+               log_warning(LOG_PROTOCOL, _("IMAP error: LIST error\n"));
+               break;
+       case MAILIMAP_ERROR_LOGIN:
+               log_warning(LOG_PROTOCOL, _("IMAP error: LOGIN error\n"));
+               break;
+       case MAILIMAP_ERROR_LSUB:
+               log_warning(LOG_PROTOCOL, _("IMAP error: LSUB error\n"));
+               break;
+       case MAILIMAP_ERROR_RENAME:
+               log_warning(LOG_PROTOCOL, _("IMAP error: RENAME error\n"));
+               break;
+       case MAILIMAP_ERROR_SEARCH:
+               log_warning(LOG_PROTOCOL, _("IMAP error: SEARCH error\n"));
+               break;
+       case MAILIMAP_ERROR_UID_SEARCH:
+               log_warning(LOG_PROTOCOL, _("IMAP error: UID SEARCH error\n"));
+               break;
+       case MAILIMAP_ERROR_SELECT:
+               log_warning(LOG_PROTOCOL, _("IMAP error: SELECT error\n"));
+               break;
+       case MAILIMAP_ERROR_STATUS:
+               log_warning(LOG_PROTOCOL, _("IMAP error: STATUS error\n"));
+               break;
+       case MAILIMAP_ERROR_STORE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: STORE error\n"));
+               break;
+       case MAILIMAP_ERROR_UID_STORE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: UID STORE error\n"));
+               break;
+       case MAILIMAP_ERROR_SUBSCRIBE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: SUBSCRIBE error\n"));
+               break;
+       case MAILIMAP_ERROR_UNSUBSCRIBE:
+               log_warning(LOG_PROTOCOL, _("IMAP error: UNSUBSCRIBE error\n"));
+               break;
+       case MAILIMAP_ERROR_STARTTLS:
+               log_warning(LOG_PROTOCOL, _("IMAP error: STARTTLS error\n"));
+               break;
+       case MAILIMAP_ERROR_INVAL:
+               log_warning(LOG_PROTOCOL, _("IMAP error: INVAL error\n"));
+               break;
+       case MAILIMAP_ERROR_EXTENSION:
+               log_warning(LOG_PROTOCOL, _("IMAP error: EXTENSION error\n"));
+               break;
+       case MAILIMAP_ERROR_SASL:
+               log_warning(LOG_PROTOCOL, _("IMAP error: SASL error\n"));
+               break;
+#if (LIBETPAN_VERSION_MAJOR > 0 || LIBETPAN_VERSION_MINOR > 48)
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
+       case MAILIMAP_ERROR_SSL:
+               log_warning(LOG_PROTOCOL, _("IMAP error: SSL error\n"));
+               break;
+#endif
+#endif
+       default:
+               log_warning(LOG_PROTOCOL, _("IMAP error: Unknown error [%d]\n"),
+                       libetpan_errcode);
+               break;
+       }
+
+       if (session && is_fatal(libetpan_errcode)) {
+               imap_disc_session_destroy(IMAP_SESSION(session)->folder);
+       } else if (session && !is_fatal(libetpan_errcode)) {
+               if (IMAP_SESSION(session)->busy)
+                       unlock_session(IMAP_SESSION(session));
+       }
+}
+
 static Folder *imap_folder_new(const gchar *name, const gchar *path)
 {
        Folder *folder;
@@ -496,6 +738,7 @@ static void imap_folder_init(Folder *folder, const gchar *name,
                             const gchar *path)
 {
        folder_remote_folder_init((Folder *)folder, name, path);
+       IMAP_FOLDER(folder)->max_set_size = IMAP_SET_MAX_COUNT;
 }
 
 static FolderItem *imap_folder_item_new(Folder *folder)
@@ -540,7 +783,7 @@ static void imap_reset_uid_lists(Folder *folder)
        g_node_traverse(folder->node, G_IN_ORDER, G_TRAVERSE_ALL, -1, imap_reset_uid_lists_func, NULL); 
 }
 
-int imap_get_capabilities(IMAPSession *session)
+static int imap_get_capabilities(IMAPSession *session)
 {
        struct mailimap_capability_data *capabilities = NULL;
        clistiter *cur;
@@ -552,6 +795,7 @@ int imap_get_capabilities(IMAPSession *session)
        capabilities = imap_threaded_capability(session->folder, &result);
 
        if (result != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), result);
                return MAILIMAP_ERROR_CAPABILITY;
        }
 
@@ -574,7 +818,7 @@ int imap_get_capabilities(IMAPSession *session)
        return MAILIMAP_NO_ERROR;
 }
 
-gboolean imap_has_capability(IMAPSession *session, const gchar *cap) 
+static gboolean imap_has_capability(IMAPSession *session, const gchar *cap) 
 {
        GSList *cur;
        for (cur = session->capability; cur; cur = cur->next) {
@@ -587,13 +831,14 @@ gboolean imap_has_capability(IMAPSession *session, const gchar *cap)
 static gint imap_auth(IMAPSession *session, const gchar *user, const gchar *pass,
                      IMAPAuthType type)
 {
-       gint ok = IMAP_ERROR;
+       gint ok = MAILIMAP_ERROR_BAD_STATE;
        static time_t last_login_err = 0;
        gchar *ext_info = "";
-       
-       if (imap_get_capabilities(session) != MAILIMAP_NO_ERROR)
-               return IMAP_ERROR;
-
+       int r;
+       if ((r = imap_get_capabilities(session)) != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
+               return r;
+       }
        switch(type) {
        case IMAP_AUTH_ANON:
                ok = imap_cmd_login(session, user, pass, "ANONYMOUS");
@@ -619,13 +864,15 @@ static gint imap_auth(IMAPSession *session, const gchar *user, const gchar *pass
                        imap_has_capability(session, "GSSAPI"));
                if (imap_has_capability(session, "CRAM-MD5"))
                        ok = imap_cmd_login(session, user, pass, "CRAM-MD5");
-               if (ok == IMAP_ERROR && imap_has_capability(session, "GSSAPI"))
+               if ((ok == MAILIMAP_ERROR_BAD_STATE ||
+                    ok == MAILIMAP_ERROR_LOGIN) && imap_has_capability(session, "GSSAPI"))
                        ok = imap_cmd_login(session, user, pass, "GSSAPI");
-               if (ok == IMAP_ERROR) /* we always try LOGIN before giving up */
+               if (ok == MAILIMAP_ERROR_BAD_STATE||
+                   ok == MAILIMAP_ERROR_LOGIN) /* we always try LOGIN before giving up */
                        ok = imap_cmd_login(session, user, pass, "LOGIN");
        }
 
-       if (ok == IMAP_SUCCESS)
+       if (ok == MAILIMAP_NO_ERROR)
                session->authenticated = TRUE;
        else {
                if (type == IMAP_AUTH_CRAM_MD5) {
@@ -640,7 +887,7 @@ static gint imap_auth(IMAPSession *session, const gchar *user, const gchar *pass
                                        "login refused.%s"),
                                        SESSION(session)->server, ext_info);
                        } else {
-                               log_error(_("Connection to %s failed: "
+                               log_error(LOG_PROTOCOL, _("Connection to %s failed: "
                                        "login refused.%s\n"),
                                        SESSION(session)->server, ext_info);
                        }
@@ -657,12 +904,13 @@ static IMAPSession *imap_reconnect_if_possible(Folder *folder, IMAPSession *sess
           connection, if yes we don't try to reconnect */
        debug_print("reconnecting\n");
        if (rfolder->session == NULL) {
-               log_warning(_("Connecting to %s failed"),
+               log_warning(LOG_PROTOCOL, _("Connecting to %s failed"),
                            folder->account->recv_server);
                session_destroy(SESSION(session));
                session = NULL;
        } else {
-               log_warning(_("IMAP4 connection to %s has been"
+               rfolder->session = NULL;
+               log_warning(LOG_PROTOCOL, _("IMAP4 connection to %s has been"
                            " disconnected. Reconnecting...\n"),
                            folder->account->recv_server);
                statusbar_print_all(_("IMAP4 connection to %s has been"
@@ -674,7 +922,7 @@ static IMAPSession *imap_reconnect_if_possible(Folder *folder, IMAPSession *sess
                   a new session, because of rfolder->session == NULL
                   it will not try to reconnect again and so avoid an
                   endless loop */
-               rfolder->session = NULL;
+               debug_print("getting session...\n");
                session = imap_session_get(folder);
                rfolder->session = SESSION(session);
                statusbar_pop_all();
@@ -682,28 +930,19 @@ static IMAPSession *imap_reconnect_if_possible(Folder *folder, IMAPSession *sess
        return session;
 }
 
-#define lock_session() {\
-       debug_print("locking session\n"); \
-       session->busy = TRUE;\
-}
-
-#define unlock_session() {\
-       debug_print("unlocking session\n"); \
-       session->busy = FALSE;\
-}
-
 static IMAPSession *imap_session_get(Folder *folder)
 {
        RemoteFolder *rfolder = REMOTE_FOLDER(folder);
        IMAPSession *session = NULL;
+       gint r = MAILIMAP_NO_ERROR;
 
        g_return_val_if_fail(folder != NULL, NULL);
        g_return_val_if_fail(FOLDER_CLASS(folder) == &imap_class, NULL);
        g_return_val_if_fail(folder->account != NULL, NULL);
        
        if (prefs_common.work_offline && 
-           !inc_offline_should_override(
-               _("Sylpheed-Claws needs network access in order "
+           !inc_offline_should_override(FALSE,
+               _("Claws Mail needs network access in order "
                  "to access the IMAP server."))) {
                return NULL;
        }
@@ -711,18 +950,19 @@ static IMAPSession *imap_session_get(Folder *folder)
        /* Make sure we have a session */
        if (rfolder->session != NULL) {
                session = IMAP_SESSION(rfolder->session);
-               /* don't do that yet... 
-               if (session->busy) {
-                       return NULL;
-               } */
+       } else if (rfolder->connecting) {
+               debug_print("already connecting\n");
+               return NULL;
        } else {
                imap_reset_uid_lists(folder);
                if (time(NULL) - rfolder->last_failure <= 2)
                        return NULL;
+               rfolder->connecting = TRUE;
                session = imap_session_new(folder, folder->account);
        }
        if(session == NULL) {
                rfolder->last_failure = time(NULL);
+               rfolder->connecting = FALSE;
                return NULL;
        }
 
@@ -732,10 +972,11 @@ static IMAPSession *imap_session_get(Folder *folder)
        
        if (!IMAP_SESSION(session)->authenticated) {
                imap_threaded_disconnect(session->folder);
+               rfolder->session = NULL;
                SESSION(session)->state = SESSION_DISCONNECTED;
                session_destroy(SESSION(session));
-               rfolder->session = NULL;
                rfolder->last_failure = time(NULL);
+               rfolder->connecting = FALSE;
                return NULL;
        }
 
@@ -747,16 +988,22 @@ static IMAPSession *imap_session_get(Folder *folder)
         * A better solution than sending a NOOP every time would be
         * for every command to be prepared to retry until it is
         * successfully sent. -- mbp */
-       if (time(NULL) - SESSION(session)->last_access_time > SESSION_TIMEOUT_INTERVAL) {
+       if ((time(NULL) - SESSION(session)->last_access_time > SESSION_TIMEOUT_INTERVAL) || session->cancelled) {
                /* verify that the session is still alive */
-               if (imap_cmd_noop(session) != IMAP_SUCCESS) {
+               if ((r = imap_cmd_noop(session)) != MAILIMAP_NO_ERROR) {
                        debug_print("disconnected!\n");
-                       session = imap_reconnect_if_possible(folder, session);
+                       if (!is_fatal(r))
+                               session = imap_reconnect_if_possible(folder, session);
+                       else
+                               session = imap_session_get(folder);
                }
+               if (session)
+                       session->cancelled = FALSE;
        }
 
        rfolder->session = SESSION(session);
-       
+       rfolder->connecting = FALSE;
+
        return IMAP_SESSION(session);
 }
 
@@ -768,7 +1015,7 @@ static IMAPSession *imap_session_new(Folder * folder,
        int r;
        int authenticated = FALSE;
        
-#ifdef USE_OPENSSL
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
        /* FIXME: IMAP over SSL only... */ 
        SSLType ssl_type;
 
@@ -780,7 +1027,7 @@ static IMAPSession *imap_session_new(Folder * folder,
                if (alertpanel_full(_("Insecure connection"),
                        _("This connection is configured to be secured "
                          "using SSL, but SSL is not available in this "
-                         "build of Sylpheed-Claws. \n\n"
+                         "build of Claws Mail. \n\n"
                          "Do you want to continue connecting to this "
                          "server? The communication would not be "
                          "secure."),
@@ -794,15 +1041,18 @@ static IMAPSession *imap_session_new(Folder * folder,
 #endif
 
        imap_init(folder);
-       statusbar_print_all(_("Connecting to IMAP4 server: %s..."), folder->account->recv_server);
+       statuswindow_print_all(_("Connecting to IMAP4 server: %s..."), folder->account->recv_server);
+#ifndef G_OS_WIN32
        if (account->set_tunnelcmd) {
                r = imap_threaded_connect_cmd(folder,
                                              account->tunnelcmd,
                                              account->recv_server,
                                              port);
        }
-       else {
-#ifdef USE_OPENSSL
+       else 
+#endif
+       {
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
                if (ssl_type == SSL_TUNNEL) {
                        r = imap_threaded_connect_ssl(folder,
                                                      account->recv_server,
@@ -817,7 +1067,7 @@ static IMAPSession *imap_session_new(Folder * folder,
                }
        }
        
-       statusbar_pop_all();
+       statuswindow_pop_all();
        if (r == MAILIMAP_NO_ERROR_AUTHENTICATED) {
                authenticated = TRUE;
        }
@@ -825,11 +1075,20 @@ static IMAPSession *imap_session_new(Folder * folder,
                authenticated = FALSE;
        }
        else {
+#if (LIBETPAN_VERSION_MAJOR > 0 || LIBETPAN_VERSION_MINOR > 48)
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
+               if (r == MAILIMAP_ERROR_SSL)
+                       log_error(LOG_PROTOCOL, _("SSL handshake failed\n"));
+               else
+#endif
+#endif
+                       imap_handle_error(NULL, r);
+
                if(!prefs_common.no_recv_err_panel) {
-                       alertpanel_error(_("Can't connect to IMAP4 server: %s:%d"),
+                       alertpanel_error_log(_("Can't connect to IMAP4 server: %s:%d"),
                                         account->recv_server, port);
                } else {
-                       log_error(_("Can't connect to IMAP4 server: %s:%d\n"),
+                       log_error(LOG_PROTOCOL, _("Can't connect to IMAP4 server: %s:%d\n"),
                                         account->recv_server, port);
                } 
                
@@ -848,17 +1107,20 @@ static IMAPSession *imap_session_new(Folder * folder,
        
        session->authenticated = authenticated;
        session->mbox = NULL;
+       session->exists = 0;
+       session->recent = 0;
+       session->expunge = 0;
        session->cmd_count = 0;
        session->folder = folder;
        IMAP_FOLDER(session->folder)->last_seen_separator = 0;
 
-#if USE_OPENSSL
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
        if (account->ssl_imap == SSL_STARTTLS) {
                gint ok;
 
                ok = imap_cmd_starttls(session);
-               if (ok != IMAP_SUCCESS) {
-                       log_warning(_("Can't start TLS session.\n"));
+               if (ok != MAILIMAP_NO_ERROR) {
+                       log_warning(LOG_PROTOCOL, _("Can't start TLS session.\n"));
                        session_destroy(SESSION(session));
                        return NULL;
                }
@@ -869,7 +1131,7 @@ static IMAPSession *imap_session_new(Folder * folder,
                session->cmd_count = 1;
        }
 #endif
-       log_message("IMAP connection is %s-authenticated\n",
+       log_message(LOG_PROTOCOL, "IMAP connection is %s-authenticated\n",
                    (session->authenticated) ? "pre" : "un");
        
        return session;
@@ -895,9 +1157,9 @@ try_again:
        } else if (account->imap_auth_type == IMAP_AUTH_ANON) {
                pass = "";
        }
-       statusbar_print_all(_("Connecting to IMAP4 server %s...\n"),
+       statuswindow_print_all(_("Connecting to IMAP4 server %s...\n"),
                                account->recv_server);
-       if (imap_auth(session, account->userid, pass, account->imap_auth_type) != IMAP_SUCCESS) {
+       if (imap_auth(session, account->userid, pass, account->imap_auth_type) != MAILIMAP_NO_ERROR) {
                statusbar_pop_all();
                
                if (!failed) {
@@ -906,7 +1168,7 @@ try_again:
                        goto try_again;
                } else {
                        if (prefs_common.no_recv_err_panel) {
-                               log_error(_("Couldn't login to IMAP server %s."), account->recv_server);
+                               log_error(LOG_PROTOCOL, _("Couldn't login to IMAP server %s."), account->recv_server);
                                mainwindow_show_error();
                        } else
                                alertpanel_error_log(_("Couldn't login to IMAP server %s."), account->recv_server);
@@ -915,7 +1177,7 @@ try_again:
                return;
        } 
 
-       statusbar_pop_all();
+       statuswindow_pop_all();
        session->authenticated = TRUE;
        return;
 }
@@ -936,22 +1198,22 @@ static gchar *imap_fetch_msg(Folder *folder, FolderItem *item, gint uid)
        return imap_fetch_msg_full(folder, item, uid, TRUE, TRUE);
 }
 
-static guint get_size_with_crs(MsgInfo *info
+static guint get_file_size_with_crs(const gchar *filename
 {
        FILE *fp = NULL;
        guint cnt = 0;
        gchar buf[4096];
        
-       if (info == NULL)
+       if (filename == NULL)
                return -1;
        
-       fp = procmsg_open_message(info);
+       fp = fopen(filename, "rb");
        if (!fp)
                return -1;
        
        while (fgets(buf, sizeof (buf), fp) != NULL) {
                cnt += strlen(buf);
-               if (!strstr(buf, "\r") && strstr(buf, "\n"))
+               if (!strstr(buf, "\r\n") && strstr(buf, "\n"))
                        cnt++;
        }
        
@@ -959,6 +1221,140 @@ static guint get_size_with_crs(MsgInfo *info)
        return cnt;
 }
 
+static void imap_remove_cached_msg(Folder *folder, FolderItem *item, MsgInfo *msginfo)
+{
+       gchar *path, *filename;
+
+       path = folder_item_get_path(item);
+
+       if (!is_dir_exist(path)) {
+               g_free(path);
+               return;
+       }
+
+       filename = g_strconcat(path, G_DIR_SEPARATOR_S, itos(msginfo->msgnum), NULL);
+       g_free(path);
+
+       if (is_file_exist(filename)) {
+               g_unlink(filename);
+       }
+       g_free(filename);
+}
+
+typedef struct _TagsData {
+       gchar *str;
+       GSList *msglist;
+       IMAPFolderItem *item;
+} TagsData;
+
+static void imap_commit_tags(FolderItem *item, MsgInfo *msginfo, GSList *tags_set, GSList *tags_unset)
+{
+       IMAPSession *session;
+       gint ok, can_create_tags;
+       Folder *folder = NULL;
+       TagsData *ht_data = NULL;
+       GSList *cur;
+
+       g_return_if_fail(item != NULL);
+       g_return_if_fail(msginfo != NULL);
+
+       folder = item->folder;
+       debug_print("getting session...\n");
+       session = imap_session_get(folder);
+       
+       if (!session) {
+               debug_print("can't get session\n");
+               return;
+       }
+
+       ok = imap_select(session, IMAP_FOLDER(folder), item,
+                        NULL, NULL, NULL, NULL, &can_create_tags, FALSE);
+
+       if (ok != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), ok);
+               return;
+       }
+
+       
+       if (IMAP_FOLDER_ITEM(item)->can_create_flags != ITEM_CAN_CREATE_FLAGS)
+               return;
+       
+       if (IMAP_FOLDER_ITEM(item)->batching) {
+               /* instead of performing an UID STORE command for each message change,
+                * as a lot of them can change "together", we just fill in hashtables
+                * and defer the treatment so that we're able to send only one
+                * command.
+                */
+               debug_print("IMAP batch mode on, deferring tags change\n");
+               for (cur = tags_set; cur; cur = cur->next) {
+                       gint cur_tag = GPOINTER_TO_INT(cur->data);
+                       if (cur_tag) {
+                               ht_data = g_hash_table_lookup(IMAP_FOLDER_ITEM(item)->tags_set_table, 
+                                       GINT_TO_POINTER(cur_tag));
+                               if (ht_data == NULL) {
+                                       ht_data = g_new0(TagsData, 1);
+                                       ht_data->str = g_strdup(tags_get_tag(cur_tag));
+                                       ht_data->item = IMAP_FOLDER_ITEM(item);
+                                       g_hash_table_insert(IMAP_FOLDER_ITEM(item)->tags_set_table, 
+                                               GINT_TO_POINTER(cur_tag), ht_data);
+                               }
+                               ht_data->msglist = g_slist_prepend(ht_data->msglist, GINT_TO_POINTER(msginfo->msgnum));
+                       } 
+               }
+               for (cur = tags_unset; cur; cur = cur->next) {
+                       gint cur_tag = GPOINTER_TO_INT(cur->data);
+                       if (cur_tag) {
+                               ht_data = g_hash_table_lookup(IMAP_FOLDER_ITEM(item)->tags_unset_table, 
+                                       GINT_TO_POINTER(cur_tag));
+                               if (ht_data == NULL) {
+                                       ht_data = g_new0(TagsData, 1);
+                                       ht_data->str = g_strdup(tags_get_tag(cur_tag));
+                                       ht_data->item = IMAP_FOLDER_ITEM(item);
+                                       g_hash_table_insert(IMAP_FOLDER_ITEM(item)->tags_unset_table, 
+                                               GINT_TO_POINTER(cur_tag), ht_data);
+                               }
+                               ht_data->msglist = g_slist_prepend(ht_data->msglist, GINT_TO_POINTER(msginfo->msgnum));
+                       }
+               }
+       } else {
+               GSList *list_set = NULL;
+               GSList *list_unset = NULL;
+               GSList numlist;
+               
+               numlist.data = GINT_TO_POINTER(msginfo->msgnum);
+               numlist.next = NULL;
+       
+               debug_print("IMAP changing tags NOW\n");
+               for (cur = tags_set; cur; cur = cur->next) {
+                       gint cur_tag = GPOINTER_TO_INT(cur->data);
+                       const gchar *str = tags_get_tag(cur_tag);
+                       list_set = g_slist_prepend(list_set, g_strdup(str));
+               }
+               if (list_set) {
+                       ok = imap_set_message_flags(session, &numlist, 0, list_set, TRUE);
+                       slist_free_strings(list_set);
+                       g_slist_free(list_set);
+                       if (ok != MAILIMAP_NO_ERROR) {
+                               return;
+                       }
+               }
+
+               for (cur = tags_unset; cur; cur = cur->next) {
+                       gint cur_tag = GPOINTER_TO_INT(cur->data);
+                       const gchar *str = tags_get_tag(cur_tag);
+                       list_unset = g_slist_prepend(list_unset, g_strdup(str));
+               }
+               if (list_unset) {
+                       ok = imap_set_message_flags(session, &numlist, 0, list_unset, FALSE);
+                       slist_free_strings(list_unset);
+                       g_slist_free(list_unset);
+                       if (ok != MAILIMAP_NO_ERROR) {
+                               return;
+                       }
+               }
+       }
+}
+
 static gchar *imap_fetch_msg_full(Folder *folder, FolderItem *item, gint uid,
                                  gboolean headers, gboolean body)
 {
@@ -982,65 +1378,151 @@ static gchar *imap_fetch_msg_full(Folder *folder, FolderItem *item, gint uid,
                /* see whether the local file represents the whole message
                 * or not. As the IMAP server reports size with \r chars,
                 * we have to update the local file (UNIX \n only) size */
-               MsgInfo *msginfo = imap_parse_msg(filename, item);
                MsgInfo *cached = msgcache_get_msg(item->cache,uid);
-               guint have_size = get_size_with_crs(msginfo);
+               guint have_size = -1;
 
                if (cached)
-                       debug_print("message %d has been already %scached (%d/%d).\n", uid,
-                               have_size >= cached->size ? "fully ":"",
-                               have_size, (int)cached->size);
+                       debug_print("message %d has been already %scached.\n", uid,
+                               MSG_IS_FULLY_CACHED(cached->flags) ? "fully ":"");
                
-               if (cached && (cached->size <= have_size || !body)) {
+               if (!cached || !MSG_IS_FULLY_CACHED(cached->flags)) {
+                       have_size = get_file_size_with_crs(filename);
+                       if (cached && (cached->size <= have_size || !body)) {
+                               procmsg_msginfo_free(cached);
+                               ok = file_strip_crs(filename);
+                               if (ok == 0 && cached && cached->size <= have_size) {
+                                       /* we have it all and stripped */
+                                       debug_print("...fully cached in fact; setting flag.\n");
+                                       procmsg_msginfo_set_flags(cached, MSG_FULLY_CACHED, 0);
+                               }
+                               return filename;
+                       } else if (!cached && time(NULL) - get_file_mtime(filename) < 60) {
+                               debug_print("message not cached and file recent, considering file complete\n");
+                               ok = file_strip_crs(filename);
+                               if (ok == 0)
+                                       return filename;
+                       } else {
+                               procmsg_msginfo_free(cached);
+                       }
+               }
+               if (cached && MSG_IS_FULLY_CACHED(cached->flags)) {
                        procmsg_msginfo_free(cached);
-                       procmsg_msginfo_free(msginfo);
-                       file_strip_crs(filename);
-                       return filename;
-               } else if (!cached && time(NULL) - get_file_mtime(filename) < 60) {
-                       debug_print("message not cached and file recent, considering file complete\n");
-                       procmsg_msginfo_free(msginfo);
-                       file_strip_crs(filename);
                        return filename;
-               } else {
+               }
+       } else {
+               MsgInfo *cached = msgcache_get_msg(item->cache,uid);
+               if (cached) {
+                       procmsg_msginfo_unset_flags(cached, MSG_FULLY_CACHED, 0);
                        procmsg_msginfo_free(cached);
-                       procmsg_msginfo_free(msginfo);
                }
        }
 
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        
        if (!session) {
                g_free(filename);
                return NULL;
        }
-
-       lock_session();
+       session_set_access_time(SESSION(session));
+       lock_session(session); /* unlocked later in the function */
 
        debug_print("IMAP fetching messages\n");
-       ok = imap_select(session, IMAP_FOLDER(folder), item->path,
-                        NULL, NULL, NULL, NULL, FALSE);
-       if (ok != IMAP_SUCCESS) {
+       ok = imap_select(session, IMAP_FOLDER(folder), item,
+                        NULL, NULL, NULL, NULL, NULL, FALSE);
+       if (ok != MAILIMAP_NO_ERROR) {
                g_warning("can't select mailbox %s\n", item->path);
                g_free(filename);
-               unlock_session();
                return NULL;
        }
 
+       session_set_access_time(SESSION(session));
+
        debug_print("getting message %d...\n", uid);
        ok = imap_cmd_fetch(session, (guint32)uid, filename, headers, body);
 
-       if (ok != IMAP_SUCCESS) {
+       if (ok != MAILIMAP_NO_ERROR) {
                g_warning("can't fetch message %d\n", uid);
                g_free(filename);
-               unlock_session();
                return NULL;
        }
 
-       unlock_session();
-       file_strip_crs(filename);
+       session_set_access_time(SESSION(session));
+       unlock_session(session);
+
+       ok = file_strip_crs(filename);
+
+       if (ok == 0 && headers && body) {
+               MsgInfo *cached = msgcache_get_msg(item->cache,uid);
+               if (cached) {
+                       procmsg_msginfo_set_flags(cached, MSG_FULLY_CACHED, 0);
+                       procmsg_msginfo_free(cached);
+               }
+       } else if (ok == -1) {
+               MsgInfo *cached = msgcache_get_msg(item->cache,uid);
+               if (cached) {
+                       procmsg_msginfo_unset_flags(cached, MSG_FULLY_CACHED, 0);
+                       procmsg_msginfo_free(cached);
+               }
+       }
        return filename;
 }
 
+static gboolean imap_is_msg_fully_cached(Folder *folder, FolderItem *item, gint uid)
+{
+       gchar *path, *filename;
+       guint size = 0;
+       MsgInfo *cached = msgcache_get_msg(item->cache,uid);
+       
+       if (!cached)
+               return FALSE;
+
+       if (MSG_IS_FULLY_CACHED(cached->flags)) {
+               procmsg_msginfo_free(cached);
+               return TRUE;
+       }
+       path = folder_item_get_path(item);
+       if (!is_dir_exist(path))
+               return FALSE;
+
+       filename = g_strconcat(path, G_DIR_SEPARATOR_S, itos(uid), NULL);
+       g_free(path);
+       if (is_file_exist(filename)) {
+               if (cached && cached->total_size == cached->size) {
+                       /* fast path */
+                       g_free(filename);
+                       procmsg_msginfo_set_flags(cached, MSG_FULLY_CACHED, 0);
+                       return TRUE;
+               }
+               size = get_file_size_with_crs(filename);
+               g_free(filename);
+       }
+       if (cached && size >= cached->size) {
+               cached->total_size = cached->size;
+               procmsg_msginfo_set_flags(cached, MSG_FULLY_CACHED, 0);
+               procmsg_msginfo_free(cached);
+               return TRUE;
+       }
+       if (cached)
+               procmsg_msginfo_free(cached);
+       return FALSE;   
+}
+
+void imap_cache_msg(FolderItem *item, gint msgnum)
+{
+       Folder *folder = NULL;
+       
+       if (!item)
+               return;
+       folder = item->folder;
+       
+       if (!imap_is_msg_fully_cached(folder, item, msgnum)) {
+               gchar *tmp = imap_fetch_msg_full(folder, item, msgnum, TRUE, TRUE);
+               debug_print("fetched %s\n", tmp);
+               g_free(tmp);
+       }
+}
+
 static gint imap_add_msg(Folder *folder, FolderItem *dest, 
                         const gchar *file, MsgFlags *flags)
 {
@@ -1068,20 +1550,20 @@ static gint imap_add_msgs(Folder *folder, FolderItem *dest, GSList *file_list,
        guint32 last_uid = 0;
        GSList *cur;
        MsgFileInfo *fileinfo;
-       gint ok;
+       gint ok = MAILIMAP_NO_ERROR;
        gint curnum = 0, total = 0;
-
+       gboolean missing_uids = FALSE;
 
        g_return_val_if_fail(folder != NULL, -1);
        g_return_val_if_fail(dest != NULL, -1);
        g_return_val_if_fail(file_list != NULL, -1);
        
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        if (!session) {
                return -1;
        }
-       lock_session();
-       destdir = imap_get_real_path(IMAP_FOLDER(folder), dest->path);
+       destdir = imap_get_real_path(session, IMAP_FOLDER(folder), dest->path);
 
        statusbar_print_all(_("Adding messages..."));
        total = g_slist_length(file_list);
@@ -1091,7 +1573,7 @@ static gint imap_add_msgs(Folder *folder, FolderItem *dest, GSList *file_list,
                gchar *real_file = NULL;
                fileinfo = (MsgFileInfo *)cur->data;
 
-               statusbar_progress_all(curnum, total, 1);
+               statusbar_progress_all(curnum, total, total < 10 ? 1:10);
                curnum++;
 
                if (fileinfo->flags) {
@@ -1115,11 +1597,10 @@ static gint imap_add_msgs(Folder *folder, FolderItem *dest, GSList *file_list,
                ok = imap_cmd_append(session, destdir, real_file, iflags, 
                                     &new_uid);
 
-               if (ok != IMAP_SUCCESS) {
+               if (ok != MAILIMAP_NO_ERROR) {
                        g_warning("can't append message %s\n", real_file);
                        g_free(real_file);
                        g_free(destdir);
-                       unlock_session();
                        statusbar_progress_all(0,0,0);
                        statusbar_pop_all();
                        return -1;
@@ -1127,6 +1608,11 @@ static gint imap_add_msgs(Folder *folder, FolderItem *dest, GSList *file_list,
                        debug_print("appended new message as %d\n", new_uid);
                        /* put the local file in the imapcache, so that we don't
                         * have to fetch it back later. */
+                       
+                       if (new_uid == 0) {
+                               missing_uids = TRUE;
+                               debug_print("Missing UID (0)\n");
+                       }
                        if (new_uid > 0) {
                                gchar *cache_path = folder_item_get_path(dest);
                                if (!is_dir_exist(cache_path))
@@ -1136,7 +1622,7 @@ static gint imap_add_msgs(Folder *folder, FolderItem *dest, GSList *file_list,
                                                cache_path, G_DIR_SEPARATOR_S, 
                                                itos(new_uid), NULL);
                                        copy_file(real_file, cache_file, TRUE);
-                                       debug_print("copied to cache: %s\n", cache_file);
+                                       debug_print("got UID %d, copied to cache: %s\n", new_uid, cache_file);
                                        g_free(cache_file);
                                }
                                g_free(cache_path);
@@ -1146,27 +1632,60 @@ static gint imap_add_msgs(Folder *folder, FolderItem *dest, GSList *file_list,
                if (relation != NULL)
                        g_relation_insert(relation, fileinfo->msginfo != NULL ? 
                                          (gpointer) fileinfo->msginfo : (gpointer) fileinfo,
-                                         GINT_TO_POINTER(dest->last_num + 1));
-               if (new_uid == 0) {
-                       new_uid = dest->last_num+1;
-               }
+                                         GINT_TO_POINTER(new_uid));
                if (last_uid < new_uid) {
                        last_uid = new_uid;
                }
 
                g_free(real_file);
        }
+       
        statusbar_progress_all(0,0,0);
        statusbar_pop_all();
        
-       imap_cmd_expunge(session);
-       unlock_session();
        
        g_free(destdir);
 
+       imap_scan_required(folder, dest);
+
+       session = imap_session_get(folder);
+       if (!session) {
+               return -1;
+       }
+       if (missing_uids) {
+               gint a;
+               ok = imap_select(session, IMAP_FOLDER(folder), dest,
+                        &a, NULL, NULL, NULL, NULL, FALSE);
+       }
        return last_uid;
 }
 
+static GSList *flatten_mailimap_set(struct mailimap_set * set) 
+{
+       GSList *result = NULL;
+       clistiter *list;
+       int start, end, t;
+       GSList *cur;
+
+       for (list = clist_begin(set->set_list); list; list = clist_next(list)) {
+               struct mailimap_set_item *item = (struct mailimap_set_item *)clist_content(list);
+               start = item->set_first;
+               end = item->set_last;
+               for (t = start; t <= end; t++) {
+                       result = g_slist_prepend(result, GINT_TO_POINTER(t));
+               }
+       }
+       result = g_slist_reverse(result);
+       if (debug_get_mode()) {
+               debug_print("flat imap set: ");
+               for (cur = result; cur; cur = cur->next) {
+                       debug_print("%d ", GPOINTER_TO_INT(cur->data));
+               }
+               debug_print("\n");
+       }
+       
+       return result;
+}
 static gint imap_do_copy_msgs(Folder *folder, FolderItem *dest, 
                              MsgInfoList *msglist, GRelation *relation)
 {
@@ -1175,7 +1694,7 @@ static gint imap_do_copy_msgs(Folder *folder, FolderItem *dest,
        GSList *seq_list, *cur;
        MsgInfo *msginfo;
        IMAPSession *session;
-       gint ok = IMAP_SUCCESS;
+       gint ok = MAILIMAP_NO_ERROR;
        GRelation *uid_mapping;
        gint last_num = 0;
        gboolean single = FALSE;
@@ -1184,19 +1703,19 @@ static gint imap_do_copy_msgs(Folder *folder, FolderItem *dest,
        g_return_val_if_fail(dest != NULL, -1);
        g_return_val_if_fail(msglist != NULL, -1);
        
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        
        if (!session) {
                return -1;
        }
-       lock_session();
+
        msginfo = (MsgInfo *)msglist->data;
        if (msglist->next == NULL)
                single = TRUE;
        src = msginfo->folder;
        if (src == dest) {
                g_warning("the src folder is identical to the dest.\n");
-               unlock_session();
                return -1;
        }
 
@@ -1221,15 +1740,18 @@ static gint imap_do_copy_msgs(Folder *folder, FolderItem *dest,
                return res;
        } 
 
-       ok = imap_select(session, IMAP_FOLDER(folder), msginfo->folder->path,
-                        NULL, NULL, NULL, NULL, FALSE);
-       if (ok != IMAP_SUCCESS) {
-               unlock_session();
+       lock_session(session); /* unlocked later in the function */
+
+       ok = imap_select(session, IMAP_FOLDER(folder), msginfo->folder,
+                        NULL, NULL, NULL, NULL, NULL, FALSE);
+       if (ok != MAILIMAP_NO_ERROR) {
                return ok;
        }
 
-       destdir = imap_get_real_path(IMAP_FOLDER(folder), dest->path);
-       seq_list = imap_get_lep_set_from_msglist(msglist);
+       unlock_session(session);
+
+       destdir = imap_get_real_path(session, IMAP_FOLDER(folder), dest->path);
+       seq_list = imap_get_lep_set_from_msglist(IMAP_FOLDER(folder), msglist);
        uid_mapping = g_relation_new(2);
        g_relation_index(uid_mapping, 0, g_direct_hash, g_direct_equal);
        
@@ -1243,20 +1765,33 @@ static gint imap_do_copy_msgs(Folder *folder, FolderItem *dest,
                debug_print("Copying messages from %s to %s ...\n",
                            src->path, destdir);
 
+               lock_session(session); /* unlocked later in the function */
                ok = imap_cmd_copy(session, seq_set, destdir, uid_mapping,
                        &source, &dest);
                
-               if (ok == IMAP_SUCCESS) {
-                       if (single && relation && source && dest) {
-                               clistiter *l = clist_begin(source->set_list);
-                               struct mailimap_set_item *i = (struct mailimap_set_item *)clist_content(l);
-                               int snum = i->set_first;
-                               int dnum = 0;
-                               l = clist_begin(dest->set_list);
-                               i = (struct mailimap_set_item *)clist_content(l);
-                               dnum = i->set_first;
-                               g_relation_insert(uid_mapping, GINT_TO_POINTER(snum), 
-                                       GINT_TO_POINTER(dnum));
+               if (is_fatal(ok)) {
+                       session = NULL;
+               }
+
+               if (ok == MAILIMAP_NO_ERROR) {
+                       unlock_session(session);
+                       if (relation && source && dest) {
+                               GSList *s_list = flatten_mailimap_set(source);
+                               GSList *d_list = flatten_mailimap_set(dest);
+                               GSList *s_cur, *d_cur;
+                               if (g_slist_length(s_list) == g_slist_length(d_list)) {
+
+                                       for (s_cur = s_list, d_cur = d_list; 
+                                            s_cur && d_cur; 
+                                            s_cur = s_cur->next, d_cur = d_cur->next) {
+                                               g_relation_insert(uid_mapping, s_cur->data, d_cur->data);
+                                       }
+
+                               } else {
+                                       debug_print("hhhmm, source list length != dest list length.\n");
+                               }
+                               g_slist_free(s_list);
+                               g_slist_free(d_list);
                        }
                }
 
@@ -1266,10 +1801,10 @@ static gint imap_do_copy_msgs(Folder *folder, FolderItem *dest,
                if (dest)
                        mailimap_set_free(dest);
 
-               if (ok != IMAP_SUCCESS) {
+               if (ok != MAILIMAP_NO_ERROR) {
                        g_relation_destroy(uid_mapping);
                        imap_lep_set_free(seq_list);
-                       unlock_session();
+                       statusbar_pop_all();
                        return -1;
                }
        }
@@ -1287,7 +1822,7 @@ static gint imap_do_copy_msgs(Folder *folder, FolderItem *dest,
                                          GINT_TO_POINTER(num));
                        if (num > last_num)
                                last_num = num;
-                       debug_print("copied new message as %d\n", num);
+                       debug_print("copied message %d as %d\n", msginfo->msgnum, num);
                        /* put the local file in the imapcache, so that we don't
                         * have to fetch it back later. */
                        if (num > 0) {
@@ -1328,8 +1863,8 @@ static gint imap_do_copy_msgs(Folder *folder, FolderItem *dest,
        g_slist_free(IMAP_FOLDER_ITEM(dest)->uid_list);
        IMAP_FOLDER_ITEM(dest)->uid_list = NULL;
 
-       unlock_session();
-       if (ok == IMAP_SUCCESS)
+       imap_scan_required(folder, dest);
+       if (ok == MAILIMAP_NO_ERROR)
                return last_num;
        else
                return -1;
@@ -1372,28 +1907,30 @@ static gint imap_do_remove_msgs(Folder *folder, FolderItem *dest,
        GSList *numlist = NULL, *cur;
        MsgInfo *msginfo;
        IMAPSession *session;
-       gint ok = IMAP_SUCCESS;
+       gint ok = MAILIMAP_NO_ERROR;
        GRelation *uid_mapping;
        
        g_return_val_if_fail(folder != NULL, -1);
        g_return_val_if_fail(dest != NULL, -1);
        g_return_val_if_fail(msglist != NULL, -1);
 
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        if (!session) {
                return -1;
        }
-       lock_session();
+
+       lock_session(session); /* unlocked later in the function */
+
        msginfo = (MsgInfo *)msglist->data;
 
-       ok = imap_select(session, IMAP_FOLDER(folder), msginfo->folder->path,
-                        NULL, NULL, NULL, NULL, FALSE);
-       if (ok != IMAP_SUCCESS) {
-               unlock_session();
+       ok = imap_select(session, IMAP_FOLDER(folder), msginfo->folder,
+                        NULL, NULL, NULL, NULL, NULL, FALSE);
+       if (ok != MAILIMAP_NO_ERROR) {
                return ok;
        }
 
-       destdir = imap_get_real_path(IMAP_FOLDER(folder), dest->path);
+       destdir = imap_get_real_path(session, IMAP_FOLDER(folder), dest->path);
        for (cur = msglist; cur; cur = cur->next) {
                msginfo = (MsgInfo *)cur->data;
                if (!MSG_IS_DELETED(msginfo->flags))
@@ -1404,21 +1941,23 @@ static gint imap_do_remove_msgs(Folder *folder, FolderItem *dest,
        uid_mapping = g_relation_new(2);
        g_relation_index(uid_mapping, 0, g_direct_hash, g_direct_equal);
 
-       ok = imap_set_message_flags
-               (IMAP_SESSION(REMOTE_FOLDER(folder)->session),
-               numlist, IMAP_FLAG_DELETED, TRUE);
-       if (ok != IMAP_SUCCESS) {
-               log_warning(_("can't set deleted flags\n"));
-               unlock_session();
-               return ok;
-       }
+       if (numlist != NULL) {
+               ok = imap_set_message_flags
+                       (session, numlist, IMAP_FLAG_DELETED, NULL, TRUE);
+               if (ok != MAILIMAP_NO_ERROR) {
+                       log_warning(LOG_PROTOCOL, _("can't set deleted flags\n"));
+                       return ok;
+               }
+       } /* else we just need to expunge */
        ok = imap_cmd_expunge(session);
-       if (ok != IMAP_SUCCESS) {
-               log_warning(_("can't expunge\n"));
-               unlock_session();
+       if (ok != MAILIMAP_NO_ERROR) {
+               log_warning(LOG_PROTOCOL, _("can't expunge\n"));
                return ok;
        }
        
+       session->folder_content_changed = TRUE;
+       unlock_session(session);
+
        dir = folder_item_get_path(msginfo->folder);
        if (is_dir_exist(dir)) {
                for (cur = msglist; cur; cur = cur->next) {
@@ -1431,9 +1970,10 @@ static gint imap_do_remove_msgs(Folder *folder, FolderItem *dest,
        g_relation_destroy(uid_mapping);
        g_slist_free(numlist);
 
+       imap_scan_required(folder, dest);
+
        g_free(destdir);
-       unlock_session();
-       if (ok == IMAP_SUCCESS)
+       if (ok == MAILIMAP_NO_ERROR)
                return 0;
        else
                return -1;
@@ -1475,7 +2015,7 @@ static gint imap_close(Folder *folder, FolderItem *item)
        return 0;
 }
 
-static gint imap_scan_tree(Folder *folder)
+static gint imap_scan_tree_real(Folder *folder, gboolean subs_only)
 {
        FolderItem *item = NULL;
        IMAPSession *session;
@@ -1484,6 +2024,7 @@ static gint imap_scan_tree(Folder *folder)
        g_return_val_if_fail(folder != NULL, -1);
        g_return_val_if_fail(folder->account != NULL, -1);
 
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        if (!session) {
                if (!folder->node) {
@@ -1495,43 +2036,50 @@ static gint imap_scan_tree(Folder *folder)
                return -1;
        }
 
-       lock_session();
        if (folder->account->imap_dir && *folder->account->imap_dir) {
                gchar *real_path;
                int r;
                clist * lep_list;
 
-               Xstrdup_a(root_folder, folder->account->imap_dir, {unlock_session();return -1;});
+               Xstrdup_a(root_folder, folder->account->imap_dir, {return -1;});
                extract_quote(root_folder, '"');
                subst_char(root_folder,
-                          imap_get_path_separator(IMAP_FOLDER(folder),
+                          imap_get_path_separator(session, IMAP_FOLDER(folder),
                                                   root_folder),
                           '/');
                strtailchomp(root_folder, '/');
                real_path = imap_get_real_path
-                       (IMAP_FOLDER(folder), root_folder);
+                       (session, IMAP_FOLDER(folder), root_folder);
                debug_print("IMAP root directory: %s\n", real_path);
 
                /* check if root directory exist */
 
                r = imap_threaded_list(session->folder, "", real_path,
                                       &lep_list);
+
+               if (r != MAILIMAP_NO_ERROR)
+                       imap_handle_error(SESSION(session), r);
+
                if ((r != MAILIMAP_NO_ERROR) || (clist_count(lep_list) == 0)) {
                        if (!folder->node) {
                                item = folder_item_new(folder, folder->name, NULL);
                                item->folder = folder;
                                folder->node = item->node = g_node_new(item);
                        }
-                       unlock_session();
                        return -1;
                }
                mailimap_list_result_free(lep_list);
-               
+                               
                g_free(real_path);
        }
 
        if (folder->node)
                item = FOLDER_ITEM(folder->node->data);
+               
+       if (item && !item->path && root_folder) {
+               item->path = g_strdup(root_folder);
+       }
+
        if (!item || ((item->path || root_folder) &&
                      strcmp2(item->path, root_folder) != 0)) {
                folder_tree_destroy(folder);
@@ -1540,14 +2088,23 @@ static gint imap_scan_tree(Folder *folder)
                folder->node = item->node = g_node_new(item);
        }
 
-       imap_scan_tree_recursive(session, FOLDER_ITEM(folder->node->data));
+       imap_scan_tree_recursive(session, FOLDER_ITEM(folder->node->data), subs_only);
        imap_create_missing_folders(folder);
-       unlock_session();
 
        return 0;
 }
 
-static gint imap_scan_tree_recursive(IMAPSession *session, FolderItem *item)
+static gint imap_scan_tree(Folder *folder)
+{
+       gboolean subs_only = FALSE;
+       if (folder->account) {
+               debug_print(" scanning only subs %d\n", folder->account->imap_subsonly);
+               subs_only = folder->account->imap_subsonly;
+       }
+       return imap_scan_tree_real(folder, subs_only);
+}
+
+static gint imap_scan_tree_recursive(IMAPSession *session, FolderItem *item, gboolean subs_only)
 {
        Folder *folder;
        IMAPFolder *imapfolder;
@@ -1568,7 +2125,7 @@ static gint imap_scan_tree_recursive(IMAPSession *session, FolderItem *item)
        folder = item->folder;
        imapfolder = IMAP_FOLDER(folder);
 
-       separator = imap_get_path_separator(imapfolder, item->path);
+       separator = imap_get_path_separator(session, imapfolder, item->path);
 
        if (folder->ui_func)
                folder->ui_func(folder, item, folder->ui_func_data);
@@ -1577,7 +2134,7 @@ static gint imap_scan_tree_recursive(IMAPSession *session, FolderItem *item)
                wildcard[0] = separator;
                wildcard[1] = '%';
                wildcard[2] = '\0';
-               real_path = imap_get_real_path(imapfolder, item->path);
+               real_path = imap_get_real_path(session, imapfolder, item->path);
        } else {
                wildcard[0] = '%';
                wildcard[1] = '\0';
@@ -1585,10 +2142,16 @@ static gint imap_scan_tree_recursive(IMAPSession *session, FolderItem *item)
        }
 
        Xstrcat_a(wildcard_path, real_path, wildcard,
-                 {g_free(real_path); return IMAP_ERROR;});
+                 {g_free(real_path); return MAILIMAP_ERROR_BAD_STATE;});
        lep_list = NULL;
-       r = imap_threaded_list(folder, "", wildcard_path, &lep_list);
+       
+       if (subs_only)
+               r = imap_threaded_lsub(folder, "", wildcard_path, &lep_list);
+       else
+               r = imap_threaded_list(folder, "", wildcard_path, &lep_list);
+
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                item_list = NULL;
        }
        else {
@@ -1613,9 +2176,13 @@ static gint imap_scan_tree_recursive(IMAPSession *session, FolderItem *item)
                        }
                }
                if (!new_item) {
-                       debug_print("folder '%s' not found. removing...\n",
-                                   old_item->path);
-                       folder_item_remove(old_item);
+                       if (old_item && old_item->path && !strcmp(old_item->path, "INBOX")) {
+                               debug_print("not removing INBOX\n");
+                       } else {
+                               debug_print("folder '%s' not found. removing...\n",
+                                           old_item->path);
+                               folder_item_remove(old_item);
+                       }
                } else {
                        old_item->no_sub = new_item->no_sub;
                        old_item->no_select = new_item->no_select;
@@ -1675,12 +2242,110 @@ static gint imap_scan_tree_recursive(IMAPSession *session, FolderItem *item)
                }
 
                if (new_item->no_sub == FALSE)
-                       imap_scan_tree_recursive(session, new_item);
+                       imap_scan_tree_recursive(session, new_item, subs_only);
+       }
+
+       g_slist_free(item_list);
+
+       return MAILIMAP_NO_ERROR;
+}
+
+GList *imap_scan_subtree(Folder *folder, FolderItem *item, gboolean unsubs_only, gboolean recursive)
+{
+       IMAPSession *session = imap_session_get(folder);
+       gchar *real_path;
+       gchar *wildcard_path;
+       gchar separator;
+       gchar wildcard[3];
+       clist * lep_list;
+       GSList *item_list = NULL, *cur;
+       GList *child_list = NULL, *tmplist = NULL;
+       GSList *sub_list = NULL;
+       int r;
+
+       if (!session)
+               return NULL;
+
+       separator = imap_get_path_separator(session, IMAP_FOLDER(folder), item->path);
+
+       if (item->path) {
+               wildcard[0] = separator;
+               wildcard[1] = '%';
+               wildcard[2] = '\0';
+               real_path = imap_get_real_path(session, IMAP_FOLDER(folder), item->path);
+       } else {
+               wildcard[0] = '%';
+               wildcard[1] = '\0';
+               real_path = g_strdup("");
+       }
+
+       Xstrcat_a(wildcard_path, real_path, wildcard,
+                 {g_free(real_path); return NULL;});
+       lep_list = NULL;
+       
+       if (unsubs_only)
+               statusbar_print_all(_("Looking for unsubscribed folders in %s..."), 
+                               item->path?item->path:item->name);
+       else
+               statusbar_print_all(_("Looking for subfolders of %s..."), 
+                               item->path?item->path:item->name);
+
+       r = imap_threaded_list(folder, "", wildcard_path, &lep_list);
+       if (r) {
+               statusbar_pop_all();
+               return NULL;
        }
+       item_list = imap_list_from_lep(IMAP_FOLDER(folder),
+                              lep_list, real_path, FALSE);
+       mailimap_list_result_free(lep_list);
 
+       for (cur = item_list; cur != NULL; cur = cur->next) {
+               FolderItem *cur_item = FOLDER_ITEM(cur->data);
+               if (recursive) {
+                       tmplist = imap_scan_subtree(folder, cur_item, 
+                                       unsubs_only, recursive);
+                       if (tmplist)
+                               child_list = g_list_concat(child_list, tmplist);
+               }
+               child_list = g_list_prepend(child_list,
+                               imap_get_real_path(session, 
+                                       IMAP_FOLDER(folder), cur_item->path));
+               
+               folder_item_destroy(cur_item);
+       }
+       child_list = g_list_reverse(child_list);
        g_slist_free(item_list);
 
-       return IMAP_SUCCESS;
+       if (unsubs_only) {
+               r = imap_threaded_lsub(folder, "", wildcard_path, &lep_list);
+               if (r) {
+                       statusbar_pop_all();
+                       return NULL;
+               }
+               sub_list = imap_list_from_lep(IMAP_FOLDER(folder),
+                                      lep_list, real_path, FALSE);
+               mailimap_list_result_free(lep_list);
+
+               for (cur = sub_list; cur != NULL; cur = cur->next) {
+                       FolderItem *cur_item = FOLDER_ITEM(cur->data);
+                       GList *oldlitem = NULL;
+                       gchar *tmp = imap_get_real_path(session, 
+                                       IMAP_FOLDER(folder), cur_item->path);
+                       folder_item_destroy(cur_item);
+                       oldlitem = g_list_find_custom(
+                                       child_list, tmp, (GCompareFunc)strcmp2);
+                       if (oldlitem) {
+                               child_list = g_list_remove_link(child_list, oldlitem);
+                               g_free(oldlitem->data);
+                               g_list_free(oldlitem);
+                       }
+                       g_free(tmp);
+               }
+       }
+
+       statusbar_pop_all();
+
+       return child_list;
 }
 
 static gint imap_create_tree(Folder *folder)
@@ -1805,18 +2470,19 @@ static FolderItem *imap_create_folder(Folder *folder, FolderItem *parent,
        const gchar *p;
        gint ok;
        gboolean no_select = FALSE, no_sub = FALSE;
+       gboolean exist = FALSE;
        
        g_return_val_if_fail(folder != NULL, NULL);
        g_return_val_if_fail(folder->account != NULL, NULL);
        g_return_val_if_fail(parent != NULL, NULL);
        g_return_val_if_fail(name != NULL, NULL);
 
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        if (!session) {
                return NULL;
        }
 
-       lock_session();
        if (!folder_item_parent(parent) && strcmp(name, "INBOX") == 0) {
                dirpath = g_strdup(name);
        }else if (parent->path)
@@ -1826,7 +2492,7 @@ static FolderItem *imap_create_folder(Folder *folder, FolderItem *parent,
        else if (folder->account->imap_dir && *folder->account->imap_dir) {
                gchar *imap_dir;
 
-               Xstrdup_a(imap_dir, folder->account->imap_dir, {unlock_session();return NULL;});
+               Xstrdup_a(imap_dir, folder->account->imap_dir, {return NULL;});
                strtailchomp(imap_dir, '/');
                dirpath = g_strconcat(imap_dir, "/", name, NULL);
        } else
@@ -1836,34 +2502,32 @@ static FolderItem *imap_create_folder(Folder *folder, FolderItem *parent,
 
        /* keep trailing directory separator to create a folder that contains
           sub folder */
-       imap_path = imap_utf8_to_modified_utf7(dirpath);
+       imap_path = imap_utf8_to_modified_utf7(dirpath, FALSE);
 
        strtailchomp(dirpath, '/');
        Xstrdup_a(new_name, name, {
                g_free(dirpath); 
-               unlock_session();               
                return NULL;});
 
-       separator = imap_get_path_separator(IMAP_FOLDER(folder), imap_path);
+       separator = imap_get_path_separator(session, IMAP_FOLDER(folder), imap_path);
        imap_path_separator_subst(imap_path, separator);
        /* remove trailing / for display */
        strtailchomp(new_name, '/');
 
        if (strcmp(dirpath, "INBOX") != 0) {
                GPtrArray *argbuf;
-               gboolean exist = FALSE;
                int r;
                clist * lep_list;
                
                argbuf = g_ptr_array_new();
                r = imap_threaded_list(folder, "", imap_path, &lep_list);
                if (r != MAILIMAP_NO_ERROR) {
-                       log_warning(_("can't create mailbox: LIST failed\n"));
+                       imap_handle_error(SESSION(session), r);
+                       log_warning(LOG_PROTOCOL, _("can't create mailbox: LIST failed\n"));
                        g_free(imap_path);
                        g_free(dirpath);
                        ptr_array_free_strings(argbuf);
                        g_ptr_array_free(argbuf, TRUE);
-                       unlock_session();
                        return NULL;
                }
                
@@ -1873,11 +2537,10 @@ static FolderItem *imap_create_folder(Folder *folder, FolderItem *parent,
                lep_list = NULL;
                if (!exist) {
                        ok = imap_cmd_create(session, imap_path);
-                       if (ok != IMAP_SUCCESS) {
-                               log_warning(_("can't create mailbox\n"));
+                       if (ok != MAILIMAP_NO_ERROR) {
+                               log_warning(LOG_PROTOCOL, _("can't create mailbox\n"));
                                g_free(imap_path);
                                g_free(dirpath);
-                               unlock_session();
                                return NULL;
                        }
                        r = imap_threaded_list(folder, "", imap_path, &lep_list);
@@ -1891,9 +2554,11 @@ static FolderItem *imap_create_folder(Folder *folder, FolderItem *parent,
                                        g_slist_free(item_list);
                                } 
                                mailimap_list_result_free(lep_list);
+                       } else {
+                               imap_handle_error(SESSION(session), r);
                        }
-
                }
+               imap_threaded_subscribe(folder, imap_path, TRUE);
        } else {
                clist *lep_list;
                int r;
@@ -1909,6 +2574,8 @@ static FolderItem *imap_create_folder(Folder *folder, FolderItem *parent,
                                g_slist_free(item_list);
                        } 
                        mailimap_list_result_free(lep_list);
+               } else {
+                       imap_handle_error(SESSION(session), r);
                }
        }
 
@@ -1923,7 +2590,13 @@ static FolderItem *imap_create_folder(Folder *folder, FolderItem *parent,
        if (!is_dir_exist(dirpath))
                make_dir_hier(dirpath);
        g_free(dirpath);
-       unlock_session();
+
+       if (exist) {
+               /* folder existed, scan it */
+               imap_scan_required(folder, new_item);
+               folder_item_scan_full(new_item, FALSE);
+       }
+
        return new_item;
 }
 
@@ -1948,32 +2621,33 @@ static gint imap_rename_folder(Folder *folder, FolderItem *item,
        g_return_val_if_fail(item->path != NULL, -1);
        g_return_val_if_fail(name != NULL, -1);
 
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        if (!session) {
                return -1;
        }
-       lock_session();
 
-       if (strchr(name, imap_get_path_separator(IMAP_FOLDER(folder), item->path)) != NULL) {
+       if (strchr(name, imap_get_path_separator(session, IMAP_FOLDER(folder), item->path)) != NULL) {
                g_warning(_("New folder name must not contain the namespace "
                            "path separator"));
-               unlock_session();
                return -1;
        }
 
-       real_oldpath = imap_get_real_path(IMAP_FOLDER(folder), item->path);
+       real_oldpath = imap_get_real_path(session, IMAP_FOLDER(folder), item->path);
 
        g_free(session->mbox);
        session->mbox = NULL;
+       session->exists = 0;
+       session->recent = 0;
+       session->expunge = 0;
        ok = imap_cmd_examine(session, "INBOX",
                              &exists, &recent, &unseen, &uid_validity, FALSE);
-       if (ok != IMAP_SUCCESS) {
+       if (ok != MAILIMAP_NO_ERROR) {
                g_free(real_oldpath);
-               unlock_session();
                return -1;
        }
 
-       separator = imap_get_path_separator(IMAP_FOLDER(folder), item->path);
+       separator = imap_get_path_separator(session, IMAP_FOLDER(folder), item->path);
        if (strchr(item->path, G_DIR_SEPARATOR)) {
                dirpath = g_path_get_dirname(item->path);
                newpath = g_strconcat(dirpath, G_DIR_SEPARATOR_S, name, NULL);
@@ -1981,20 +2655,18 @@ static gint imap_rename_folder(Folder *folder, FolderItem *item,
        } else
                newpath = g_strdup(name);
 
-       real_newpath = imap_utf8_to_modified_utf7(newpath);
+       real_newpath = imap_utf8_to_modified_utf7(newpath, FALSE);
        imap_path_separator_subst(real_newpath, separator);
 
        ok = imap_cmd_rename(session, real_oldpath, real_newpath);
-       if (ok != IMAP_SUCCESS) {
-               log_warning(_("can't rename mailbox: %s to %s\n"),
+       if (ok != MAILIMAP_NO_ERROR) {
+               log_warning(LOG_PROTOCOL, _("can't rename mailbox: %s to %s\n"),
                            real_oldpath, real_newpath);
                g_free(real_oldpath);
                g_free(newpath);
                g_free(real_newpath);
-               unlock_session();
                return -1;
        }
-
        g_free(item->name);
        item->name = g_strdup(name);
 
@@ -2018,41 +2690,79 @@ static gint imap_rename_folder(Folder *folder, FolderItem *item,
        g_free(newpath);
        g_free(real_oldpath);
        g_free(real_newpath);
-       unlock_session();
        return 0;
 }
 
+gint imap_subscribe(Folder *folder, FolderItem *item, gchar *rpath, gboolean sub)
+{
+       gchar *path;
+       gint r = -1;
+       IMAPSession *session;
+       debug_print("getting session...\n");
+
+       session = imap_session_get(folder);
+       if (!session) {
+               return -1;
+       }
+       if (item && item->path) {
+               path = imap_get_real_path(session, IMAP_FOLDER(folder), item->path);
+               if (!path)
+                       return -1;
+               if (!strcmp(path, "INBOX") && sub == FALSE)
+                       return -1;
+               debug_print("%ssubscribing %s\n", sub?"":"un", path);
+               r = imap_threaded_subscribe(folder, path, sub);
+               g_free(path);
+       } else if (rpath) {
+               r = imap_threaded_subscribe(folder, rpath, sub);
+       } else
+               return -1;
+       return r;
+}
+
 static gint imap_remove_folder_real(Folder *folder, FolderItem *item)
 {
        gint ok;
        IMAPSession *session;
        gchar *path;
        gchar *cache_dir;
+       gboolean selected_folder;
 
        g_return_val_if_fail(folder != NULL, -1);
        g_return_val_if_fail(item != NULL, -1);
        g_return_val_if_fail(item->path != NULL, -1);
 
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        if (!session) {
                return -1;
        }
-       lock_session();
-       path = imap_get_real_path(IMAP_FOLDER(folder), item->path);
+       path = imap_get_real_path(session, IMAP_FOLDER(folder), item->path);
 
+       imap_threaded_subscribe(folder, path, FALSE);
+
+       selected_folder = (session->mbox != NULL) &&
+                         (!strcmp(session->mbox, item->path));
+       if (selected_folder) {
+               ok = imap_cmd_close(session);
+               if (ok != MAILIMAP_NO_ERROR) {
+                       debug_print("close err %d\n", ok);
+                       imap_handle_error(SESSION(session), ok);
+                       return ok;
+               }
+       }
        ok = imap_cmd_delete(session, path);
-       if (ok != IMAP_SUCCESS) {
+       if (ok != MAILIMAP_NO_ERROR) {
                gchar *tmp = g_strdup_printf("%s%c", path, 
-                               imap_get_path_separator(IMAP_FOLDER(folder), path));
+                               imap_get_path_separator(session, IMAP_FOLDER(folder), path));
                g_free(path);
                path = tmp;
                ok = imap_cmd_delete(session, path);
        }
 
-       if (ok != IMAP_SUCCESS) {
-               log_warning(_("can't delete mailbox\n"));
+       if (ok != MAILIMAP_NO_ERROR) {
+               log_warning(LOG_PROTOCOL, _("can't delete mailbox\n"));
                g_free(path);
-               unlock_session();
                return -1;
        }
 
@@ -2062,7 +2772,6 @@ static gint imap_remove_folder_real(Folder *folder, FolderItem *item)
                g_warning("can't remove directory '%s'\n", cache_dir);
        g_free(cache_dir);
        folder_item_remove(item);
-       unlock_session();
        return 0;
 }
 
@@ -2095,6 +2804,7 @@ typedef struct _uncached_data {
        guint cur;
        guint total;
        gboolean done;
+       int ok;
 } uncached_data;
 
 static void *imap_get_uncached_messages_thread(void *data)
@@ -2103,10 +2813,10 @@ static void *imap_get_uncached_messages_thread(void *data)
        IMAPSession *session = stuff->session;
        FolderItem *item = stuff->item;
        MsgNumberList *numlist = stuff->numlist;
-       
        GSList *newlist = NULL;
        GSList *llast = NULL;
        GSList *seq_list, *cur;
+       gboolean got_alien_tags = FALSE;
 
        debug_print("uncached_messages\n");
        
@@ -2116,7 +2826,7 @@ static void *imap_get_uncached_messages_thread(void *data)
                return NULL;
        }
        
-       seq_list = imap_get_lep_set_from_numlist(numlist);
+       seq_list = imap_get_lep_set_from_numlist(IMAP_FOLDER(item->folder), numlist);
        debug_print("get msgs info\n");
        for (cur = seq_list; cur != NULL; cur = g_slist_next(cur)) {
                struct mailimap_set * imapset;
@@ -2125,24 +2835,59 @@ static void *imap_get_uncached_messages_thread(void *data)
                carray * env_list;
                int count;
                
+               if (session->cancelled)
+                       break;
+               
                imapset = cur->data;
                
                r = imap_threaded_fetch_env(session->folder,
                                            imapset, &env_list);
-               if (r != MAILIMAP_NO_ERROR)
+               if (r != MAILIMAP_NO_ERROR) {
+                       imap_handle_error(SESSION(session), r);
+                       if (is_fatal(r)) {
+                               stuff->ok = r;
+                               return NULL;
+                       }
                        continue;
-               
+               }
+
                session_set_access_time(SESSION(session));
 
                count = 0;
-               for(i = 0 ; i < carray_count(env_list) ; i ++) {
+               for(i = 0 ; i < carray_count(env_list) ; i += 2) {
                        struct imap_fetch_env_info * info;
                        MsgInfo * msginfo;
-                       
+                       GSList *tags = NULL, *cur = NULL;
                        info = carray_get(env_list, i);
+                       tags = carray_get(env_list, i+1);
                        msginfo = imap_envelope_from_lep(info, item);
-                       if (msginfo == NULL)
+                       if (msginfo == NULL) {
+                               slist_free_strings(tags);
+                               g_slist_free(tags);
                                continue;
+                       }
+                       if (tags != NULL) {
+                               g_slist_free(msginfo->tags);
+                               msginfo->tags = NULL;
+                       }
+                       for (cur = tags; cur; cur = cur->next) {
+                               gchar *real_tag = imap_modified_utf7_to_utf8(cur->data, TRUE);
+                               gint id = 0;
+                               id = tags_get_id_for_str(real_tag);
+                               printf("tag %s %d\n", real_tag, id);
+                               if (id == -1) {
+                                       id = tags_add_tag(real_tag);
+                                       got_alien_tags = TRUE;
+                               }
+                               if (!g_slist_find(msginfo->tags, GINT_TO_POINTER(id))) {
+                                       msginfo->tags = g_slist_append(
+                                                       msginfo->tags,
+                                                       GINT_TO_POINTER(id));
+                               }
+                               g_free(real_tag);
+                       }
+                       slist_free_strings(tags);
+                       g_slist_free(tags);
                        msginfo->folder = item;
                        if (!newlist)
                                llast = newlist = g_slist_append(newlist, msginfo);
@@ -2156,6 +2901,11 @@ static void *imap_get_uncached_messages_thread(void *data)
                imap_fetch_env_free(env_list);
        }
        
+       if (got_alien_tags) {
+               tags_write_tags();
+               main_window_reflect_tags_changes(mainwindow_get_mainwindow());
+       }
+
        for (cur = seq_list; cur != NULL; cur = g_slist_next(cur)) {
                struct mailimap_set * imapset;
                
@@ -2172,7 +2922,8 @@ static void *imap_get_uncached_messages_thread(void *data)
 
 static GSList *imap_get_uncached_messages(IMAPSession *session,
                                        FolderItem *item,
-                                       MsgNumberList *numlist)
+                                       MsgNumberList *numlist,
+                                       int *r)
 {
        GSList *result = NULL;
        GSList * cur;
@@ -2182,6 +2933,7 @@ static GSList *imap_get_uncached_messages(IMAPSession *session,
        finished = 0;
        cur = numlist;
        data->total = g_slist_length(numlist);
+       data->ok = MAILIMAP_NO_ERROR;
        debug_print("messages list : %i\n", data->total);
 
        while (cur != NULL) {
@@ -2218,8 +2970,8 @@ static GSList *imap_get_uncached_messages(IMAPSession *session,
                data->cur += count;
                
                if (prefs_common.work_offline && 
-                   !inc_offline_should_override(
-                       _("Sylpheed-Claws needs network access in order "
+                   !inc_offline_should_override(FALSE,
+                       _("Claws Mail needs network access in order "
                          "to access the IMAP server."))) {
                        g_free(data);
                        return NULL;
@@ -2227,13 +2979,17 @@ static GSList *imap_get_uncached_messages(IMAPSession *session,
                
                partial_result =
                        (GSList *)imap_get_uncached_messages_thread(data);
-               
+               *r = data->ok;
+               if (data->ok != MAILIMAP_NO_ERROR) {
+                       goto bail;
+               }
                statusbar_progress_all(data->cur,data->total, 1);
                
                g_slist_free(newlist);
                
                result = g_slist_concat(result, partial_result);
        }
+bail:
        g_free(data);
        
        statusbar_progress_all(0,0,0);
@@ -2260,51 +3016,13 @@ static void imap_delete_all_cached_messages(FolderItem *item)
        debug_print("done.\n");
 }
 
-static IMAPNameSpace *imap_find_namespace_from_list(GList *ns_list,
-                                                   const gchar *path)
-{
-       IMAPNameSpace *namespace = NULL;
-       gchar *tmp_path, *name;
-
-       if (!path) path = "";
-
-       for (; ns_list != NULL; ns_list = ns_list->next) {
-               IMAPNameSpace *tmp_ns = ns_list->data;
-
-               Xstrcat_a(tmp_path, path, "/", return namespace);
-               Xstrdup_a(name, tmp_ns->name, return namespace);
-               if (tmp_ns->separator && tmp_ns->separator != '/') {
-                       subst_char(tmp_path, tmp_ns->separator, '/');
-                       subst_char(name, tmp_ns->separator, '/');
-               }
-               if (strncmp(tmp_path, name, strlen(name)) == 0)
-                       namespace = tmp_ns;
-       }
-
-       return namespace;
-}
-
-static IMAPNameSpace *imap_find_namespace(IMAPFolder *folder,
-                                         const gchar *path)
-{
-       IMAPNameSpace *namespace;
-
-       g_return_val_if_fail(folder != NULL, NULL);
-
-       namespace = imap_find_namespace_from_list(folder->ns_personal, path);
-       if (namespace) return namespace;
-       namespace = imap_find_namespace_from_list(folder->ns_others, path);
-       if (namespace) return namespace;
-       namespace = imap_find_namespace_from_list(folder->ns_shared, path);
-       if (namespace) return namespace;
-
-       return NULL;
-}
-
 gchar imap_get_path_separator_for_item(FolderItem *item)
 {
        Folder *folder = NULL;
        IMAPFolder *imap_folder = NULL;
+       IMAPSession *session = NULL;
+       gchar result = '/';
+       
        if (!item)
                return '/';
        folder = item->folder;
@@ -2317,33 +3035,49 @@ gchar imap_get_path_separator_for_item(FolderItem *item)
        if (!imap_folder)
                return '/';
        
-       return imap_get_path_separator(imap_folder, item->path);
+       debug_print("getting session...");
+       session = imap_session_get(FOLDER(folder));
+       result = imap_get_path_separator(session, imap_folder, item->path);
+       return result;
 }
 
-static gchar imap_get_path_separator(IMAPFolder *folder, const gchar *path)
+static gchar imap_refresh_path_separator(IMAPSession *session, IMAPFolder *folder, const gchar *subfolder)
 {
-       IMAPNameSpace *namespace;
-       gchar separator = '/';
-       IMAPSession *session = imap_session_get(FOLDER(folder));
+       clist * lep_list;
+       int r;
+       gchar separator = '\0';
+       
        g_return_val_if_fail(session != NULL, '/');
+       r = imap_threaded_list((Folder *)folder, "", subfolder, &lep_list);
+       
+       if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
+               log_warning(LOG_PROTOCOL, _("LIST failed\n"));
+               return '\0';
+       }
+
+       if (clist_count(lep_list) > 0) {
+               clistiter * iter = clist_begin(lep_list); 
+               struct mailimap_mailbox_list * mb;
+               mb = clist_content(iter);
+
+               separator = mb->mb_delimiter;
+               debug_print("got separator: %c\n", folder->last_seen_separator);
+       }
+       mailimap_list_result_free(lep_list);
+       return separator;
+}
+
+static gchar imap_get_path_separator(IMAPSession *session, IMAPFolder *folder, const gchar *path)
+{
+       gchar separator = '/';
 
        if (folder->last_seen_separator == 0) {
-               clist * lep_list;
-               int r = imap_threaded_list((Folder *)folder, "", "", &lep_list);
-               if (r != MAILIMAP_NO_ERROR) {
-                       log_warning(_("LIST failed\n"));
-                       return '/';
-               }
-               
-               if (clist_count(lep_list) > 0) {
-                       clistiter * iter = clist_begin(lep_list); 
-                       struct mailimap_mailbox_list * mb;
-                       mb = clist_content(iter);
-               
-                       folder->last_seen_separator = mb->mb_delimiter;
-                       debug_print("got separator: %c\n", folder->last_seen_separator);
-               }
-               mailimap_list_result_free(lep_list);
+               folder->last_seen_separator = imap_refresh_path_separator(session, folder, "");
+       }
+
+       if (folder->last_seen_separator == 0) {
+               folder->last_seen_separator = imap_refresh_path_separator(session, folder, "INBOX");
        }
 
        if (folder->last_seen_separator != 0) {
@@ -2351,14 +3085,10 @@ static gchar imap_get_path_separator(IMAPFolder *folder, const gchar *path)
                return folder->last_seen_separator;
        }
 
-       namespace = imap_find_namespace(folder, path);
-       if (namespace && namespace->separator)
-               separator = namespace->separator;
-
        return separator;
 }
 
-static gchar *imap_get_real_path(IMAPFolder *folder, const gchar *path)
+static gchar *imap_get_real_path(IMAPSession *session, IMAPFolder *folder, const gchar *path)
 {
        gchar *real_path;
        gchar separator;
@@ -2366,8 +3096,8 @@ static gchar *imap_get_real_path(IMAPFolder *folder, const gchar *path)
        g_return_val_if_fail(folder != NULL, NULL);
        g_return_val_if_fail(path != NULL, NULL);
 
-       real_path = imap_utf8_to_modified_utf7(path);
-       separator = imap_get_path_separator(folder, path);
+       real_path = imap_utf8_to_modified_utf7(path, FALSE);
+       separator = imap_get_path_separator(session, folder, path);
        imap_path_separator_subst(real_path, separator);
 
        return real_path;
@@ -2376,26 +3106,60 @@ static gchar *imap_get_real_path(IMAPFolder *folder, const gchar *path)
 static gint imap_set_message_flags(IMAPSession *session,
                                   MsgNumberList *numlist,
                                   IMAPFlags flags,
+                                  GSList *tags,
                                   gboolean is_set)
 {
        gint ok = 0;
        GSList *seq_list;
        GSList * cur;
+       gint total = 0;
+       IMAPFolder *folder = NULL;
+       GSList *sorted_list = NULL;
+
+       if (numlist == NULL || session == NULL)
+               return MAILIMAP_ERROR_BAD_STATE;
+       
+       folder = IMAP_FOLDER(session->folder);
+       
+       sorted_list = g_slist_copy(numlist);
+       sorted_list = g_slist_sort(sorted_list, g_int_compare);
+       
+       cur = g_slist_last(sorted_list);
 
-       seq_list = imap_get_lep_set_from_numlist(numlist);
+       if (cur)
+               total = GPOINTER_TO_INT(cur->data);
        
+       seq_list = imap_get_lep_set_from_numlist(IMAP_FOLDER(session->folder), sorted_list);
+
+       statusbar_print_all(_("Flagging messages..."));
+
        for(cur = seq_list ; cur != NULL ; cur = g_slist_next(cur)) {
-               struct mailimap_set * imapset;
-               
-               imapset = cur->data;
-               
+               struct mailimap_set * imapset = (struct mailimap_set *)cur->data;
+               struct mailimap_set_item *set_item = clist_content(
+                                                       clist_begin(imapset->set_list));
+
+               statusbar_progress_all(set_item->set_first, total, 1);
+
                ok = imap_cmd_store(session, imapset,
-                                   flags, is_set);
+                                   flags, tags, is_set);
+               statusbar_progress_all(set_item->set_last, total, 1);
+               if (ok != MAILIMAP_NO_ERROR && folder->max_set_size > 20) {
+                       /* reduce max set size */
+                       folder->max_set_size /= 2;
+               }
+               if (ok != MAILIMAP_NO_ERROR && is_fatal(ok)) {
+                       break;
+               }
        }
        
+       g_slist_free(sorted_list);
+
+       statusbar_progress_all(0,0,0);
+       statusbar_pop_all();
+
        imap_lep_set_free(seq_list);
        
-       return IMAP_SUCCESS;
+       return ok;
 }
 
 typedef struct _select_data {
@@ -2409,18 +3173,31 @@ typedef struct _select_data {
 } select_data;
 
 static gint imap_select(IMAPSession *session, IMAPFolder *folder,
-                       const gchar *path,
+                       FolderItem *item,
                        gint *exists, gint *recent, gint *unseen,
-                       guint32 *uid_validity, gboolean block)
+                       guint32 *uid_validity, gint *can_create_flags,
+                       gboolean block)
 {
        gchar *real_path;
        gint ok;
        gint exists_, recent_, unseen_;
        guint32 uid_validity_;
-       
-       if (!exists && !recent && !unseen && !uid_validity) {
+       gint can_create_flags_;
+       const gchar *path = item ? item->path:NULL;
+
+       if (!item) {
+               return MAILIMAP_ERROR_BAD_STATE;
+       }
+
+       if (!exists && !recent && !unseen && !uid_validity && !can_create_flags) {
                if (session->mbox && strcmp(session->mbox, path) == 0)
-                       return IMAP_SUCCESS;
+                       return MAILIMAP_NO_ERROR;
+       }
+       if (!exists && !recent && !unseen && !uid_validity && can_create_flags) {
+               if (session->mbox && strcmp(session->mbox, path) == 0) {
+                       if (IMAP_FOLDER_ITEM(item)->can_create_flags != ITEM_CAN_CREATE_FLAGS_UNKNOWN)
+                               return MAILIMAP_NO_ERROR;
+               }
        }
        if (!exists)
                exists = &exists_;
@@ -2430,19 +3207,37 @@ static gint imap_select(IMAPSession *session, IMAPFolder *folder,
                unseen = &unseen_;
        if (!uid_validity)
                uid_validity = &uid_validity_;
+       if (!can_create_flags)
+               can_create_flags = &can_create_flags_;
 
        g_free(session->mbox);
        session->mbox = NULL;
+       session->exists = 0;
+       session->recent = 0;
+       session->expunge = 0;
 
-       real_path = imap_get_real_path(folder, path);
+       real_path = imap_get_real_path(session, folder, path);
 
        ok = imap_cmd_select(session, real_path,
-                            exists, recent, unseen, uid_validity, block);
-       if (ok != IMAP_SUCCESS)
-               log_warning(_("can't select folder: %s\n"), real_path);
-       else {
+                            exists, recent, unseen, uid_validity, can_create_flags, block);
+       if (ok != MAILIMAP_NO_ERROR) {
+               log_warning(LOG_PROTOCOL, _("can't select folder: %s\n"), real_path);
+       else {
                session->mbox = g_strdup(path);
                session->folder_content_changed = FALSE;
+               session->exists = *exists;
+               session->recent = *recent;
+               session->expunge = 0;
+               session->unseen = *unseen;
+               session->uid_validity = *uid_validity;
+               debug_print("select: exists %d recent %d expunge %d uid_validity %d can_create_flags %d\n", 
+                       session->exists, session->recent, session->expunge,
+                       session->uid_validity, *can_create_flags);
+       }
+       if (*can_create_flags) {
+               IMAP_FOLDER_ITEM(item)->can_create_flags = ITEM_CAN_CREATE_FLAGS;
+       } else {
+               IMAP_FOLDER_ITEM(item)->can_create_flags = ITEM_CANNOT_CREATE_FLAGS;
        }
        g_free(real_path);
 
@@ -2462,33 +3257,49 @@ static gint imap_status(IMAPSession *session, IMAPFolder *folder,
        gchar *real_path;
        guint mask = 0;
        
-       real_path = imap_get_real_path(folder, path);
+       real_path = imap_get_real_path(session, folder, path);
 
        if (messages) {
                mask |= 1 << 0;
+               *messages = 0;
        }
        if (uid_next) {
                mask |= 1 << 2;
+               *uid_next = 0;
        }
        if (uid_validity) {
                mask |= 1 << 3;
+               *uid_validity = 0;
        }
        if (unseen) {
                mask |= 1 << 4;
+               *unseen = 0;
+       }
+       
+       if (session->mbox != NULL &&
+           !strcmp(session->mbox, item->item.path)) {
+               r = imap_cmd_close(session);
+               if (r != MAILIMAP_NO_ERROR) {
+                       imap_handle_error(SESSION(session), r);
+                       debug_print("close err %d\n", r);
+                       return r;
+               }
        }
+       
        r = imap_threaded_status(FOLDER(folder), real_path, 
                &data_status, mask);
 
        g_free(real_path);
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                debug_print("status err %d\n", r);
-               return IMAP_ERROR;
+               return r;
        }
        
        if (data_status->st_info_list == NULL) {
                mailimap_mailbox_data_status_free(data_status);
                debug_print("status->st_info_list == NULL\n");
-               return IMAP_ERROR;
+               return MAILIMAP_ERROR_BAD_STATE;
        }
        
        got_values = 0;
@@ -2499,33 +3310,40 @@ static gint imap_status(IMAPSession *session, IMAPFolder *folder,
                info = clist_content(iter);
                switch (info->st_att) {
                case MAILIMAP_STATUS_ATT_MESSAGES:
-                       * messages = info->st_value;
-                       got_values |= 1 << 0;
+                       if (messages) {
+                               * messages = info->st_value;
+                               got_values |= 1 << 0;
+                       }
                        break;
                        
                case MAILIMAP_STATUS_ATT_UIDNEXT:
-                       * uid_next = info->st_value;
-                       got_values |= 1 << 2;
+                       if (uid_next) {
+                               * uid_next = info->st_value;
+                               got_values |= 1 << 2;
+                       }
                        break;
                        
                case MAILIMAP_STATUS_ATT_UIDVALIDITY:
-                       * uid_validity = info->st_value;
-                       got_values |= 1 << 3;
+                       if (uid_validity) {
+                               * uid_validity = info->st_value;
+                               got_values |= 1 << 3;
+                       }
                        break;
                        
                case MAILIMAP_STATUS_ATT_UNSEEN:
-                       * unseen = info->st_value;
-                       got_values |= 1 << 4;
+                       if (unseen) {
+                               * unseen = info->st_value;
+                               got_values |= 1 << 4;
+                       }
                        break;
                }
        }
        mailimap_mailbox_data_status_free(data_status);
        
        if (got_values != mask) {
-               debug_print("status: incomplete values received (%d)\n", got_values);
-               return IMAP_ERROR;
+               g_warning("status: incomplete values received (%d)\n", got_values);
        }
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 static void imap_free_capabilities(IMAPSession *session)
@@ -2545,49 +3363,51 @@ static gint imap_cmd_login(IMAPSession *session,
        gint ok;
 
        if (!strcmp(type, "LOGIN") && imap_has_capability(session, "LOGINDISABLED")) {
-               gint ok = IMAP_ERROR;
+               gint ok = MAILIMAP_ERROR_BAD_STATE;
                if (imap_has_capability(session, "STARTTLS")) {
-#if USE_OPENSSL
-                       log_warning(_("Server requires TLS to log in.\n"));
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
+                       log_warning(LOG_PROTOCOL, _("Server requires TLS to log in.\n"));
                        ok = imap_cmd_starttls(session);
-                       if (ok != IMAP_SUCCESS) {
-                               log_warning(_("Can't start TLS session.\n"));
-                               return IMAP_ERROR;
+                       if (ok != MAILIMAP_NO_ERROR) {
+                               log_warning(LOG_PROTOCOL, _("Can't start TLS session.\n"));
+                               return ok;
                        } else {
                                /* refresh capas */
                                imap_free_capabilities(session);
-                               if (imap_get_capabilities(session) != MAILIMAP_NO_ERROR) {
-                                       log_warning(_("Can't refresh capabilities.\n"));
-                                       return IMAP_ERROR;
+                               if ((r = imap_get_capabilities(session)) != MAILIMAP_NO_ERROR) {
+                                       imap_handle_error(SESSION(session), r);
+                                       log_warning(LOG_PROTOCOL, _("Can't refresh capabilities.\n"));
+                                       return r;
                                }
                        }
 #else          
-                       log_error(_("Connection to %s failed: "
-                                       "server requires TLS, but Sylpheed-Claws "
+                       log_error(LOG_PROTOCOL, _("Connection to %s failed: "
+                                       "server requires TLS, but Claws Mail "
                                        "has been compiled without OpenSSL "
                                        "support.\n"),
                                        SESSION(session)->server);
-                       return IMAP_ERROR;
+                       return MAILIMAP_ERROR_BAD_STATE;
 #endif
                } else {
-                       log_error(_("Server logins are disabled.\n"));
-                       return IMAP_ERROR;
+                       log_error(LOG_PROTOCOL, _("Server logins are disabled.\n"));
+                       return MAILIMAP_ERROR_BAD_STATE;
                }
        }
 
-       log_print("IMAP4> Logging %s to %s using %s\n", 
+       log_print(LOG_PROTOCOL, "IMAP4> Logging %s to %s using %s\n", 
                        user,
                        SESSION(session)->server,
                        type);
        r = imap_threaded_login(session->folder, user, pass, type);
        if (r != MAILIMAP_NO_ERROR) {
-               log_print("IMAP4< Error logging in to %s\n",
+               imap_handle_error(SESSION(session), r);
+               log_print(LOG_PROTOCOL, "IMAP4< Error logging in to %s\n",
                                SESSION(session)->server);
-               ok = IMAP_ERROR;
+               ok = r;
        } else {
-               log_print("IMAP4< Login to %s successful\n",
+               log_print(LOG_PROTOCOL, "IMAP4< Login to %s successful\n",
                                SESSION(session)->server);
-               ok = IMAP_SUCCESS;
+               ok = MAILIMAP_NO_ERROR;
        }
        return ok;
 }
@@ -2595,47 +3415,91 @@ static gint imap_cmd_login(IMAPSession *session,
 static gint imap_cmd_noop(IMAPSession *session)
 {
        int r;
-       unsigned int exists;
+       unsigned int exists, recent, expunge, unseen, uidnext, uidval;
        
-       r = imap_threaded_noop(session->folder, &exists);
+       r = imap_threaded_noop(session->folder, &exists, &recent, &expunge, &unseen, &uidnext, &uidval);
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                debug_print("noop err %d\n", r);
-               return IMAP_ERROR;
+               return r;
        }
-       session->exists = exists;
-       session_set_access_time(SESSION(session));
 
-       return IMAP_SUCCESS;
-}
+       session->folder_content_changed = FALSE;
 
-#if USE_OPENSSL
-static gint imap_cmd_starttls(IMAPSession *session)
+       if ((exists && exists != session->exists)
+        || (recent && recent != session->recent)
+        || (expunge && expunge != session->expunge)
+        || (unseen && unseen != session->unseen)) {
+               session->folder_content_changed = TRUE;
+       }
+       if (uidnext != 0 && uidnext != session->uid_next) {
+               session->uid_next = uidnext;
+               session->folder_content_changed = TRUE;
+       }
+       if (uidval != 0 && uidval != session->uid_validity) {
+               session->uid_validity = uidval;
+               session->folder_content_changed = TRUE;
+       }
+
+       session->exists = exists;
+       session->recent = recent;
+       session->expunge = expunge;
+       session->unseen = unseen;
+
+       session_set_access_time(SESSION(session));
+
+       return MAILIMAP_NO_ERROR;
+}
+
+#if (defined(USE_OPENSSL) || defined (USE_GNUTLS))
+static gint imap_cmd_starttls(IMAPSession *session)
 {
        int r;
        
        r = imap_threaded_starttls(session->folder, 
                SESSION(session)->server, SESSION(session)->port);
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                debug_print("starttls err %d\n", r);
-               return IMAP_ERROR;
+               return r;
        }
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 #endif
 
 static gint imap_cmd_select(IMAPSession *session, const gchar *folder,
                            gint *exists, gint *recent, gint *unseen,
-                           guint32 *uid_validity, gboolean block)
+                           guint32 *uid_validity, gint *can_create_flags,
+                           gboolean block)
 {
        int r;
 
        r = imap_threaded_select(session->folder, folder,
-                                exists, recent, unseen, uid_validity);
+                                exists, recent, unseen, uid_validity, can_create_flags);
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                debug_print("select err %d\n", r);
-               return IMAP_ERROR;
+               return r;
+       }
+       return MAILIMAP_NO_ERROR;
+}
+
+static gint imap_cmd_close(IMAPSession *session)
+{
+       int r;
+
+       r = imap_threaded_close(session->folder);
+       if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
+               debug_print("close err %d\n", r);
+               return r;
        }
-       return IMAP_SUCCESS;
+       g_free(session->mbox);
+       session->mbox = NULL;
+       session->exists = 0;
+       session->recent = 0;
+       session->expunge = 0;
+       return MAILIMAP_NO_ERROR;
 }
 
 static gint imap_cmd_examine(IMAPSession *session, const gchar *folder,
@@ -2647,11 +3511,12 @@ static gint imap_cmd_examine(IMAPSession *session, const gchar *folder,
        r = imap_threaded_examine(session->folder, folder,
                                  exists, recent, unseen, uid_validity);
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                debug_print("examine err %d\n", r);
                
-               return IMAP_ERROR;
+               return r;
        }
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 static gint imap_cmd_create(IMAPSession *session, const gchar *folder)
@@ -2660,11 +3525,12 @@ static gint imap_cmd_create(IMAPSession *session, const gchar *folder)
 
        r = imap_threaded_create(session->folder, folder);
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                
-               return IMAP_ERROR;
+               return r;
        }
 
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 static gint imap_cmd_rename(IMAPSession *session, const gchar *old_folder,
@@ -2675,11 +3541,11 @@ static gint imap_cmd_rename(IMAPSession *session, const gchar *old_folder,
        r = imap_threaded_rename(session->folder, old_folder,
                                 new_folder);
        if (r != MAILIMAP_NO_ERROR) {
-               
-               return IMAP_ERROR;
+               imap_handle_error(SESSION(session), r);
+               return r;
        }
 
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 static gint imap_cmd_delete(IMAPSession *session, const gchar *folder)
@@ -2689,11 +3555,11 @@ static gint imap_cmd_delete(IMAPSession *session, const gchar *folder)
 
        r = imap_threaded_delete(session->folder, folder);
        if (r != MAILIMAP_NO_ERROR) {
-               
-               return IMAP_ERROR;
+               imap_handle_error(SESSION(session), r);
+               return r;
        }
 
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 typedef struct _fetch_data {
@@ -2722,10 +3588,11 @@ static void *imap_cmd_fetch_thread(void *data)
                                                uid, 0, filename);
        }
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                debug_print("fetch err %d\n", r);
-               return GINT_TO_POINTER(IMAP_ERROR);
+               return GINT_TO_POINTER(MAILIMAP_ERROR_BAD_STATE);
        }
-       return GINT_TO_POINTER(IMAP_SUCCESS);
+       return GINT_TO_POINTER(MAILIMAP_NO_ERROR);
 }
 
 static gint imap_cmd_fetch(IMAPSession *session, guint32 uid,
@@ -2742,8 +3609,8 @@ static gint imap_cmd_fetch(IMAPSession *session, guint32 uid,
        data->body = body;
 
        if (prefs_common.work_offline && 
-           !inc_offline_should_override(
-               _("Sylpheed-Claws needs network access in order "
+           !inc_offline_should_override(FALSE,
+               _("Claws Mail needs network access in order "
                  "to access the IMAP server."))) {
                g_free(data);
                return -1;
@@ -2763,18 +3630,23 @@ static gint imap_cmd_append(IMAPSession *session, const gchar *destfolder,
        struct mailimap_flag_list * flag_list;
        int r;
        
-       g_return_val_if_fail(file != NULL, IMAP_ERROR);
+       g_return_val_if_fail(file != NULL, MAILIMAP_ERROR_BAD_STATE);
 
-       flag_list = imap_flag_to_lep(flags);
+       flag_list = imap_flag_to_lep(flags, NULL);
+       lock_session(session);
        r = imap_threaded_append(session->folder, destfolder,
                         file, flag_list, (int *)new_uid);
        mailimap_flag_list_free(flag_list);
 
        if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
                debug_print("append err %d\n", r);
-               return IMAP_ERROR;
+               return r;
        }
-       return IMAP_SUCCESS;
+
+       unlock_session(session);
+
+       return MAILIMAP_NO_ERROR;
 }
 
 static gint imap_cmd_copy(IMAPSession *session, struct mailimap_set * set,
@@ -2783,28 +3655,28 @@ static gint imap_cmd_copy(IMAPSession *session, struct mailimap_set * set,
 {
        int r;
        
-       g_return_val_if_fail(session != NULL, IMAP_ERROR);
-       g_return_val_if_fail(set != NULL, IMAP_ERROR);
-       g_return_val_if_fail(destfolder != NULL, IMAP_ERROR);
+       g_return_val_if_fail(session != NULL, MAILIMAP_ERROR_BAD_STATE);
+       g_return_val_if_fail(set != NULL, MAILIMAP_ERROR_BAD_STATE);
+       g_return_val_if_fail(destfolder != NULL, MAILIMAP_ERROR_BAD_STATE);
 
        r = imap_threaded_copy(session->folder, set, destfolder, source, dest);
        if (r != MAILIMAP_NO_ERROR) {
-               
-               return IMAP_ERROR;
+               imap_handle_error(SESSION(session), r);
+               return r;
        }
 
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 static gint imap_cmd_store(IMAPSession *session, struct mailimap_set * set,
-                          IMAPFlags flags, int do_add)
+                          IMAPFlags flags, GSList *tags, int do_add)
 {
        int r;
-       struct mailimap_flag_list * flag_list;
+       struct mailimap_flag_list * flag_list = NULL;
        struct mailimap_store_att_flags * store_att_flags;
        
-       flag_list = imap_flag_to_lep(flags);
-       
+       flag_list = imap_flag_to_lep(flags, tags);
+
        if (do_add)
                store_att_flags =
                        mailimap_store_att_flags_new_add_flags_silent(flag_list);
@@ -2815,11 +3687,11 @@ static gint imap_cmd_store(IMAPSession *session, struct mailimap_set * set,
        r = imap_threaded_store(session->folder, set, store_att_flags);
        mailimap_store_att_flags_free(store_att_flags);
        if (r != MAILIMAP_NO_ERROR) {
-               
-               return IMAP_ERROR;
+               imap_handle_error(SESSION(session), r);
+               return r;
        }
        
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 static gint imap_cmd_expunge(IMAPSession *session)
@@ -2827,19 +3699,19 @@ static gint imap_cmd_expunge(IMAPSession *session)
        int r;
        
        if (prefs_common.work_offline && 
-           !inc_offline_should_override(
-               _("Sylpheed-Claws needs network access in order "
+           !inc_offline_should_override(FALSE,
+               _("Claws Mail needs network access in order "
                  "to access the IMAP server."))) {
                return -1;
        }
 
        r = imap_threaded_expunge(session->folder);
        if (r != MAILIMAP_NO_ERROR) {
-               
-               return IMAP_ERROR;
+               imap_handle_error(SESSION(session), r);
+               return r;
        }
 
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 static void imap_path_separator_subst(gchar *str, gchar separator)
@@ -2859,194 +3731,17 @@ static void imap_path_separator_subst(gchar *str, gchar separator)
        }
 }
 
-static gchar *imap_modified_utf7_to_utf8(const gchar *mutf7_str)
-{
-       static iconv_t cd = (iconv_t)-1;
-       static gboolean iconv_ok = TRUE;
-       GString *norm_utf7;
-       gchar *norm_utf7_p;
-       size_t norm_utf7_len;
-       const gchar *p;
-       gchar *to_str, *to_p;
-       size_t to_len;
-       gboolean in_escape = FALSE;
-
-       if (!iconv_ok) return g_strdup(mutf7_str);
-
-       if (cd == (iconv_t)-1) {
-               cd = iconv_open(CS_INTERNAL, CS_UTF_7);
-               if (cd == (iconv_t)-1) {
-                       g_warning("iconv cannot convert UTF-7 to %s\n",
-                                 CS_INTERNAL);
-                       iconv_ok = FALSE;
-                       return g_strdup(mutf7_str);
-               }
-       }
-
-       /* modified UTF-7 to normal UTF-7 conversion */
-       norm_utf7 = g_string_new(NULL);
-
-       for (p = mutf7_str; *p != '\0'; p++) {
-               /* replace: '&'  -> '+',
-                           "&-" -> '&',
-                           escaped ','  -> '/' */
-               if (!in_escape && *p == '&') {
-                       if (*(p + 1) != '-') {
-                               g_string_append_c(norm_utf7, '+');
-                               in_escape = TRUE;
-                       } else {
-                               g_string_append_c(norm_utf7, '&');
-                               p++;
-                       }
-               } else if (in_escape && *p == ',') {
-                       g_string_append_c(norm_utf7, '/');
-               } else if (in_escape && *p == '-') {
-                       g_string_append_c(norm_utf7, '-');
-                       in_escape = FALSE;
-               } else {
-                       g_string_append_c(norm_utf7, *p);
-               }
-       }
-
-       norm_utf7_p = norm_utf7->str;
-       norm_utf7_len = norm_utf7->len;
-       to_len = strlen(mutf7_str) * 5;
-       to_p = to_str = g_malloc(to_len + 1);
-
-       if (iconv(cd, (ICONV_CONST gchar **)&norm_utf7_p, &norm_utf7_len,
-                 &to_p, &to_len) == -1) {
-               g_warning(_("iconv cannot convert UTF-7 to %s\n"),
-                         conv_get_locale_charset_str());
-               g_string_free(norm_utf7, TRUE);
-               g_free(to_str);
-               return g_strdup(mutf7_str);
-       }
-
-       /* second iconv() call for flushing */
-       iconv(cd, NULL, NULL, &to_p, &to_len);
-       g_string_free(norm_utf7, TRUE);
-       *to_p = '\0';
-
-       return to_str;
-}
-
-static gchar *imap_utf8_to_modified_utf7(const gchar *from)
-{
-       static iconv_t cd = (iconv_t)-1;
-       static gboolean iconv_ok = TRUE;
-       gchar *norm_utf7, *norm_utf7_p;
-       size_t from_len, norm_utf7_len;
-       GString *to_str;
-       gchar *from_tmp, *to, *p;
-       gboolean in_escape = FALSE;
-
-       if (!iconv_ok) return g_strdup(from);
-
-       if (cd == (iconv_t)-1) {
-               cd = iconv_open(CS_UTF_7, CS_INTERNAL);
-               if (cd == (iconv_t)-1) {
-                       g_warning(_("iconv cannot convert %s to UTF-7\n"),
-                                 CS_INTERNAL);
-                       iconv_ok = FALSE;
-                       return g_strdup(from);
-               }
-       }
-
-       /* UTF-8 to normal UTF-7 conversion */
-       Xstrdup_a(from_tmp, from, return g_strdup(from));
-       from_len = strlen(from);
-       norm_utf7_len = from_len * 5;
-       Xalloca(norm_utf7, norm_utf7_len + 1, return g_strdup(from));
-       norm_utf7_p = norm_utf7;
-
-#define IS_PRINT(ch) (isprint(ch) && IS_ASCII(ch))
-
-       while (from_len > 0) {
-               if (*from_tmp == '+') {
-                       *norm_utf7_p++ = '+';
-                       *norm_utf7_p++ = '-';
-                       norm_utf7_len -= 2;
-                       from_tmp++;
-                       from_len--;
-               } else if (IS_PRINT(*(guchar *)from_tmp)) {
-                       /* printable ascii char */
-                       *norm_utf7_p = *from_tmp;
-                       norm_utf7_p++;
-                       norm_utf7_len--;
-                       from_tmp++;
-                       from_len--;
-               } else {
-                       size_t conv_len = 0;
-
-                       /* unprintable char: convert to UTF-7 */
-                       p = from_tmp;
-                       while (!IS_PRINT(*(guchar *)p) && conv_len < from_len) {
-                               conv_len += g_utf8_skip[*(guchar *)p];
-                               p += g_utf8_skip[*(guchar *)p];
-                       }
-
-                       from_len -= conv_len;
-                       if (iconv(cd, (ICONV_CONST gchar **)&from_tmp,
-                                 &conv_len,
-                                 &norm_utf7_p, &norm_utf7_len) == -1) {
-                               g_warning(_("iconv cannot convert UTF-8 to UTF-7\n"));
-                               return g_strdup(from);
-                       }
-
-                       /* second iconv() call for flushing */
-                       iconv(cd, NULL, NULL, &norm_utf7_p, &norm_utf7_len);
-               }
-       }
-
-#undef IS_PRINT
-
-       *norm_utf7_p = '\0';
-       to_str = g_string_new(NULL);
-       for (p = norm_utf7; p < norm_utf7_p; p++) {
-               /* replace: '&' -> "&-",
-                           '+' -> '&',
-                           "+-" -> '+',
-                           BASE64 '/' -> ',' */
-               if (!in_escape && *p == '&') {
-                       g_string_append(to_str, "&-");
-               } else if (!in_escape && *p == '+') {
-                       if (*(p + 1) == '-') {
-                               g_string_append_c(to_str, '+');
-                               p++;
-                       } else {
-                               g_string_append_c(to_str, '&');
-                               in_escape = TRUE;
-                       }
-               } else if (in_escape && *p == '/') {
-                       g_string_append_c(to_str, ',');
-               } else if (in_escape && *p == '-') {
-                       g_string_append_c(to_str, '-');
-                       in_escape = FALSE;
-               } else {
-                       g_string_append_c(to_str, *p);
-               }
-       }
-
-       if (in_escape) {
-               in_escape = FALSE;
-               g_string_append_c(to_str, '-');
-       }
-
-       to = to_str->str;
-       g_string_free(to_str, FALSE);
-
-       return to;
-}
-
 static gboolean imap_rename_folder_func(GNode *node, gpointer data)
 {
        FolderItem *item = node->data;
        gchar **paths = data;
        const gchar *oldpath = paths[0];
        const gchar *newpath = paths[1];
+       gchar *real_oldpath, *real_newpath;
        gchar *base;
        gchar *new_itempath;
        gint oldpathlen;
+       IMAPSession *session = imap_session_get(item->folder);
 
        oldpathlen = strlen(oldpath);
        if (strncmp(oldpath, item->path, oldpathlen) != 0) {
@@ -3061,116 +3756,89 @@ static gboolean imap_rename_folder_func(GNode *node, gpointer data)
        else
                new_itempath = g_strconcat(newpath, G_DIR_SEPARATOR_S, base,
                                           NULL);
+
+       real_oldpath = imap_get_real_path(session, IMAP_FOLDER(item->folder), item->path);
        g_free(item->path);
        item->path = new_itempath;
+       
+       real_newpath = imap_get_real_path(session, IMAP_FOLDER(item->folder), item->path);
+       
+       imap_threaded_subscribe(item->folder, real_oldpath, FALSE);
+       imap_threaded_subscribe(item->folder, real_newpath, TRUE);
 
+       g_free(real_oldpath);
+       g_free(real_newpath);
        return FALSE;
 }
 
-typedef struct _get_list_uid_data {
-       Folder *folder;
-       IMAPSession *session;
-       IMAPFolderItem *item;
-       GSList **msgnum_list;
-       gboolean done;
-} get_list_uid_data;
-
-static void *get_list_of_uids_thread(void *data)
+static gint get_list_of_uids(IMAPSession *session, Folder *folder, IMAPFolderItem *item, GSList **msgnum_list)
 {
-       get_list_uid_data *stuff = (get_list_uid_data *)data;
-       Folder *folder = stuff->folder;
-       IMAPFolderItem *item = stuff->item;
-       GSList **msgnum_list = stuff->msgnum_list;
-       gint ok, nummsgs = 0, lastuid_old;
-       IMAPSession *session;
        GSList *uidlist, *elem;
+       int r = -1;
        clist * lep_uidlist;
-       int r;
+       gint ok, nummsgs = 0, lastuid_old;
 
-       session = stuff->session;
        if (session == NULL) {
-               stuff->done = TRUE;
-               return GINT_TO_POINTER(-1);
+               return -1;
        }
-       /* no session locking here, it's already locked by caller */
-       ok = imap_select(session, IMAP_FOLDER(folder), item->item.path,
-                        NULL, NULL, NULL, NULL, TRUE);
-       if (ok != IMAP_SUCCESS) {
-               stuff->done = TRUE;
-               return GINT_TO_POINTER(-1);
+
+       ok = imap_select(session, IMAP_FOLDER(folder), FOLDER_ITEM(item),
+                        NULL, NULL, NULL, NULL, NULL, TRUE);
+       if (ok != MAILIMAP_NO_ERROR) {
+               return -1;
        }
 
+       g_slist_free(item->uid_list);
+       item->uid_list = NULL;
+
        uidlist = NULL;
        
-       r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_SIMPLE, NULL,
+       if (folder->account && folder->account->low_bandwidth) {
+               r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_SIMPLE, NULL,
                                 &lep_uidlist);
+       }
        
        if (r == MAILIMAP_NO_ERROR) {
-               GSList * fetchuid_list;
-               
-               fetchuid_list =
+               GSList * fetchuid_list =
                        imap_uid_list_from_lep(lep_uidlist);
                mailimap_search_result_free(lep_uidlist);
                
                uidlist = g_slist_concat(fetchuid_list, uidlist);
-       }
-       else {
-               GSList * fetchuid_list;
+       } else {
                carray * lep_uidtab;
-               
-               r = imap_threaded_fetch_uid(folder, item->lastuid + 1,
-                                           &lep_uidtab);
+               if (r != -1) /* inited */
+                       imap_handle_error(SESSION(session), r);
+               r = imap_threaded_fetch_uid(folder, 1,
+                                   &lep_uidtab);
                if (r == MAILIMAP_NO_ERROR) {
-                       fetchuid_list =
+                       GSList * fetchuid_list =
                                imap_uid_list_from_lep_tab(lep_uidtab);
                        imap_fetch_uid_list_free(lep_uidtab);
                        uidlist = g_slist_concat(fetchuid_list, uidlist);
                }
        }
        
+       if (r != MAILIMAP_NO_ERROR) {
+               imap_handle_error(SESSION(session), r);
+               return -1;
+       }
+
        lastuid_old = item->lastuid;
-       *msgnum_list = g_slist_copy(item->uid_list);
-       nummsgs = g_slist_length(*msgnum_list);
-       debug_print("Got %d uids from cache\n", g_slist_length(item->uid_list));
 
        for (elem = uidlist; elem != NULL; elem = g_slist_next(elem)) {
                guint msgnum;
 
                msgnum = GPOINTER_TO_INT(elem->data);
-               if (msgnum > lastuid_old) {
-                       *msgnum_list = g_slist_prepend(*msgnum_list, GINT_TO_POINTER(msgnum));
-                       item->uid_list = g_slist_prepend(item->uid_list, GINT_TO_POINTER(msgnum));
-                       nummsgs++;
 
-                       if(msgnum > item->lastuid)
-                               item->lastuid = msgnum;
-               }
+               *msgnum_list = g_slist_prepend(*msgnum_list, GINT_TO_POINTER(msgnum));
+               item->uid_list = g_slist_prepend(item->uid_list, GINT_TO_POINTER(msgnum));
+               nummsgs++;
        }
        g_slist_free(uidlist);
-       stuff->done = TRUE;
-       return GINT_TO_POINTER(nummsgs);
-}
 
-static gint get_list_of_uids(IMAPSession *session, Folder *folder, IMAPFolderItem *item, GSList **msgnum_list)
-{
-       gint result;
-       get_list_uid_data *data = g_new0(get_list_uid_data, 1);
-       data->done = FALSE;
-       data->folder = folder;
-       data->item = item;
-       data->msgnum_list = msgnum_list;
-       data->session = session;
-       if (prefs_common.work_offline && 
-           !inc_offline_should_override(
-               _("Sylpheed-Claws needs network access in order "
-                 "to access the IMAP server."))) {
-               g_free(data);
-               return -1;
-       }
+       unlock_session(session); /* locked from imap_get_num_list */
 
-       result = GPOINTER_TO_INT(get_list_of_uids_thread(data));
-       g_free(data);
-       return result;
+       return nummsgs;
 
 }
 
@@ -3178,11 +3846,11 @@ gint imap_get_num_list(Folder *folder, FolderItem *_item, GSList **msgnum_list,
 {
        IMAPFolderItem *item = (IMAPFolderItem *)_item;
        IMAPSession *session;
-       gint ok, nummsgs = 0, exists;
-       guint32 uid_next = 0, uid_val = 0;
+       gint nummsgs;
        GSList *uidlist = NULL;
        gchar *dir;
        gboolean selected_folder;
+       gint known_list_len = 0;
        debug_print("get_num_list\n");
        
        g_return_val_if_fail(folder != NULL, -1);
@@ -3191,10 +3859,31 @@ gint imap_get_num_list(Folder *folder, FolderItem *_item, GSList **msgnum_list,
        g_return_val_if_fail(FOLDER_CLASS(folder) == &imap_class, -1);
        g_return_val_if_fail(folder->account != NULL, -1);
 
+       known_list_len = g_slist_length(item->uid_list);
+       if (!item->should_update) {
+               debug_print("get_num_list: nothing to update\n");
+               *old_uids_valid = TRUE;
+               if (known_list_len == item->item.total_msgs
+                && known_list_len > 0) {
+                       *msgnum_list = g_slist_copy(item->uid_list);
+                       return known_list_len;
+               } else {
+                       debug_print("don't know the list length...\n");
+               }
+       }
+
+       if (prefs_common.work_offline && 
+           !inc_offline_should_override(FALSE,
+               _("Claws Mail needs network access in order "
+                 "to access the IMAP server."))) {
+               return -1;
+       }
+       
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        g_return_val_if_fail(session != NULL, -1);
-       lock_session();
 
+       lock_session(session); /* unlocked by get_list_of_uids */
        if (FOLDER_ITEM(item)->path) 
                statusbar_print_all(_("Scanning folder %s%c%s ..."),
                                      FOLDER_ITEM(item)->folder->name, 
@@ -3206,114 +3895,31 @@ gint imap_get_num_list(Folder *folder, FolderItem *_item, GSList **msgnum_list,
 
        selected_folder = (session->mbox != NULL) &&
                          (!strcmp(session->mbox, item->item.path));
-       if (selected_folder && time(NULL) - item->use_cache < 2) {
-               ok = imap_cmd_noop(session);
-               if (ok != IMAP_SUCCESS) {
-                       debug_print("disconnected!\n");
-                       session = imap_reconnect_if_possible(folder, session);
-                       if (session == NULL) {
-                               statusbar_pop_all();
-                               unlock_session();
-                               return -1;
-                       }
-               }
-               exists = session->exists;
+       
+       if (item->should_trash_cache) {
+               *old_uids_valid = FALSE;
+               debug_print("get_num_list: trashing num list\n");
+               debug_print("Freeing imap uid cache\n");
+               item->lastuid = 0;
+               g_slist_free(item->uid_list);
+               item->uid_list = NULL;
 
-               uid_next = item->c_uid_next;
-               uid_val = item->c_uid_validity;
-               *old_uids_valid = TRUE;
+               imap_delete_all_cached_messages((FolderItem *)item);
        } else {
-               if (item->use_cache && time(NULL) - item->use_cache < 2) {
-                       exists = item->c_messages;
-                       uid_next = item->c_uid_next;
-                       uid_val = item->c_uid_validity;
-                       ok = IMAP_SUCCESS;
-                       debug_print("using cache %d %d %d\n", exists, uid_next, uid_val);
-               } else {
-                       ok = imap_status(session, IMAP_FOLDER(folder), item->item.path, item,
-                                &exists, &uid_next, &uid_val, NULL, FALSE);
-               }
-               item->item.last_num = uid_next - 1;
-               
-               item->use_cache = (time_t)0;
-               if (ok != IMAP_SUCCESS) {
-                       statusbar_pop_all();
-                       unlock_session();
-                       return -1;
-               }
-               if(item->item.mtime == uid_val)
-                       *old_uids_valid = TRUE;
-               else {
-                       *old_uids_valid = FALSE;
-
-                       debug_print("Freeing imap uid cache (%d != %d)\n",
-                                       (int)item->item.mtime, uid_val);
-                       item->lastuid = 0;
-                       g_slist_free(item->uid_list);
-                       item->uid_list = NULL;
-               
-                       item->item.mtime = uid_val;
-
-                       imap_delete_all_cached_messages((FolderItem *)item);
-               }
-       }
-
-       /* If old uid_next matches new uid_next we can be sure no message
-          was added to the folder */
-       debug_print("uid_next is %d and item->uid_next %d \n", 
-               uid_next, item->uid_next);
-       if (uid_next == item->uid_next) {
-               nummsgs = g_slist_length(item->uid_list);
-
-               /* If number of messages is still the same we
-                   know our caches message numbers are still valid,
-                   otherwise if the number of messages has decrease
-                  we discard our cache to start a new scan to find
-                  out which numbers have been removed */
-               if (exists == nummsgs) {
-                       debug_print("exists == nummsgs\n");
-                       *msgnum_list = g_slist_copy(item->uid_list);
-                       statusbar_pop_all();
-                       unlock_session();
-                       return nummsgs;
-               } else if (exists < nummsgs) {
-                       debug_print("Freeing imap uid cache");
-                       item->lastuid = 0;
-                       g_slist_free(item->uid_list);
-                       item->uid_list = NULL;
-               }
-       }
-
-       if (exists == 0) {
-               *msgnum_list = NULL;
-               statusbar_pop_all();
-               unlock_session();
-               return 0;
+               debug_print("get_num_list: updating num list\n");
+               *old_uids_valid = TRUE;
        }
 
        nummsgs = get_list_of_uids(session, folder, item, &uidlist);
+       /* session could be broken now, in case of fatal error */
+
+       debug_print("get_num_list: got %d msgs\n", nummsgs);
 
        if (nummsgs < 0) {
                statusbar_pop_all();
-               unlock_session();
                return -1;
        }
 
-       if (nummsgs != exists) {
-               /* Cache contains more messages then folder, we have cached
-                   an old UID of a message that was removed and new messages
-                   have been added too, otherwise the uid_next check would
-                  not have failed */
-               debug_print("Freeing imap uid cache");
-               item->lastuid = 0;
-               g_slist_free(item->uid_list);
-               item->uid_list = NULL;
-
-               g_slist_free(*msgnum_list);
-
-               nummsgs = get_list_of_uids(session, folder, item, &uidlist);
-       }
-
        *msgnum_list = uidlist;
 
        dir = folder_item_get_path((FolderItem *)item);
@@ -3321,11 +3927,10 @@ gint imap_get_num_list(Folder *folder, FolderItem *_item, GSList **msgnum_list,
        remove_numbered_files_not_in_list(dir, *msgnum_list);
        g_free(dir);
        
-       item->uid_next = uid_next;
-       
        debug_print("get_num_list - ok - %i\n", nummsgs);
        statusbar_pop_all();
-       unlock_session();
+       item->should_trash_cache = FALSE;
+       item->should_update = FALSE;
        return nummsgs;
 }
 
@@ -3368,24 +3973,31 @@ GSList *imap_get_msginfos(Folder *folder, FolderItem *item,
        g_return_val_if_fail(item != NULL, NULL);
        g_return_val_if_fail(msgnum_list != NULL, NULL);
 
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        g_return_val_if_fail(session != NULL, NULL);
-       lock_session();
+
+       lock_session(session); /* unlocked later in the function */
+
        debug_print("IMAP getting msginfos\n");
-       ok = imap_select(session, IMAP_FOLDER(folder), item->path,
-                        NULL, NULL, NULL, NULL, FALSE);
-       if (ok != IMAP_SUCCESS) {
-               unlock_session();
+       ok = imap_select(session, IMAP_FOLDER(folder), item,
+                        NULL, NULL, NULL, NULL, NULL, FALSE);
+       if (ok != MAILIMAP_NO_ERROR) {
                return NULL;
        }
        if (!(folder_has_parent_of_type(item, F_DRAFT) || 
              folder_has_parent_of_type(item, F_QUEUE))) {
                ret = g_slist_concat(ret,
                        imap_get_uncached_messages(session, item,
-                                                  msgnum_list));
+                                                  msgnum_list, &ok));
+               if (ok != MAILIMAP_NO_ERROR)
+                       return NULL;
+               unlock_session(session);
        } else {
                MsgNumberList *sorted_list, *elem, *llast = NULL;
                gint startnum, lastnum;
+       
+               unlock_session(session);
 
                sorted_list = g_slist_sort(g_slist_copy(msgnum_list), g_int_compare);
 
@@ -3402,7 +4014,6 @@ GSList *imap_get_msginfos(Folder *folder, FolderItem *item,
                                int i;
                                for (i = startnum; i <= lastnum; ++i) {
                                        gchar *file;
-                       
                                        file = imap_fetch_msg(folder, item, i);
                                        if (file != NULL) {
                                                MsgInfo *msginfo = imap_parse_msg(file, item);
@@ -3417,7 +4028,6 @@ GSList *imap_get_msginfos(Folder *folder, FolderItem *item,
                                                }
                                                g_free(file);
                                        }
-                                       session_set_access_time(SESSION(session));
                                }
 
                                if (elem == NULL)
@@ -3430,7 +4040,6 @@ GSList *imap_get_msginfos(Folder *folder, FolderItem *item,
 
                g_slist_free(sorted_list);
        }
-       unlock_session();
        return ret;
 }
 
@@ -3468,49 +4077,79 @@ gboolean imap_scan_required(Folder *folder, FolderItem *_item)
        if (item->item.path == NULL)
                return FALSE;
 
+       if (item->should_update) {
+               debug_print("scan already required\n");
+               return TRUE;
+       }
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
+       
        g_return_val_if_fail(session != NULL, FALSE);
-       lock_session();
+       lock_session(session); /* unlocked later in the function */
+
        selected_folder = (session->mbox != NULL) &&
                          (!strcmp(session->mbox, item->item.path));
-       if (selected_folder && time(NULL) - item->use_cache < 2) {
-               ok = imap_cmd_noop(session);
-               if (ok != IMAP_SUCCESS) {
-                       debug_print("disconnected!\n");
-                       session = imap_reconnect_if_possible(folder, session);
-                       if (session == NULL)
-                               return FALSE;
-                       lock_session();
-               }
+       if (selected_folder) {
+               if (!session->folder_content_changed) {
+                       ok = imap_cmd_noop(session);
+                       if (ok != MAILIMAP_NO_ERROR) {
+                               debug_print("disconnected!\n");
+                               if (!is_fatal(ok))
+                                       session = imap_reconnect_if_possible(folder, session);
+                               else
+                                       session = imap_session_get(folder);
+                               if (session == NULL)
+                                       return FALSE;
+                       }
 
-               if (session->folder_content_changed
-               ||  session->exists != item->item.total_msgs) {
-                       unlock_session();
+                       if (session->folder_content_changed) {
+                               debug_print("CHANGED (self-noop)! scan_required\n");
+                               item->should_update = TRUE;
+                               if (session->uid_validity && session->uid_validity != item->item.mtime) {
+                                       item->item.mtime = session->uid_validity;
+                                       item->should_trash_cache = TRUE;
+                               }
+                               unlock_session(session);
+                               return TRUE;
+                       }
+               } else {
+                       debug_print("CHANGED (previous noop)! scan_required\n");
+                       item->should_update = TRUE;
+                       if (session->uid_validity && session->uid_validity != item->item.mtime) {
+                               item->item.mtime = session->uid_validity;
+                               item->should_trash_cache = TRUE;
+                       }
+                       unlock_session(session);
                        return TRUE;
                }
        } else {
                ok = imap_status(session, IMAP_FOLDER(folder), item->item.path, IMAP_FOLDER_ITEM(item),
                                 &exists, &uid_next, &uid_val, &unseen, FALSE);
-               if (ok != IMAP_SUCCESS) {
-                       unlock_session();
+               if (ok != MAILIMAP_NO_ERROR) {
                        return FALSE;
                }
-
-               item->use_cache = time(NULL);
-               item->c_messages = exists;
-               item->c_uid_next = uid_next;
-               item->c_uid_validity = uid_val;
-               item->c_unseen = unseen;
-               item->item.last_num = uid_next - 1;
-               debug_print("uidnext %d, item->uid_next %d, exists %d, item->item.total_msgs %d\n", 
-                       uid_next, item->uid_next, exists, item->item.total_msgs);
-               if ((uid_next != item->uid_next) || (exists != item->item.total_msgs)
-                   || unseen != item->item.unread_msgs || uid_val != item->item.mtime) {
-                       unlock_session();
+               
+               debug_print("exists %d, item->item.total_msgs %d\n", 
+                       exists, item->item.total_msgs);
+               if (exists != item->item.total_msgs
+                   || unseen != item->item.unread_msgs 
+                   || uid_next != item->uid_next
+                   || uid_val != item->item.mtime) {
+                       debug_print("CHANGED (status)! scan_required\n");
+                       item->last_change = time(NULL);
+                       item->should_update = TRUE;
+                       item->uid_next = uid_next;
+                       if (uid_val != item->item.mtime) {
+                               item->item.mtime = uid_val;
+                               item->should_trash_cache = TRUE;
+                       }
+                       unlock_session(session);
                        return TRUE;
                }
        }
-       unlock_session();
+       unlock_session(session);
+
+       item->should_update = FALSE;
        return FALSE;
 }
 
@@ -3518,7 +4157,7 @@ void imap_change_flags(Folder *folder, FolderItem *item, MsgInfo *msginfo, MsgPe
 {
        IMAPSession *session;
        IMAPFlags flags_set = 0, flags_unset = 0;
-       gint ok = IMAP_SUCCESS;
+       gint ok = MAILIMAP_NO_ERROR;
        MsgNumberList numlist;
        hashtable_data *ht_data = NULL;
 
@@ -3529,11 +4168,6 @@ void imap_change_flags(Folder *folder, FolderItem *item, MsgInfo *msginfo, MsgPe
        g_return_if_fail(msginfo != NULL);
        g_return_if_fail(msginfo->folder == item);
 
-       session = imap_session_get(folder);
-       if (!session) {
-               return;
-       }
-
        if (!MSG_IS_MARKED(msginfo->flags) &&  (newflags & MSG_MARKED))
                flags_set |= IMAP_FLAG_FLAGGED;
        if ( MSG_IS_MARKED(msginfo->flags) && !(newflags & MSG_MARKED))
@@ -3561,10 +4195,14 @@ void imap_change_flags(Folder *folder, FolderItem *item, MsgInfo *msginfo, MsgPe
                return;
        }
 
-       lock_session();
-       if ((ok = imap_select(session, IMAP_FOLDER(folder), msginfo->folder->path,
-           NULL, NULL, NULL, NULL, FALSE)) != IMAP_SUCCESS) {
-               unlock_session();
+       debug_print("getting session...\n");
+       session = imap_session_get(folder);
+       if (!session) {
+               return;
+       }
+
+       if ((ok = imap_select(session, IMAP_FOLDER(folder), msginfo->folder,
+           NULL, NULL, NULL, NULL, NULL, FALSE)) != MAILIMAP_NO_ERROR) {
                return;
        }
        numlist.next = NULL;
@@ -3582,48 +4220,41 @@ void imap_change_flags(Folder *folder, FolderItem *item, MsgInfo *msginfo, MsgPe
                                GINT_TO_POINTER(flags_set));
                        if (ht_data == NULL) {
                                ht_data = g_new0(hashtable_data, 1);
-                               ht_data->session = session;
                                ht_data->item = IMAP_FOLDER_ITEM(item);
                                g_hash_table_insert(IMAP_FOLDER_ITEM(item)->flags_set_table, 
                                        GINT_TO_POINTER(flags_set), ht_data);
                        }
-                       if (!g_slist_find(ht_data->msglist, GINT_TO_POINTER(msginfo->msgnum)))
-                               ht_data->msglist = g_slist_prepend(ht_data->msglist, GINT_TO_POINTER(msginfo->msgnum));
+                       ht_data->msglist = g_slist_prepend(ht_data->msglist, GINT_TO_POINTER(msginfo->msgnum));
                } 
                if (flags_unset) {
                        ht_data = g_hash_table_lookup(IMAP_FOLDER_ITEM(item)->flags_unset_table, 
                                GINT_TO_POINTER(flags_unset));
                        if (ht_data == NULL) {
                                ht_data = g_new0(hashtable_data, 1);
-                               ht_data->session = session;
                                ht_data->item = IMAP_FOLDER_ITEM(item);
                                g_hash_table_insert(IMAP_FOLDER_ITEM(item)->flags_unset_table, 
                                        GINT_TO_POINTER(flags_unset), ht_data);
                        }
-                       if (!g_slist_find(ht_data->msglist, GINT_TO_POINTER(msginfo->msgnum)))
-                               ht_data->msglist = g_slist_prepend(ht_data->msglist, 
+                       ht_data->msglist = g_slist_prepend(ht_data->msglist, 
                                        GINT_TO_POINTER(msginfo->msgnum));              
                }
        } else {
                debug_print("IMAP changing flags\n");
                if (flags_set) {
-                       ok = imap_set_message_flags(session, &numlist, flags_set, TRUE);
-                       if (ok != IMAP_SUCCESS) {
-                               unlock_session();
+                       ok = imap_set_message_flags(session, &numlist, flags_set, NULL, TRUE);
+                       if (ok != MAILIMAP_NO_ERROR) {
                                return;
                        }
                }
 
                if (flags_unset) {
-                       ok = imap_set_message_flags(session, &numlist, flags_unset, FALSE);
-                       if (ok != IMAP_SUCCESS) {
-                               unlock_session();
+                       ok = imap_set_message_flags(session, &numlist, flags_unset, NULL, FALSE);
+                       if (ok != MAILIMAP_NO_ERROR) {
                                return;
                        }
                }
        }
        msginfo->flags.perm_flags = newflags;
-       unlock_session();
        return;
 }
 
@@ -3638,24 +4269,22 @@ static gint imap_remove_msg(Folder *folder, FolderItem *item, gint uid)
        g_return_val_if_fail(FOLDER_CLASS(folder) == &imap_class, -1);
        g_return_val_if_fail(item != NULL, -1);
 
+       debug_print("getting session...\n");
        session = imap_session_get(folder);
        if (!session) return -1;
-       lock_session();
-       ok = imap_select(session, IMAP_FOLDER(folder), item->path,
-                        NULL, NULL, NULL, NULL, FALSE);
-       if (ok != IMAP_SUCCESS) {
-               unlock_session();
+
+       ok = imap_select(session, IMAP_FOLDER(folder), item,
+                        NULL, NULL, NULL, NULL, NULL, FALSE);
+       if (ok != MAILIMAP_NO_ERROR) {
                return ok;
        }
        numlist.next = NULL;
        numlist.data = GINT_TO_POINTER(uid);
        
        ok = imap_set_message_flags
-               (IMAP_SESSION(REMOTE_FOLDER(folder)->session),
-               &numlist, IMAP_FLAG_DELETED, TRUE);
-       if (ok != IMAP_SUCCESS) {
-               log_warning(_("can't set deleted flags: %d\n"), uid);
-               unlock_session();
+               (session, &numlist, IMAP_FLAG_DELETED, NULL, TRUE);
+       if (ok != MAILIMAP_NO_ERROR) {
+               log_warning(LOG_PROTOCOL, _("can't set deleted flags: %d\n"), uid);
                return ok;
        }
 
@@ -3668,9 +4297,8 @@ static gint imap_remove_msg(Folder *folder, FolderItem *item, gint uid)
                ok = imap_cmd_expunge(session);
                g_free(uidstr);
        }
-       if (ok != IMAP_SUCCESS) {
-               log_warning(_("can't expunge\n"));
-               unlock_session();
+       if (ok != MAILIMAP_NO_ERROR) {
+               log_warning(LOG_PROTOCOL, _("can't expunge\n"));
                return ok;
        }
 
@@ -3680,8 +4308,7 @@ static gint imap_remove_msg(Folder *folder, FolderItem *item, gint uid)
        if (is_dir_exist(dir))
                remove_numbered_files(dir, uid, uid);
        g_free(dir);
-       unlock_session();
-       return IMAP_SUCCESS;
+       return MAILIMAP_NO_ERROR;
 }
 
 static gint compare_msginfo(gconstpointer a, gconstpointer b)
@@ -3724,42 +4351,41 @@ static /*gint*/ void *imap_get_flags_thread(void *data)
 {
        get_flags_data *stuff = (get_flags_data *)data;
        Folder *folder = stuff->folder;
-       FolderItem *item = stuff->item;
+       FolderItem *fitem = (FolderItem *) stuff->item;
        MsgInfoList *msginfo_list = stuff->msginfo_list;
        GRelation *msgflags = stuff->msgflags;
-       gboolean full_search = stuff->full_search;
+       GSList *elem;
+       carray * lep_uidtab;
        IMAPSession *session;
+       gint ok;
+       int r = MAILIMAP_NO_ERROR;
+       GHashTable *flags_hash = NULL;
+       GHashTable *tags_hash = NULL;
+       gboolean full_search = stuff->full_search;
        GSList *sorted_list = NULL;
        GSList *unseen = NULL, *answered = NULL, *flagged = NULL, *deleted = NULL;
        GSList *p_unseen, *p_answered, *p_flagged, *p_deleted;
-       GSList *elem;
        GSList *seq_list, *cur;
        gboolean reverse_seen = FALSE;
-       GString *cmd_buf;
-       gint ok;
-       gint exists_cnt, unseen_cnt;
        gboolean selected_folder;
-       
-       if (folder == NULL || item == NULL) {
-               stuff->done = TRUE;
-               return GINT_TO_POINTER(-1);
-       }
+       gint exists_cnt, unseen_cnt;
+       gboolean got_alien_tags = FALSE;
 
        session = imap_session_get(folder);
+
        if (session == NULL) {
                stuff->done = TRUE;
                return GINT_TO_POINTER(-1);
        }
-       lock_session();
        selected_folder = (session->mbox != NULL) &&
-                         (!strcmp(session->mbox, item->path));
+                         (!strcmp(session->mbox, fitem->path));
 
+       lock_session(session);
        if (!selected_folder) {
-               ok = imap_select(session, IMAP_FOLDER(folder), item->path,
-                       &exists_cnt, NULL, &unseen_cnt, NULL, TRUE);
-               if (ok != IMAP_SUCCESS) {
+               ok = imap_select(session, IMAP_FOLDER(folder), fitem,
+                       &exists_cnt, NULL, &unseen_cnt, NULL, NULL, TRUE);
+               if (ok != MAILIMAP_NO_ERROR) {
                        stuff->done = TRUE;
-                       unlock_session();
                        return GINT_TO_POINTER(-1);
                }
 
@@ -3767,69 +4393,48 @@ static /*gint*/ void *imap_get_flags_thread(void *data)
                        reverse_seen = TRUE;
        } 
        else {
-               if (item->unread_msgs > item->total_msgs / 2)
+               if (fitem->unread_msgs > fitem->total_msgs / 2)
                        reverse_seen = TRUE;
        }
 
-       cmd_buf = g_string_new(NULL);
-
        sorted_list = g_slist_sort(g_slist_copy(msginfo_list), compare_msginfo);
        if (!full_search) {
-               seq_list = imap_get_lep_set_from_msglist(msginfo_list);
+               seq_list = imap_get_lep_set_from_msglist(IMAP_FOLDER(folder), msginfo_list);
        } else {
                struct mailimap_set * set;
                set = mailimap_set_new_interval(1, 0);
                seq_list = g_slist_append(NULL, set);
        }
 
-       for (cur = seq_list; cur != NULL; cur = g_slist_next(cur)) {
-               struct mailimap_set * imapset;
-               clist * lep_uidlist;
-               int r;
-               
-               imapset = cur->data;
-               if (reverse_seen) {
-                       r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_SEEN,
-                                                full_search ? NULL:imapset, &lep_uidlist);
-               }
-               else {
-                       r = imap_threaded_search(folder,
-                                                IMAP_SEARCH_TYPE_UNSEEN,
-                                                full_search ? NULL:imapset, &lep_uidlist);
-               }
-               if (r == MAILIMAP_NO_ERROR) {
-                       GSList * uidlist;
-                       
-                       uidlist = imap_uid_list_from_lep(lep_uidlist);
-                       mailimap_search_result_free(lep_uidlist);
-                       
-                       unseen = g_slist_concat(unseen, uidlist);
-               }
-               
-               r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_FLAGGED,
-                                        full_search ? NULL:imapset, &lep_uidlist);
-               if (r == MAILIMAP_NO_ERROR) {
-                       GSList * uidlist;
-
-                       uidlist = imap_uid_list_from_lep(lep_uidlist);
-                       mailimap_search_result_free(lep_uidlist);
+       if (folder->account && folder->account->low_bandwidth) {
+               for (cur = seq_list; cur != NULL; cur = g_slist_next(cur)) {
+                       struct mailimap_set * imapset;
+                       clist * lep_uidlist;
+                       int r;
 
-                       flagged = g_slist_concat(flagged, uidlist);
-               }
-
-               if (item->opened || item->processing_pending || item == folder->inbox) {
-                       r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_ANSWERED,
-                                                full_search ? NULL:imapset, &lep_uidlist);
+                       imapset = cur->data;
+                       if (reverse_seen) {
+                               r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_SEEN,
+                                                        full_search ? NULL:imapset, &lep_uidlist);
+                       }
+                       else {
+                               r = imap_threaded_search(folder,
+                                                        IMAP_SEARCH_TYPE_UNSEEN,
+                                                        full_search ? NULL:imapset, &lep_uidlist);
+                       }
                        if (r == MAILIMAP_NO_ERROR) {
                                GSList * uidlist;
 
                                uidlist = imap_uid_list_from_lep(lep_uidlist);
                                mailimap_search_result_free(lep_uidlist);
 
-                               answered = g_slist_concat(answered, uidlist);
+                               unseen = g_slist_concat(unseen, uidlist);
+                       } else {
+                               imap_handle_error(SESSION(session), r);
+                               goto bail;
                        }
 
-                       r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_DELETED,
+                       r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_FLAGGED,
                                                 full_search ? NULL:imapset, &lep_uidlist);
                        if (r == MAILIMAP_NO_ERROR) {
                                GSList * uidlist;
@@ -3837,56 +4442,158 @@ static /*gint*/ void *imap_get_flags_thread(void *data)
                                uidlist = imap_uid_list_from_lep(lep_uidlist);
                                mailimap_search_result_free(lep_uidlist);
 
-                               deleted = g_slist_concat(deleted, uidlist);
+                               flagged = g_slist_concat(flagged, uidlist);
+                       } else {
+                               imap_handle_error(SESSION(session), r);
+                               goto bail;
+                       }
+
+                       if (fitem->opened || fitem->processing_pending || fitem == folder->inbox) {
+                               r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_ANSWERED,
+                                                        full_search ? NULL:imapset, &lep_uidlist);
+                               if (r == MAILIMAP_NO_ERROR) {
+                                       GSList * uidlist;
+
+                                       uidlist = imap_uid_list_from_lep(lep_uidlist);
+                                       mailimap_search_result_free(lep_uidlist);
+
+                                       answered = g_slist_concat(answered, uidlist);
+                               } else {
+                                       imap_handle_error(SESSION(session), r);
+                                       goto bail;
+                               }
+
+                               r = imap_threaded_search(folder, IMAP_SEARCH_TYPE_DELETED,
+                                                        full_search ? NULL:imapset, &lep_uidlist);
+                               if (r == MAILIMAP_NO_ERROR) {
+                                       GSList * uidlist;
+
+                                       uidlist = imap_uid_list_from_lep(lep_uidlist);
+                                       mailimap_search_result_free(lep_uidlist);
+
+                                       deleted = g_slist_concat(deleted, uidlist);
+                               } else {
+                                       imap_handle_error(SESSION(session), r);
+                                       goto bail;
+                               }
                        }
                }
-       }
+               p_unseen = unseen;
+               p_answered = answered;
+               p_flagged = flagged;
+               p_deleted = deleted;
 
-       p_unseen = unseen;
-       p_answered = answered;
-       p_flagged = flagged;
-       p_deleted = deleted;
+       } else {
+               r = imap_threaded_fetch_uid_flags(folder, 1, &lep_uidtab);
+               if (r == MAILIMAP_NO_ERROR) {
+                       flags_hash = g_hash_table_new_full(g_direct_hash, g_direct_equal, NULL, NULL);
+                       tags_hash = g_hash_table_new_full(g_direct_hash, g_direct_equal, NULL, NULL);
+                       imap_flags_hash_from_lep_uid_flags_tab(lep_uidtab, flags_hash, tags_hash);
+                       imap_fetch_uid_flags_list_free(lep_uidtab);
+               } else {
+                       imap_handle_error(SESSION(session), r);
+                       goto bail;
+               }
+       }
 
+bail:
+       if (r == MAILIMAP_NO_ERROR)
+               unlock_session(session);
+       
        for (elem = sorted_list; elem != NULL; elem = g_slist_next(elem)) {
                MsgInfo *msginfo;
-               MsgPermFlags flags;
+               MsgPermFlags flags, oldflags;
                gboolean wasnew;
-               
+
                msginfo = (MsgInfo *) elem->data;
                flags = msginfo->flags.perm_flags;
                wasnew = (flags & MSG_NEW);
-               if (item->opened || item->processing_pending || item == folder->inbox) {
-                       flags &= ~((reverse_seen ? 0 : MSG_UNREAD | MSG_NEW) | MSG_REPLIED | MSG_MARKED);
-               } else {
-                       flags &= ~((reverse_seen ? 0 : MSG_UNREAD | MSG_NEW | MSG_MARKED));
-               }
-               if (reverse_seen)
-                       flags |= MSG_UNREAD | (wasnew ? MSG_NEW : 0);
-               if (gslist_find_next_num(&p_unseen, msginfo->msgnum) == msginfo->msgnum) {
-                       if (!reverse_seen) {
-                               flags |= MSG_UNREAD | (wasnew ? MSG_NEW : 0);
+               oldflags = flags & ~(MSG_NEW|MSG_UNREAD|MSG_REPLIED|MSG_MARKED|MSG_DELETED);
+
+               if (folder->account && folder->account->low_bandwidth) {
+                       if (fitem->opened || fitem->processing_pending || fitem == folder->inbox) {
+                               flags &= ~((reverse_seen ? 0 : MSG_UNREAD | MSG_NEW) | MSG_REPLIED | MSG_MARKED);
                        } else {
-                               flags &= ~(MSG_UNREAD | MSG_NEW);
+                               flags &= ~((reverse_seen ? 0 : MSG_UNREAD | MSG_NEW | MSG_MARKED));
+                       }
+                       if (reverse_seen)
+                               flags |= MSG_UNREAD | (wasnew ? MSG_NEW : 0);
+                       if (gslist_find_next_num(&p_unseen, msginfo->msgnum) == msginfo->msgnum) {
+                               if (!reverse_seen) {
+                                       flags |= MSG_UNREAD | (wasnew ? MSG_NEW : 0);
+                               } else {
+                                       flags &= ~(MSG_UNREAD | MSG_NEW);
+                               }
                        }
-               }
-               
-               if (gslist_find_next_num(&p_flagged, msginfo->msgnum) == msginfo->msgnum)
-                       flags |= MSG_MARKED;
-               else
-                       flags &= ~MSG_MARKED;
 
-               if (item->opened || item->processing_pending || item == folder->inbox) {
-                       if (gslist_find_next_num(&p_answered, msginfo->msgnum) == msginfo->msgnum)
-                               flags |= MSG_REPLIED;
-                       else
-                               flags &= ~MSG_REPLIED;
-                       if (gslist_find_next_num(&p_deleted, msginfo->msgnum) == msginfo->msgnum)
-                               flags |= MSG_DELETED;
+                       if (gslist_find_next_num(&p_flagged, msginfo->msgnum) == msginfo->msgnum)
+                               flags |= MSG_MARKED;
                        else
-                               flags &= ~MSG_DELETED;
+                               flags &= ~MSG_MARKED;
+
+                       if (fitem->opened || fitem->processing_pending || fitem == folder->inbox) {
+                               if (gslist_find_next_num(&p_answered, msginfo->msgnum) == msginfo->msgnum)
+                                       flags |= MSG_REPLIED;
+                               else
+                                       flags &= ~MSG_REPLIED;
+                               if (gslist_find_next_num(&p_deleted, msginfo->msgnum) == msginfo->msgnum)
+                                       flags |= MSG_DELETED;
+                               else
+                                       flags &= ~MSG_DELETED;
+                       }
+               } else {
+                       if (flags_hash != NULL) {
+
+                               flags = GPOINTER_TO_INT(g_hash_table_lookup(flags_hash, 
+                                               GINT_TO_POINTER(msginfo->msgnum)));
+                       }
+
+                       if ((flags & MSG_UNREAD) == 0)
+                               flags &= ~MSG_NEW;
+                       else if (wasnew)
+                               flags |= MSG_NEW;
+                       flags |= oldflags;
+                       
+                       if (tags_hash != NULL) {
+                               GSList *tags = g_hash_table_lookup(tags_hash, GINT_TO_POINTER(msginfo->msgnum));
+                               GSList *cur;
+
+                               if (tags != NULL) {
+                                       g_slist_free(msginfo->tags);
+                                       msginfo->tags = NULL;
+                               }
+                               for (cur = tags; cur; cur = cur->next) {
+                                       gchar *real_tag = imap_modified_utf7_to_utf8(cur->data, TRUE);
+                                       gint id = 0;
+                                       id = tags_get_id_for_str(real_tag);
+                                       if (id == -1) {
+                                               id = tags_add_tag(real_tag);
+                                               got_alien_tags = TRUE;
+                                       }
+                                       if (!g_slist_find(msginfo->tags, GINT_TO_POINTER(id))) {
+                                               msginfo->tags = g_slist_append(
+                                                               msginfo->tags,
+                                                               GINT_TO_POINTER(id));
+                                       }
+                                       g_free(real_tag);
+                               }
+                               slist_free_strings(tags);
+                               g_slist_free(tags);
+                       }
                }
+
                g_relation_insert(msgflags, msginfo, GINT_TO_POINTER(flags));
        }
+       
+       if (got_alien_tags) {
+               tags_write_tags();
+               main_window_reflect_tags_changes(mainwindow_get_mainwindow());
+       }
+
+       if (flags_hash)
+               g_hash_table_destroy(flags_hash);
+       if (tags_hash)
+               g_hash_table_destroy(tags_hash);
 
        imap_lep_set_free(seq_list);
        g_slist_free(flagged);
@@ -3894,10 +4601,8 @@ static /*gint*/ void *imap_get_flags_thread(void *data)
        g_slist_free(answered);
        g_slist_free(unseen);
        g_slist_free(sorted_list);
-       g_string_free(cmd_buf, TRUE);
 
        stuff->done = TRUE;
-       unlock_session();
        return GINT_TO_POINTER(0);
 }
 
@@ -3916,8 +4621,8 @@ static gint imap_get_flags(Folder *folder, FolderItem *item,
        GSList *tmp = NULL, *cur;
        
        if (prefs_common.work_offline && 
-           !inc_offline_should_override(
-               _("Sylpheed-Claws needs network access in order "
+           !inc_offline_should_override(FALSE,
+               _("Claws Mail needs network access in order "
                  "to access the IMAP server."))) {
                g_free(data);
                return -1;
@@ -3947,8 +4652,11 @@ static gboolean process_flags(gpointer key, gpointer value, gpointer user_data)
        hashtable_data *data = (hashtable_data *)value;
        IMAPFolderItem *_item = data->item;
        FolderItem *item = (FolderItem *)_item;
-       gint ok = IMAP_ERROR;
-       IMAPSession *session = imap_session_get(item->folder);
+       gint ok = MAILIMAP_ERROR_BAD_STATE;
+       IMAPSession *session = NULL;
+       
+       debug_print("getting session...\n");
+       session = imap_session_get(item->folder);
 
        data->msglist = g_slist_reverse(data->msglist);
        
@@ -3957,19 +4665,64 @@ static gboolean process_flags(gpointer key, gpointer value, gpointer user_data)
                flags_value,
                g_slist_length(data->msglist));
        
+       lock_session(session);
        if (session) {
-               lock_session();
-               ok = imap_select(session, IMAP_FOLDER(item->folder), item->path,
-                        NULL, NULL, NULL, NULL, FALSE);
+               ok = imap_select(session, IMAP_FOLDER(item->folder), item,
+                        NULL, NULL, NULL, NULL, NULL, FALSE);
+       }
+       if (ok == MAILIMAP_NO_ERROR) {
+               ok = imap_set_message_flags(session, data->msglist, flags_value, NULL, flags_set);
+       } else {
+               g_warning("can't select mailbox %s\n", item->path);
        }
-       if (ok == IMAP_SUCCESS) {
-               imap_set_message_flags(data->session, data->msglist, flags_value, flags_set);
+
+       if (!is_fatal(ok))
+               unlock_session(session);
+
+       g_slist_free(data->msglist);    
+       g_free(data);
+       return TRUE;
+}
+
+static gboolean process_tags(gpointer key, gpointer value, gpointer user_data)
+{
+       gboolean tags_set = GPOINTER_TO_INT(user_data);
+       TagsData *data = (TagsData *)value;
+       IMAPFolderItem *_item = data->item;
+       FolderItem *item = (FolderItem *)_item;
+       gchar *str = data->str;
+       gint ok = MAILIMAP_ERROR_BAD_STATE;
+       IMAPSession *session = NULL;
+       
+       debug_print("getting session...\n");
+       session = imap_session_get(item->folder);
+
+       data->msglist = g_slist_reverse(data->msglist);
+       
+       debug_print("IMAP %ssetting tags %s for %d messages\n",
+               tags_set?"":"un",
+               str,
+               g_slist_length(data->msglist));
+       
+       lock_session(session);
+       if (session) {
+               ok = imap_select(session, IMAP_FOLDER(item->folder), item,
+                        NULL, NULL, NULL, NULL, NULL, FALSE);
+       }
+       if (ok == MAILIMAP_NO_ERROR) {
+               GSList list;
+               list.data = str;
+               list.next = NULL;
+               ok = imap_set_message_flags(session, data->msglist, 0, &list, tags_set);
        } else {
                g_warning("can't select mailbox %s\n", item->path);
        }
-       if (session)
-               unlock_session();
+
+       if (!is_fatal(ok))
+               unlock_session(session);
+
        g_slist_free(data->msglist);    
+       g_free(data->str);
        g_free(data);
        return TRUE;
 }
@@ -3986,11 +4739,23 @@ static void process_hashtable(IMAPFolderItem *item)
                g_hash_table_destroy(item->flags_unset_table);
                item->flags_unset_table = NULL;
        }
+       if (item->tags_set_table) {
+               g_hash_table_foreach_remove(item->tags_set_table, process_tags, GINT_TO_POINTER(TRUE));
+               g_hash_table_destroy(item->tags_set_table);
+               item->tags_set_table = NULL;
+       }
+       if (item->tags_unset_table) {
+               g_hash_table_foreach_remove(item->tags_unset_table, process_tags, GINT_TO_POINTER(FALSE));
+               g_hash_table_destroy(item->tags_unset_table);
+               item->tags_unset_table = NULL;
+       }
+       
 }
 
 static void imap_set_batch (Folder *folder, FolderItem *_item, gboolean batch)
 {
        IMAPFolderItem *item = (IMAPFolderItem *)_item;
+       IMAPSession *session;
 
        g_return_if_fail(item != NULL);
        
@@ -4006,17 +4771,33 @@ static void imap_set_batch (Folder *folder, FolderItem *_item, gboolean batch)
                if (!item->flags_unset_table) {
                        item->flags_unset_table = g_hash_table_new(NULL, g_direct_equal);
                }
+               if (!item->tags_set_table) {
+                       item->tags_set_table = g_hash_table_new(NULL, g_direct_equal);
+               }
+               if (!item->tags_unset_table) {
+                       item->tags_unset_table = g_hash_table_new(NULL, g_direct_equal);
+               }
+               session = imap_session_get(folder);
+               if (session) {
+                       imap_refresh_sensitivity(session);
+                       session->sens_update_block = TRUE;
+               }
        } else {
                debug_print("IMAP switching away from batch mode\n");
                /* process stuff */
                process_hashtable(item);
                item->batching = FALSE;
+               session = imap_session_get(folder);
+               if (session) {
+                       session->sens_update_block = FALSE;
+                       imap_refresh_sensitivity(session);
+               }
        }
 }
 
 
 
-/* data types conversion libetpan <-> sylpheed */
+/* data types conversion libetpan <-> claws */
 
 
 
@@ -4101,21 +4882,18 @@ static GSList * imap_list_from_lep(IMAPFolder * folder,
                        free(dup_name);
                        continue;
                }
-               
-               if (!all && strcmp(dup_name, real_path) == 0) {
+               if (!all && path_cmp(name, real_path) == 0) {
                        g_free(base);
                        free(dup_name);
                        continue;
                }
 
                if (!all && dup_name[strlen(dup_name)-1] == '/') {
-                       g_free(base);
-                       free(dup_name);
-                       continue;
+                       dup_name[strlen(dup_name)-1] = '\0';
                }
                
-               loc_name = imap_modified_utf7_to_utf8(base);
-               loc_path = imap_modified_utf7_to_utf8(dup_name);
+               loc_name = imap_modified_utf7_to_utf8(base, FALSE);
+               loc_path = imap_modified_utf7_to_utf8(dup_name, FALSE);
                
                new_item = folder_item_new(FOLDER(folder), loc_name, loc_path);
                if ((flags & ETPAN_IMAP_MB_NOINFERIORS) != 0)
@@ -4141,19 +4919,17 @@ static GSList * imap_list_from_lep(IMAPFolder * folder,
        return item_list;
 }
 
-static GSList * imap_get_lep_set_from_numlist(MsgNumberList *numlist)
+static GSList * imap_get_lep_set_from_numlist(IMAPFolder *folder, MsgNumberList *numlist)
 {
        GSList *sorted_list, *cur;
        guint first, last, next;
        GSList *ret_list = NULL, *llast = NULL;
-       unsigned int count;
        struct mailimap_set * current_set;
        unsigned int item_count;
        
        if (numlist == NULL)
                return NULL;
        
-       count = 0;
        current_set = mailimap_set_new_empty();
        
        sorted_list = g_slist_copy(numlist);
@@ -4174,16 +4950,15 @@ static GSList * imap_get_lep_set_from_numlist(MsgNumberList *numlist)
                else
                        next = 0;
 
-               if (last + 1 != next || next == 0) {
+               if (last + 1 != next || next == 0 || item_count >= folder->max_set_size) {
 
                        struct mailimap_set_item * item;
                        item = mailimap_set_item_new(first, last);
                        mailimap_set_add(current_set, item);
-                       count ++;
                        
                        first = next;
-                       
-                       if (count >= IMAP_SET_MAX_COUNT) {
+
+                       if (item_count >= folder->max_set_size) {
                                if (ret_list == NULL)
                                        llast = ret_list = g_slist_append(ret_list,
                                                          current_set);
@@ -4191,11 +4966,11 @@ static GSList * imap_get_lep_set_from_numlist(MsgNumberList *numlist)
                                        llast = g_slist_append(llast, current_set);
                                        llast = llast->next;
                                }
+
                                current_set = mailimap_set_new_empty();
-                               count = 0;
                                item_count = 0;
                        }
-               }
+               } 
        }
        
        if (clist_count(current_set->set_list) > 0) {
@@ -4208,7 +4983,7 @@ static GSList * imap_get_lep_set_from_numlist(MsgNumberList *numlist)
        return ret_list;
 }
 
-static GSList * imap_get_lep_set_from_msglist(MsgInfoList *msglist)
+static GSList * imap_get_lep_set_from_msglist(IMAPFolder *folder, MsgInfoList *msglist)
 {
        MsgNumberList *numlist = NULL;
        MsgInfoList *cur;
@@ -4220,7 +4995,7 @@ static GSList * imap_get_lep_set_from_msglist(MsgInfoList *msglist)
                numlist = g_slist_prepend(numlist, GINT_TO_POINTER(msginfo->msgnum));
        }
        numlist = g_slist_reverse(numlist);
-       seq_list = imap_get_lep_set_from_numlist(numlist);
+       seq_list = imap_get_lep_set_from_numlist(folder, numlist);
        g_slist_free(numlist);
 
        return seq_list;
@@ -4262,6 +5037,29 @@ static GSList * imap_uid_list_from_lep_tab(carray * list)
        return result;
 }
 
+static void imap_flags_hash_from_lep_uid_flags_tab(carray * list,
+                                                  GHashTable * hash,
+                                                  GHashTable * tags_hash)
+{
+       unsigned int i;
+       GSList * result;
+       
+       result = NULL;
+       
+       for(i = 0 ; i < carray_count(list) ; i += 3) {
+               uint32_t * puid;
+               int * pflags;
+               GSList *tags;
+               
+               puid = carray_get(list, i);
+               pflags = carray_get(list, i + 1);
+               tags = carray_get(list, i + 2);
+               
+               g_hash_table_insert(hash, GINT_TO_POINTER(*puid), GINT_TO_POINTER(* pflags));
+               g_hash_table_insert(tags_hash, GINT_TO_POINTER(*puid), tags);
+       }
+}
+
 static MsgInfo *imap_envelope_from_lep(struct imap_fetch_env_info * info,
                                       FolderItem *item)
 {
@@ -4306,10 +5104,11 @@ static void imap_lep_set_free(GSList *seq_list)
        g_slist_free(seq_list);
 }
 
-static struct mailimap_flag_list * imap_flag_to_lep(IMAPFlags flags)
+static struct mailimap_flag_list * imap_flag_to_lep(IMAPFlags flags, GSList *tags)
 {
        struct mailimap_flag_list * flag_list;
-       
+       GSList *cur = tags;
+
        flag_list = mailimap_flag_list_new_empty();
        
        if (IMAP_IS_SEEN(flags))
@@ -4328,6 +5127,15 @@ static struct mailimap_flag_list * imap_flag_to_lep(IMAPFlags flags)
                mailimap_flag_list_add(flag_list,
                                       mailimap_flag_new_draft());
        
+       for (; cur; cur = cur->next) {
+               gchar *enc_str = 
+                       imap_utf8_to_modified_utf7(cur->data, TRUE);
+               g_strstrip(enc_str);
+       
+               mailimap_flag_list_add(flag_list,
+                       mailimap_flag_new_flag_keyword(enc_str));
+       }
+
        return flag_list;
 }
 
@@ -4365,6 +5173,63 @@ void imap_folder_unref(Folder *folder)
                ((IMAPFolder *)folder)->refcnt--;
 }
 
+void imap_cancel_all(void)
+{
+       GList *folderlist;
+       GList *cur;
+       
+       folderlist = folder_get_list();
+       for (cur = folderlist; cur != NULL; cur = g_list_next(cur)) {
+               Folder *folder = (Folder *) cur->data;
+
+               if (folder->klass == &imap_class) {
+                       if (imap_is_busy(folder)) {
+                               IMAPSession *imap_session;
+                               RemoteFolder *rfolder;
+                               
+                               g_printerr("cancelled\n");
+                               imap_threaded_cancel(folder);
+                               rfolder = (RemoteFolder *) folder;
+                               imap_session = (IMAPSession *) rfolder->session;
+                               if (imap_session)
+                                       imap_session->cancelled = 1;
+                       }
+               }
+       }
+}
+
+gboolean imap_cancel_all_enabled(void)
+{
+       GList *folderlist;
+       GList *cur;
+       
+       folderlist = folder_get_list();
+       for (cur = folderlist; cur != NULL; cur = g_list_next(cur)) {
+               Folder *folder = (Folder *) cur->data;
+
+               if (folder->klass == &imap_class) {
+                       if (imap_is_busy(folder)) {
+                               return TRUE;
+                       }
+               }
+       }
+       
+       return FALSE;
+}
+
+static gboolean imap_is_busy(Folder *folder)
+{
+       IMAPSession *imap_session;
+       RemoteFolder *rfolder;
+       
+       rfolder = (RemoteFolder *) folder;
+       imap_session = (IMAPSession *) rfolder->session;
+       if (imap_session == NULL)
+               return FALSE;
+       
+       return imap_session->busy;
+}
+
 #else /* HAVE_LIBETPAN */
 
 static FolderClass imap_class;
@@ -4381,12 +5246,12 @@ static Folder   *imap_folder_new        (const gchar    *name,
                alertpanel_error(
                        _("You have one or more IMAP accounts "
                          "defined. However this version of "
-                         "Sylpheed-Claws has been built without "
+                         "Claws Mail has been built without "
                          "IMAP support; your IMAP account(s) are "
                          "disabled.\n\n"
                          "You probably need to "
                          "install libetpan and recompile "
-                         "Sylpheed-Claws."));
+                         "Claws Mail."));
        }
        return NULL;
 }
@@ -4438,12 +5303,43 @@ void imap_disconnect_all(void)
 {
 }
 
+gint imap_subscribe(Folder *folder, FolderItem *item, gchar *rpath, gboolean sub)
+{
+       return -1;
+}
+
+GList * imap_scan_subtree(Folder *folder, FolderItem *item, gboolean unsubs_only, gboolean recursive)
+{
+       return NULL;
+}
+
+void imap_cache_msg(FolderItem *item, gint msgnum)
+{
+}
+
+void imap_cancel_all(void)
+{
+}
+
+gboolean imap_cancel_all_enabled(void)
+{
+       return FALSE;
+}
+
 #endif
 
-void imap_synchronise(FolderItem *item) 
+#ifdef HAVE_LIBETPAN
+static void imap_synchronise(FolderItem *item, gint days) 
 {
-       imap_gtk_synchronise(item);
+       if (IMAP_FOLDER_ITEM(item)->last_sync == IMAP_FOLDER_ITEM(item)->last_change) {
+               debug_print("%s already synced\n", item->path?item->path:item->name);
+               return;
+       }
+       debug_print("syncing %s\n", item->path?item->path:item->name);
+       imap_gtk_synchronise(item, days);
+       IMAP_FOLDER_ITEM(item)->last_sync = IMAP_FOLDER_ITEM(item)->last_change;
 }
+#endif
 
 static void imap_item_set_xml(Folder *folder, FolderItem *item, XMLTag *tag)
 {
@@ -4459,7 +5355,13 @@ static void imap_item_set_xml(Folder *folder, FolderItem *item, XMLTag *tag)
                if (!attr || !attr->name || !attr->value) continue;
                if (!strcmp(attr->name, "uidnext"))
                        IMAP_FOLDER_ITEM(item)->uid_next = atoi(attr->value);
+               if (!strcmp(attr->name, "last_sync"))
+                       IMAP_FOLDER_ITEM(item)->last_sync = atoi(attr->value);
+               if (!strcmp(attr->name, "last_change"))
+                       IMAP_FOLDER_ITEM(item)->last_change = atoi(attr->value);
        }
+       if (IMAP_FOLDER_ITEM(item)->last_change == 0)
+               IMAP_FOLDER_ITEM(item)->last_change = time(NULL);
 #endif
 }
 
@@ -4472,7 +5374,234 @@ static XMLTag *imap_item_get_xml(Folder *folder, FolderItem *item)
 #ifdef HAVE_LIBETPAN
        xml_tag_add_attr(tag, xml_attr_new_int("uidnext", 
                        IMAP_FOLDER_ITEM(item)->uid_next));
+       xml_tag_add_attr(tag, xml_attr_new_int("last_sync", 
+                       IMAP_FOLDER_ITEM(item)->last_sync));
+       xml_tag_add_attr(tag, xml_attr_new_int("last_change", 
+                       IMAP_FOLDER_ITEM(item)->last_change));
 
 #endif
        return tag;
 }
+
+/* ===================================================================
+ * UTF-7 conversion routines as in RFC 2192
+ * =================================================================== 
+ * These two functions from: 
+ * libimap library.
+ * Copyright (C) 2003-2004 Pawel Salek. */
+
+/* UTF7 modified base64 alphabet */
+static char base64chars[] =
+  "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+,";
+#define UNDEFINED 64
+
+/* UTF16 definitions */
+#define UTF16MASK       0x03FFUL
+#define UTF16SHIFT      10
+#define UTF16BASE       0x10000UL
+#define UTF16HIGHSTART  0xD800UL
+#define UTF16HIGHEND    0xDBFFUL
+#define UTF16LOSTART    0xDC00UL
+#define UTF16LOEND      0xDFFFUL
+
+
+/* Convert an IMAP mailbox to a UTF-8 string.
+ *  dst needs to have roughly 4 times the storage space of src
+ *    Hex encoding can triple the size of the input
+ *    UTF-7 can be slightly denser than UTF-8
+ *     (worst case: 8 octets UTF-7 becomes 9 octets UTF-8)
+ */
+char* imap_modified_utf7_to_utf8(const char *mbox, gboolean change_spaces)
+{
+  unsigned c, i, bitcount;
+  unsigned long ucs4, utf16, bitbuf;
+  unsigned char base64[256];
+  const char *src;
+  char *dst, *res  = malloc(2*strlen(mbox)+1);
+
+  bitbuf = 0;
+  dst = res;
+  src = mbox;
+  if(!dst) return NULL;
+  /* initialize modified base64 decoding table */
+  memset(base64, UNDEFINED, sizeof (base64));
+  for (i = 0; i < sizeof (base64chars); ++i) {
+    base64[(unsigned)base64chars[i]] = i;
+  }
+
+  /* loop until end of string */
+  while (*src != '\0') {
+    c = *src++;
+    /* deal with literal characters and &- */
+    if (c != '&' || *src == '-') {
+      /* encode literally */
+      if (change_spaces && c == '_')
+       *dst++ = ' ';
+      else
+        *dst++ = c;
+      /* skip over the '-' if this is an &- sequence */
+      if (c == '&') ++src;
+    } else {
+      /* convert modified UTF-7 -> UTF-16 -> UCS-4 -> UTF-8 -> HEX */
+      bitbuf = 0;
+      bitcount = 0;
+      ucs4 = 0;
+      while ((c = base64[(unsigned char) *src]) != UNDEFINED) {
+        ++src;
+        bitbuf = (bitbuf << 6) | c;
+        bitcount += 6;
+        /* enough bits for a UTF-16 character? */
+        if (bitcount >= 16) {
+          bitcount -= 16;
+          utf16 = (bitcount ? bitbuf >> bitcount
+                   : bitbuf) & 0xffff;
+          /* convert UTF16 to UCS4 */
+          if
+            (utf16 >= UTF16HIGHSTART && utf16 <= UTF16HIGHEND) {
+            ucs4 = (utf16 - UTF16HIGHSTART) << UTF16SHIFT;
+            continue;
+          } else if
+            (utf16 >= UTF16LOSTART && utf16 <= UTF16LOEND) {
+            ucs4 += utf16 - UTF16LOSTART + UTF16BASE;
+          } else {
+            ucs4 = utf16;
+          }
+
+          /* convert UTF-16 range of UCS4 to UTF-8 */
+          if (ucs4 <= 0x7fUL) {
+            dst[0] = ucs4;
+            dst += 1;
+          } else if (ucs4 <= 0x7ffUL) {
+            dst[0] = 0xc0 | (ucs4 >> 6);
+            dst[1] = 0x80 | (ucs4 & 0x3f);
+            dst += 2;
+          } else if (ucs4 <= 0xffffUL) {
+            dst[0] = 0xe0 | (ucs4 >> 12);
+            dst[1] = 0x80 | ((ucs4 >> 6) & 0x3f);
+            dst[2] = 0x80 | (ucs4 & 0x3f);
+            dst += 3;
+          } else {
+            dst[0] = 0xf0 | (ucs4 >> 18);
+            dst[1] = 0x80 | ((ucs4 >> 12) & 0x3f);
+            dst[2] = 0x80 | ((ucs4 >> 6) & 0x3f);
+            dst[3] = 0x80 | (ucs4 & 0x3f);
+            dst += 4;
+          }
+        }
+      }
+      /* skip over trailing '-' in modified UTF-7 encoding */
+      if (*src == '-') ++src;
+    }
+  }
+  /* terminate destination string */
+  *dst = '\0';
+  return res;
+}
+
+/* Convert hex coded UTF-8 string to modified UTF-7 IMAP mailbox
+ *  dst should be about twice the length of src to deal with non-hex
+ *  coded URLs
+ */
+char* imap_utf8_to_modified_utf7(const char *src, gboolean change_spaces)
+{
+  unsigned int utf8pos, utf8total, c, utf7mode, bitstogo, utf16flag;
+  unsigned long ucs4 = 0, bitbuf = 0;
+
+  /* initialize hex lookup table */
+  char *dst, *res = malloc(2*strlen(src)+1);
+  dst = res;
+  if(!dst) return NULL;
+
+  utf7mode = 0;
+  utf8total = 0;
+  bitstogo = 0;
+  utf8pos = 0;
+  while ((c = (unsigned char)*src) != '\0') {
+    ++src;
+    /* normal character? */
+    if (c >= ' ' && c <= '~' && (c != '_' || !change_spaces)) {
+      /* switch out of UTF-7 mode */
+      if (utf7mode) {
+        if (bitstogo) {
+          *dst++ = base64chars[(bitbuf << (6 - bitstogo)) & 0x3F];
+        }
+        *dst++ = '-';
+        utf7mode = 0;
+        utf8pos  = 0;
+        bitstogo = 0;
+        utf8total= 0;
+      }
+      if (change_spaces && c == ' ')
+        *dst++ = '_';
+      else
+       *dst++ = c;
+      /* encode '&' as '&-' */
+      if (c == '&') {
+        *dst++ = '-';
+      }
+      continue;
+    }
+    /* switch to UTF-7 mode */
+    if (!utf7mode) {
+      *dst++ = '&';
+      utf7mode = 1;
+    }
+    /* Encode US-ASCII characters as themselves */
+    if (c < 0x80) {
+      ucs4 = c;
+      utf8total = 1;
+    } else if (utf8total) {
+      /* save UTF8 bits into UCS4 */
+      ucs4 = (ucs4 << 6) | (c & 0x3FUL);
+      if (++utf8pos < utf8total) {
+        continue;
+      }
+    } else {
+      utf8pos = 1;
+      if (c < 0xE0) {
+        utf8total = 2;
+        ucs4 = c & 0x1F;
+      } else if (c < 0xF0) {
+        utf8total = 3;
+        ucs4 = c & 0x0F;
+      } else {
+        /* NOTE: can't convert UTF8 sequences longer than 4 */
+        utf8total = 4;
+        ucs4 = c & 0x03;
+      }
+      continue;
+    }
+    /* loop to split ucs4 into two utf16 chars if necessary */
+    utf8total = 0;
+    do {
+      if (ucs4 >= UTF16BASE) {
+        ucs4 -= UTF16BASE;
+        bitbuf = (bitbuf << 16) | ((ucs4 >> UTF16SHIFT)
+                                   + UTF16HIGHSTART);
+        ucs4 = (ucs4 & UTF16MASK) + UTF16LOSTART;
+        utf16flag = 1;
+      } else {
+        bitbuf = (bitbuf << 16) | ucs4;
+        utf16flag = 0;
+      }
+      bitstogo += 16;
+      /* spew out base64 */
+      while (bitstogo >= 6) {
+        bitstogo -= 6;
+        *dst++ = base64chars[(bitstogo ? (bitbuf >> bitstogo)
+                              : bitbuf)
+                             & 0x3F];
+      }
+    } while (utf16flag);
+  }
+  /* if in UTF-7 mode, finish in ASCII */
+  if (utf7mode) {
+    if (bitstogo) {
+      *dst++ = base64chars[(bitbuf << (6 - bitstogo)) & 0x3F];
+    }
+    *dst++ = '-';
+  }
+  /* tie off string */
+  *dst = '\0';
+  return res;
+}