2010-11-04 [wwp] 3.7.6cvs62
authorTristan Chabredier <wwp@claws-mail.org>
Thu, 4 Nov 2010 05:49:54 +0000 (05:49 +0000)
committerTristan Chabredier <wwp@claws-mail.org>
Thu, 4 Nov 2010 05:49:54 +0000 (05:49 +0000)
* src/filtering.c
Fix bug 2293, "filtering: simplify expression", as simplifying
this expression doesn't make it less easy to understand and we
gain few cycles by making the logical expression more simple
by applying. Patch by Nicolas Kaiser.

ChangeLog
PATCHSETS
configure.ac
src/filtering.c

index 2e253a9..19f9ade 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2010-11-04 [wwp]       3.7.6cvs62
+
+       * src/filtering.c
+               Fix bug 2293, "filtering: simplify expression", as simplifying
+               this expression doesn't make it less easy to understand and we
+               gain few cycles by making the logical expression more simple
+               by applying. Patch by Nicolas Kaiser.
+
 2010-11-03 [colin]     3.7.6cvs61
 
        * src/addrindex.c
index 30c8ca9..177806f 100644 (file)
--- a/PATCHSETS
+++ b/PATCHSETS
 ( cvs diff -u -r 1.101.2.61 -r 1.101.2.62 src/news.c;  ) > 3.7.6cvs59.patchset
 ( cvs diff -u -r 1.12.2.61 -r 1.12.2.62 src/action.c;  cvs diff -u -r 1.5.2.27 -r 1.5.2.28 src/statusbar.c;  cvs diff -u -r 1.43.2.120 -r 1.43.2.121 src/toolbar.c;  ) > 3.7.6cvs60.patchset
 ( cvs diff -u -r 1.28.2.43 -r 1.28.2.44 src/addrindex.c;  ) > 3.7.6cvs61.patchset
+( cvs diff -u -r 1.60.2.55 -r 1.60.2.56 src/filtering.c;  ) > 3.7.6cvs62.patchset
index 51e85b7..daad8fc 100644 (file)
@@ -12,7 +12,7 @@ MINOR_VERSION=7
 MICRO_VERSION=6
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=61
+EXTRA_VERSION=62
 EXTRA_RELEASE=
 EXTRA_GTK2_VERSION=
 
index fe76665..790be57 100644 (file)
@@ -823,9 +823,10 @@ static gboolean filter_msginfo(GSList * filtering_list, MsgInfo * info, PrefsAcc
                }
        }
 
-       /* put in inbox if a final rule could not be applied, or
-        * the last rule was not a final one. */
-       if ((final && !apply_next) || !final) {
+    /* put in inbox if the last rule was not a final one, or
+     * a final rule could not be applied.
+     * Either of these cases is likely. */
+    if (!final || !apply_next) {
                return FALSE;
        }