]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
lib-storage: mail_search_arg_init() - Remove support for changing uidset
authorTimo Sirainen <timo.sirainen@open-xchange.com>
Fri, 13 Mar 2020 08:51:55 +0000 (10:51 +0200)
committertimo.sirainen <timo.sirainen@open-xchange.com>
Fri, 20 Mar 2020 08:03:57 +0000 (08:03 +0000)
This is done earlier now - there's no need to do it twice.

src/lib-storage/mail-search.c
src/lib-storage/mail-search.h
src/plugins/fts/fts-search-args.c

index 013a93cd74509a63ce1f9ef7314486c3721c31fc..f38926781c590c888407b27c5eebbbd8e097a86c 100644 (file)
@@ -84,21 +84,13 @@ mail_search_arg_change_uidset(struct mail_search_args *args,
 }
 
 void mail_search_arg_init(struct mail_search_args *args,
-                         struct mail_search_arg *arg,
-                         bool change_uidsets,
-                         const ARRAY_TYPE(seq_range) *search_saved_uidset)
+                         struct mail_search_arg *arg)
 {
        struct mail_search_args *thread_args;
        const char *keywords[2];
 
        for (; arg != NULL; arg = arg->next) {
                switch (arg->type) {
-               case SEARCH_UIDSET:
-                       if (change_uidsets) T_BEGIN {
-                               mailbox_uidset_change(arg, args->box,
-                                                     search_saved_uidset);
-                       } T_END;
-                       break;
                case SEARCH_MODSEQ:
                        if (arg->value.str == NULL)
                                break;
@@ -141,9 +133,7 @@ void mail_search_arg_init(struct mail_search_args *args,
                        /* fall through */
                case SEARCH_SUB:
                case SEARCH_OR:
-                       mail_search_arg_init(args, arg->value.subargs,
-                                            change_uidsets,
-                                            search_saved_uidset);
+                       mail_search_arg_init(args, arg->value.subargs);
                        break;
                default:
                        break;
@@ -171,8 +161,7 @@ void mail_search_args_init(struct mail_search_args *args,
        }
        if (!args->simplified)
                mail_search_args_simplify(args);
-       mail_search_arg_init(args, args->args, change_uidsets,
-                            search_saved_uidset);
+       mail_search_arg_init(args, args->args);
 }
 
 void mail_search_arg_deinit(struct mail_search_arg *arg)
index 69c5f8699878a68db38b7387eebbb60d218c270e..c1493aacbfb73e6c4511b42d4fe90b2df7da4e3a 100644 (file)
@@ -161,9 +161,7 @@ void mail_search_args_init(struct mail_search_args *args,
 /* Initialize arg and its children. args is used for getting mailbox and
    pool. */
 void mail_search_arg_init(struct mail_search_args *args,
-                         struct mail_search_arg *arg,
-                         bool change_uidsets,
-                         const ARRAY_TYPE(seq_range) *search_saved_uidset);
+                         struct mail_search_arg *arg);
 /* Free memory allocated by mail_search_args_init(). The args can initialized
    afterwards again if needed. The args can be reused for other queries after
    calling this. */
index 651f863133e79b9613a0e5a2b4c4fc949761d76c..299fbfbfe9cef3d244c0941f6b4a5bf8d93ab00f 100644 (file)
@@ -252,7 +252,7 @@ int fts_search_args_expand(struct fts_backend *backend,
 
        /* duplicated args aren't initialized */
        i_assert(args->init_refcount > 0);
-       mail_search_arg_init(args, args_dup, FALSE, NULL);
+       mail_search_arg_init(args, args_dup);
        mail_search_arg_deinit(orig_args);
        return 0;
 }