Revert "Fix a memory leak in matcherrc rule parsing."
[claws.git] / src / matcher_parser_parse.y
index 559f680d6eba63ef1ae2a24d97c1e7cc8024a1b9..dd2aee702e0728372d8ffa31f73cfcf20225b860 100644 (file)
@@ -1,13 +1,11 @@
-
 %{
-
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (c) 2001-2002 by Hiroyuki Yamamoto & The Sylpheed Claws Team.
+ * Copyright (c) 2001-2014 by Hiroyuki Yamamoto & 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,
  * 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., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * 
  */
 
 #include "defs.h"
 
 #include <glib.h>
+#include <glib/gi18n.h>
 
-#include "intl.h"
 #include "utils.h"
 #include "filtering.h"
-#include "scoring.h"
 #include "matcher.h"
 #include "matcher_parser.h"
 #include "matcher_parser_lex.h"
-#include "procmsg.h"
-
-#define MAX_COLORLABELS (MSG_CLABEL_7 - MSG_CLABEL_NONE)
+#include "colorlabel.h"
+#include "folder_item_prefs.h"
 
 static gint error = 0;
 static gint bool_op = 0;
@@ -44,67 +40,89 @@ static MatcherProp *prop;
 
 static GSList *matchers_list = NULL;
 
+static gboolean enabled = TRUE;
+static gchar *name = NULL;
+static gint account_id = 0;
 static MatcherList *cond;
-static gint score = 0;
+static GSList *action_list = NULL;
 static FilteringAction *action = NULL;
+static gboolean matcher_is_fast = TRUE;
+static gboolean disable_warnings = FALSE;
 
 static FilteringProp *filtering;
-static ScoringProp *scoring = NULL;
 
-static GSList **prefs_scoring = NULL;
 static GSList **prefs_filtering = NULL;
+static int enable_compatibility = 0;
+
+enum {
+        MATCHER_PARSE_FILE,
+        MATCHER_PARSE_NO_EOL,
+       MATCHER_PARSE_ENABLED,
+       MATCHER_PARSE_NAME,
+       MATCHER_PARSE_ACCOUNT,
+        MATCHER_PARSE_CONDITION,
+        MATCHER_PARSE_FILTERING_ACTION,
+};
+
+static int matcher_parse_op = MATCHER_PARSE_FILE;
 
-static int matcher_parser_dialog = 0;
 
 /* ******************************************************************** */
+/* redeclarations to avoid warnings */
+void matcher_parserrestart(FILE *input_file);
+void matcher_parser_init(void);
+void matcher_parser_switch_to_buffer(void * new_buffer);
+void matcher_parser_delete_buffer(void * b);
+void matcher_parserpop_buffer_state(void);
+int matcher_parserlex(void);
+
+void matcher_parser_disable_warnings(const gboolean disable)
+{
+       disable_warnings = disable;
+}
 
 void matcher_parser_start_parsing(FILE *f)
 {
+       matcher_parserlineno = 1;
        matcher_parserrestart(f);
+       account_id = 0;
        matcher_parserparse();
 }
+
+void * matcher_parser_scan_string(const char * str);
  
 FilteringProp *matcher_parser_get_filtering(gchar *str)
 {
        void *bufstate;
+       void *tmp_str = NULL;
+       
+       /* little hack to allow passing rules with no names */
+       if (!strncmp(str, "rulename ", 9))
+               tmp_str = g_strdup(str);
+       else 
+               tmp_str = g_strconcat("rulename \"\" ", str, NULL);
 
        /* bad coding to enable the sub-grammar matching
           in yacc */
        matcher_parserlineno = 1;
-       matcher_parser_dialog = 1;
+       matcher_parse_op = MATCHER_PARSE_NO_EOL;
        matcher_parserrestart(NULL);
+       matcher_parserpop_buffer_state();
         matcher_parser_init();
-       bufstate = matcher_parser_scan_string(str);
+       bufstate = matcher_parser_scan_string((const char *) tmp_str);
         matcher_parser_switch_to_buffer(bufstate);
        if (matcher_parserparse() != 0)
                filtering = NULL;
-       matcher_parser_dialog = 0;
+       matcher_parse_op = MATCHER_PARSE_FILE;
        matcher_parser_delete_buffer(bufstate);
+       g_free(tmp_str);
        return filtering;
 }
 
-ScoringProp *matcher_parser_get_scoring(gchar *str)
-{
-       void *bufstate;
-
-       /* bad coding to enable the sub-grammar matching
-          in yacc */
-       matcher_parserlineno = 1;
-       matcher_parser_dialog = 1;
-       matcher_parserrestart(NULL);
-        matcher_parser_init();
-       bufstate = matcher_parser_scan_string(str);
-        matcher_parser_switch_to_buffer(bufstate);
-       if (matcher_parserparse() != 0)
-               scoring = NULL;
-       matcher_parser_dialog = 0;
-       matcher_parser_delete_buffer(bufstate);
-       return scoring;
-}
-
 static gboolean check_quote_symetry(gchar *str)
 {
-       gchar *walk;
+       const gchar *walk;
        int ret = 0;
        
        if (str == NULL)
@@ -121,7 +139,7 @@ static gboolean check_quote_symetry(gchar *str)
        return !(ret % 2);
 }
 
-MatcherList *matcher_parser_get_cond(gchar *str)
+MatcherList *matcher_parser_get_name(gchar *str)
 {
        void *bufstate;
 
@@ -133,23 +151,119 @@ MatcherList *matcher_parser_get_cond(gchar *str)
        /* bad coding to enable the sub-grammar matching
           in yacc */
        matcher_parserlineno = 1;
-       matcher_parser_dialog = 1;
+       matcher_parse_op = MATCHER_PARSE_NAME;
        matcher_parserrestart(NULL);
+       matcher_parserpop_buffer_state();
         matcher_parser_init();
        bufstate = matcher_parser_scan_string(str);
        matcher_parserparse();
-       matcher_parser_dialog = 0;
+       matcher_parse_op = MATCHER_PARSE_FILE;
        matcher_parser_delete_buffer(bufstate);
        return cond;
 }
 
+MatcherList *matcher_parser_get_enabled(gchar *str)
+{
+       void *bufstate;
+
+       if (!check_quote_symetry(str)) {
+               cond = NULL;
+               return cond;
+       }
+       
+       /* bad coding to enable the sub-grammar matching
+          in yacc */
+       matcher_parserlineno = 1;
+       matcher_parse_op = MATCHER_PARSE_ENABLED;
+       matcher_parserrestart(NULL);
+       matcher_parserpop_buffer_state();
+       matcher_parser_init();
+       bufstate = matcher_parser_scan_string(str);
+       matcher_parserparse();
+       matcher_parse_op = MATCHER_PARSE_FILE;
+       matcher_parser_delete_buffer(bufstate);
+       return cond;
+}
+
+MatcherList *matcher_parser_get_account(gchar *str)
+{
+       void *bufstate;
+
+       if (!check_quote_symetry(str)) {
+               cond = NULL;
+               return cond;
+       }
+       
+       /* bad coding to enable the sub-grammar matching
+          in yacc */
+       matcher_parserlineno = 1;
+       matcher_parse_op = MATCHER_PARSE_ACCOUNT;
+       matcher_parserrestart(NULL);
+       matcher_parserpop_buffer_state();
+       matcher_parser_init();
+       bufstate = matcher_parser_scan_string(str);
+       matcher_parserparse();
+       matcher_parse_op = MATCHER_PARSE_FILE;
+       matcher_parser_delete_buffer(bufstate);
+       return cond;
+}
+
+MatcherList *matcher_parser_get_cond(gchar *str, gboolean *is_fast)
+{
+       void *bufstate;
+
+       if (!check_quote_symetry(str)) {
+               cond = NULL;
+               return cond;
+       }
+       
+       matcher_is_fast = TRUE;
+       /* bad coding to enable the sub-grammar matching
+          in yacc */
+       matcher_parserlineno = 1;
+       matcher_parse_op = MATCHER_PARSE_CONDITION;
+       matcher_parserrestart(NULL);
+       matcher_parserpop_buffer_state();
+        matcher_parser_init();
+       bufstate = matcher_parser_scan_string(str);
+       matcher_parserparse();
+       matcher_parse_op = MATCHER_PARSE_FILE;
+       matcher_parser_delete_buffer(bufstate);
+       if (is_fast)
+               *is_fast = matcher_is_fast;
+       return cond;
+}
+
+GSList *matcher_parser_get_action_list(gchar *str)
+{
+       void *bufstate;
+
+       if (!check_quote_symetry(str)) {
+               action_list = NULL;
+               return action_list;
+       }
+       
+       /* bad coding to enable the sub-grammar matching
+          in yacc */
+       matcher_parserlineno = 1;
+       matcher_parse_op = MATCHER_PARSE_FILTERING_ACTION;
+       matcher_parserrestart(NULL);
+       matcher_parserpop_buffer_state();
+        matcher_parser_init();
+       bufstate = matcher_parser_scan_string(str);
+       matcher_parserparse();
+       matcher_parse_op = MATCHER_PARSE_FILE;
+       matcher_parser_delete_buffer(bufstate);
+       return action_list;
+}
+
 MatcherProp *matcher_parser_get_prop(gchar *str)
 {
        MatcherList *list;
        MatcherProp *prop;
 
        matcher_parserlineno = 1;
-       list = matcher_parser_get_cond(str);
+       list = matcher_parser_get_cond(str, NULL);
        if (list == NULL)
                return NULL;
 
@@ -181,8 +295,9 @@ void matcher_parsererror(char *str)
                matchers_list = NULL;
        }
        cond = NULL;
-       g_warning("scoring / filtering parsing: %i: %s\n",
-                 matcher_parserlineno, str);
+       if (!disable_warnings)
+               g_warning("filtering parsing: %i: %s",
+                       matcher_parserlineno, str);
        error = 1;
 }
 
@@ -196,7 +311,6 @@ int matcher_parserwrap(void)
        char *str;
        int value;
 }
-
 %token MATCHER_ALL MATCHER_UNREAD  MATCHER_NOT_UNREAD 
 %token MATCHER_NEW  MATCHER_NOT_NEW  MATCHER_MARKED
 %token MATCHER_NOT_MARKED  MATCHER_DELETED  MATCHER_NOT_DELETED
@@ -205,26 +319,45 @@ int matcher_parserwrap(void)
 %token MATCHER_FROM  MATCHER_NOT_FROM  MATCHER_TO  MATCHER_NOT_TO
 %token MATCHER_CC  MATCHER_NOT_CC  MATCHER_TO_OR_CC  MATCHER_NOT_TO_AND_NOT_CC
 %token MATCHER_AGE_GREATER  MATCHER_AGE_LOWER  MATCHER_NEWSGROUPS
+%token MATCHER_AGE_GREATER_HOURS  MATCHER_AGE_LOWER_HOURS
 %token MATCHER_NOT_NEWSGROUPS  MATCHER_INREPLYTO  MATCHER_NOT_INREPLYTO
+%token MATCHER_MESSAGEID MATCHER_NOT_MESSAGEID
 %token MATCHER_REFERENCES  MATCHER_NOT_REFERENCES  MATCHER_SCORE_GREATER
 %token MATCHER_SCORE_LOWER  MATCHER_HEADER  MATCHER_NOT_HEADER
 %token MATCHER_HEADERS_PART  MATCHER_NOT_HEADERS_PART  MATCHER_MESSAGE
+%token MATCHER_HEADERS_CONT  MATCHER_NOT_HEADERS_CONT
 %token MATCHER_NOT_MESSAGE  MATCHER_BODY_PART  MATCHER_NOT_BODY_PART
-%token MATCHER_EXECUTE  MATCHER_NOT_EXECUTE  MATCHER_MATCHCASE  MATCHER_MATCH
+%token MATCHER_TEST  MATCHER_NOT_TEST  MATCHER_MATCHCASE  MATCHER_MATCH
 %token MATCHER_REGEXPCASE  MATCHER_REGEXP  MATCHER_SCORE  MATCHER_MOVE
-%token MATCHER_COPY  MATCHER_DELETE  MATCHER_MARK  MATCHER_UNMARK MATCHER_LOCK MATCHER_UNLOCK
+%token MATCHER_FOUND_IN_ADDRESSBOOK MATCHER_NOT_FOUND_IN_ADDRESSBOOK MATCHER_IN
+%token MATCHER_COPY  MATCHER_DELETE  MATCHER_MARK  MATCHER_UNMARK
+%token MATCHER_LOCK MATCHER_UNLOCK
+%token MATCHER_EXECUTE
 %token MATCHER_MARK_AS_READ  MATCHER_MARK_AS_UNREAD  MATCHER_FORWARD
-%token MATCHER_FORWARD_AS_ATTACHMENT  MATCHER_EOL  MATCHER_STRING  
+%token MATCHER_MARK_AS_SPAM MATCHER_MARK_AS_HAM
+%token MATCHER_FORWARD_AS_ATTACHMENT  MATCHER_EOL
 %token MATCHER_OR MATCHER_AND  
 %token MATCHER_COLOR MATCHER_SCORE_EQUAL MATCHER_REDIRECT 
 %token MATCHER_SIZE_GREATER MATCHER_SIZE_SMALLER MATCHER_SIZE_EQUAL
 %token MATCHER_LOCKED MATCHER_NOT_LOCKED
+%token MATCHER_PARTIAL MATCHER_NOT_PARTIAL
 %token MATCHER_COLORLABEL MATCHER_NOT_COLORLABEL
 %token MATCHER_IGNORE_THREAD MATCHER_NOT_IGNORE_THREAD
-%token MATCHER_CHANGE_SCORE
+%token MATCHER_WATCH_THREAD MATCHER_NOT_WATCH_THREAD
+%token MATCHER_CHANGE_SCORE MATCHER_SET_SCORE
+%token MATCHER_ADD_TO_ADDRESSBOOK
+%token MATCHER_STOP MATCHER_HIDE MATCHER_IGNORE MATCHER_WATCH
+%token MATCHER_SPAM MATCHER_NOT_SPAM
+%token MATCHER_HAS_ATTACHMENT MATCHER_HAS_NO_ATTACHMENT
+%token MATCHER_SIGNED MATCHER_NOT_SIGNED
+%token MATCHER_TAG MATCHER_NOT_TAG MATCHER_SET_TAG MATCHER_UNSET_TAG
+%token MATCHER_TAGGED MATCHER_NOT_TAGGED MATCHER_CLEAR_TAGS
 
 %start file
 
+%token MATCHER_ENABLED MATCHER_DISABLED
+%token MATCHER_RULENAME
+%token MATCHER_ACCOUNT
 %token <str> MATCHER_STRING
 %token <str> MATCHER_SECTION
 %token <str> MATCHER_INTEGER
@@ -233,9 +366,8 @@ int matcher_parserwrap(void)
 
 file:
 {
-       if (!matcher_parser_dialog) {
-               prefs_scoring = &global_scoring;
-               prefs_filtering = &global_processing;
+       if (matcher_parse_op == MATCHER_PARSE_FILE) {
+               prefs_filtering = &pre_global_processing;
        }
 }
 file_line_list;
@@ -248,7 +380,9 @@ file_line_list
 
 file_line:
 section_notification
-| instruction
+| 
+{ action_list = NULL; }
+instruction
 | error MATCHER_EOL
 {
        yyerrok;
@@ -260,17 +394,26 @@ MATCHER_SECTION MATCHER_EOL
        gchar *folder = $1;
        FolderItem *item = NULL;
 
-       if (!matcher_parser_dialog) {
+       if (matcher_parse_op == MATCHER_PARSE_FILE) {
+                enable_compatibility = 0;
                if (!strcmp(folder, "global")) {
-                       prefs_scoring = &global_scoring;
-                       prefs_filtering = &global_processing;
-               } else {
+                        /* backward compatibility */
+                        enable_compatibility = 1;
+                }
+               else if (!strcmp(folder, "preglobal")) {
+                       prefs_filtering = &pre_global_processing;
+                }
+               else if (!strcmp(folder, "postglobal")) {
+                       prefs_filtering = &post_global_processing;
+                }
+               else if (!strcmp(folder, "filtering")) {
+                        prefs_filtering = &filtering_rules;
+               }
+                else {
                        item = folder_find_item_from_identifier(folder);
                        if (item != NULL) {
-                               prefs_scoring = &item->prefs->scoring;
                                prefs_filtering = &item->prefs->processing;
                        } else {
-                               prefs_scoring = NULL;
                                prefs_filtering = NULL;
                        }
                }
@@ -279,57 +422,135 @@ MATCHER_SECTION MATCHER_EOL
 ;
 
 instruction:
-condition end_instr_opt
-| MATCHER_EOL
-;
-
-end_instr_opt:
-filtering_or_scoring end_action
-|
+enabled name account condition filtering MATCHER_EOL
+| enabled name account condition filtering
+| enabled name condition filtering MATCHER_EOL
+| enabled name condition filtering
+| name condition filtering MATCHER_EOL
+| name condition filtering
+{
+       if (matcher_parse_op == MATCHER_PARSE_NO_EOL)
+               YYACCEPT;
+       else {
+               matcher_parsererror("parse error [no eol]");
+               YYERROR;
+       }
+}
+| enabled
 {
-       if (matcher_parser_dialog)
+       if (matcher_parse_op == MATCHER_PARSE_ENABLED)
                YYACCEPT;
        else {
-               matcher_parsererror("parse error");
+               matcher_parsererror("parse error [enabled]");
                YYERROR;
        }
 }
-;
-
-end_action:
-MATCHER_EOL
-|
+| account
+{
+       if (matcher_parse_op == MATCHER_PARSE_ACCOUNT)
+               YYACCEPT;
+       else {
+               matcher_parsererror("parse error [account]");
+               YYERROR;
+       }
+}
+| name
+{
+       if (matcher_parse_op == MATCHER_PARSE_NAME)
+               YYACCEPT;
+       else {
+               matcher_parsererror("parse error [name]");
+               YYERROR;
+       }
+}
+| condition
+{
+       if (matcher_parse_op == MATCHER_PARSE_CONDITION)
+               YYACCEPT;
+       else {
+               matcher_parsererror("parse error [condition]");
+               YYERROR;
+       }
+}
+| filtering_action_list
 {
-       if (matcher_parser_dialog)
+       if (matcher_parse_op == MATCHER_PARSE_FILTERING_ACTION)
                YYACCEPT;
        else {
-               matcher_parsererror("parse error");
+               matcher_parsererror("parse error [filtering action]");
                YYERROR;
        }
 }
+| MATCHER_EOL
 ;
 
-filtering_or_scoring:
-filtering_action
+enabled:
+MATCHER_ENABLED
+{
+       enabled = TRUE;
+}
+| MATCHER_DISABLED
+{
+       enabled = FALSE;
+}
+;
+
+name:
+MATCHER_RULENAME MATCHER_STRING
 {
-       filtering = filteringprop_new(cond, action);
+       name = g_strdup($2);
+}
+;
+
+account:
+MATCHER_ACCOUNT MATCHER_INTEGER
+{
+       account_id = strtol($2, NULL, 10);
+}
+;
+
+filtering:
+filtering_action_list
+{
+       filtering = filteringprop_new(enabled, name, account_id, cond, action_list);
+       enabled = TRUE;
+       account_id = 0;
+       g_free(name);
+       name = NULL;
+        if (enable_compatibility) {
+                prefs_filtering = &filtering_rules;
+                if (action_list != NULL) {
+                        FilteringAction * first_action;
+                        
+                        first_action = action_list->data;
+                        
+                        if (first_action->type == MATCHACTION_CHANGE_SCORE)
+                                prefs_filtering = &pre_global_processing;
+                }
+        }
+        
        cond = NULL;
-       action = NULL;
-       if (!matcher_parser_dialog && (prefs_filtering != NULL)) {
+       action_list = NULL;
+        
+       if ((matcher_parse_op == MATCHER_PARSE_FILE) &&
+            (prefs_filtering != NULL)) {
                *prefs_filtering = g_slist_append(*prefs_filtering,
                                                  filtering);
                filtering = NULL;
        }
 }
-| scoring_rule
+;
+
+filtering_action_list:
+filtering_action_b filtering_action_list
+| filtering_action_b
+;
+
+filtering_action_b:
+filtering_action
 {
-       scoring = scoringprop_new(cond, score);
-       cond = NULL;
-       score = 0;
-       if (!matcher_parser_dialog && (prefs_scoring != NULL)) {
-               *prefs_scoring = g_slist_append(*prefs_scoring, scoring);
-               scoring = NULL;
-       }
+        action_list = g_slist_append(action_list, action);
+        action = NULL;
 }
 ;
 
@@ -389,105 +610,161 @@ MATCHER_ALL
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_ALL;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_UNREAD
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_UNREAD;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NOT_UNREAD 
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NOT_UNREAD;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NEW
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NEW;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NOT_NEW
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NOT_NEW;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_MARKED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_MARKED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NOT_MARKED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NOT_MARKED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_DELETED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_DELETED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NOT_DELETED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NOT_DELETED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_REPLIED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_REPLIED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NOT_REPLIED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NOT_REPLIED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_FORWARDED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_FORWARDED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NOT_FORWARDED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NOT_FORWARDED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_LOCKED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_LOCKED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NOT_LOCKED
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NOT_LOCKED;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_SPAM
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_SPAM;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_NOT_SPAM 
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_NOT_SPAM;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_HAS_ATTACHMENT
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_HAS_ATTACHMENT;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_HAS_NO_ATTACHMENT
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_HAS_NO_ATTACHMENT;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_SIGNED
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_SIGNED;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_NOT_SIGNED
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_NOT_SIGNED;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_PARTIAL
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_PARTIAL;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_NOT_PARTIAL
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_NOT_PARTIAL;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_COLORLABEL MATCHER_INTEGER
 {
@@ -495,10 +772,10 @@ MATCHER_ALL
        gint value = 0;
 
        criteria = MATCHCRITERIA_COLORLABEL;
-       value = atoi($2);
-       if (value < 1) value = 1;
-       else if (value > MAX_COLORLABELS) value = MAX_COLORLABELS;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 10);
+       if (value < 0) value = 0;
+       else if (value > COLORLABELS) value = COLORLABELS;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_NOT_COLORLABEL MATCHER_INTEGER
 {
@@ -506,24 +783,38 @@ MATCHER_ALL
        gint value = 0;
 
        criteria = MATCHCRITERIA_NOT_COLORLABEL;
-       value = atoi($2);
-       if (value < 1) value = 1;
-       else if (value > MAX_COLORLABELS) value = MAX_COLORLABELS;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       if (value < 0) value = 0;
+       else if (value > COLORLABELS) value = COLORLABELS;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_IGNORE_THREAD
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_IGNORE_THREAD;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_NOT_IGNORE_THREAD
 {
        gint criteria = 0;
 
        criteria = MATCHCRITERIA_NOT_IGNORE_THREAD;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_WATCH_THREAD
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_WATCH_THREAD;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_NOT_WATCH_THREAD
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_NOT_WATCH_THREAD;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_SUBJECT match_type MATCHER_STRING
 {
@@ -532,7 +823,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_SUBJECT;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_SUBJECT match_type MATCHER_STRING
 {
@@ -541,7 +832,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NOT_SUBJECT;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_FROM match_type MATCHER_STRING
 {
@@ -550,7 +841,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_FROM;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_FROM match_type MATCHER_STRING
 {
@@ -559,7 +850,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NOT_FROM;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_TO match_type MATCHER_STRING
 {
@@ -568,7 +859,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_TO;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_TO match_type MATCHER_STRING
 {
@@ -577,7 +868,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NOT_TO;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_CC match_type MATCHER_STRING
 {
@@ -586,7 +877,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_CC;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_CC match_type MATCHER_STRING
 {
@@ -595,7 +886,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NOT_CC;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_TO_OR_CC match_type MATCHER_STRING
 {
@@ -604,7 +895,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_TO_OR_CC;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_TO_AND_NOT_CC match_type MATCHER_STRING
 {
@@ -613,7 +904,39 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NOT_TO_AND_NOT_CC;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
+}
+| MATCHER_TAG match_type MATCHER_STRING
+{
+       gint criteria = 0;
+       gchar *expr = NULL;
+
+       criteria = MATCHCRITERIA_TAG;
+       expr = $3;
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
+}
+| MATCHER_NOT_TAG match_type MATCHER_STRING
+{
+       gint criteria = 0;
+       gchar *expr = NULL;
+
+       criteria = MATCHCRITERIA_NOT_TAG;
+       expr = $3;
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
+}
+| MATCHER_TAGGED
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_TAGGED;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
+}
+| MATCHER_NOT_TAGGED
+{
+       gint criteria = 0;
+
+       criteria = MATCHCRITERIA_NOT_TAGGED;
+       prop = matcherprop_new(criteria, NULL, 0, NULL, 0);
 }
 | MATCHER_AGE_GREATER MATCHER_INTEGER
 {
@@ -621,8 +944,8 @@ MATCHER_ALL
        gint value = 0;
 
        criteria = MATCHCRITERIA_AGE_GREATER;
-       value = atoi($2);
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_AGE_LOWER MATCHER_INTEGER
 {
@@ -630,8 +953,26 @@ MATCHER_ALL
        gint value = 0;
 
        criteria = MATCHCRITERIA_AGE_LOWER;
-       value = atoi($2);
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
+}
+| MATCHER_AGE_GREATER_HOURS MATCHER_INTEGER
+{
+       gint criteria = 0;
+       gint value = 0;
+
+       criteria = MATCHCRITERIA_AGE_GREATER_HOURS;
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
+}
+| MATCHER_AGE_LOWER_HOURS MATCHER_INTEGER
+{
+       gint criteria = 0;
+       gint value = 0;
+
+       criteria = MATCHCRITERIA_AGE_LOWER_HOURS;
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_NEWSGROUPS match_type MATCHER_STRING
 {
@@ -640,7 +981,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NEWSGROUPS;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_NEWSGROUPS match_type MATCHER_STRING
 {
@@ -649,7 +990,25 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NOT_NEWSGROUPS;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
+}
+| MATCHER_MESSAGEID match_type MATCHER_STRING
+{
+       gint criteria = 0;
+       gchar *expr = NULL;
+
+       criteria = MATCHCRITERIA_MESSAGEID;
+       expr = $3;
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
+}
+| MATCHER_NOT_MESSAGEID match_type MATCHER_STRING
+{
+       gint criteria = 0;
+       gchar *expr = NULL;
+
+       criteria = MATCHCRITERIA_NOT_MESSAGEID;
+       expr = $3;
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_INREPLYTO match_type MATCHER_STRING
 {
@@ -658,7 +1017,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_INREPLYTO;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_INREPLYTO match_type MATCHER_STRING
 {
@@ -667,7 +1026,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NOT_INREPLYTO;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_REFERENCES match_type MATCHER_STRING
 {
@@ -676,7 +1035,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_REFERENCES;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_REFERENCES match_type MATCHER_STRING
 {
@@ -685,7 +1044,7 @@ MATCHER_ALL
 
        criteria = MATCHCRITERIA_NOT_REFERENCES;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_SCORE_GREATER MATCHER_INTEGER
 {
@@ -693,8 +1052,8 @@ MATCHER_ALL
        gint value = 0;
 
        criteria = MATCHCRITERIA_SCORE_GREATER;
-       value = atoi($2);
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_SCORE_LOWER MATCHER_INTEGER
 {
@@ -702,8 +1061,8 @@ MATCHER_ALL
        gint value = 0;
 
        criteria = MATCHCRITERIA_SCORE_LOWER;
-       value = atoi($2);
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_SCORE_EQUAL MATCHER_INTEGER
 {
@@ -711,32 +1070,32 @@ MATCHER_ALL
        gint value = 0;
 
        criteria = MATCHCRITERIA_SCORE_EQUAL;
-       value = atoi($2);
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_SIZE_GREATER MATCHER_INTEGER 
 {
        gint criteria = 0;
        gint value    = 0;
        criteria = MATCHCRITERIA_SIZE_GREATER;
-       value = atoi($2);
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_SIZE_SMALLER MATCHER_INTEGER
 {
        gint criteria = 0;
        gint value    = 0;
        criteria = MATCHCRITERIA_SIZE_SMALLER;
-       value = atoi($2);
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_SIZE_EQUAL MATCHER_INTEGER
 {
        gint criteria = 0;
        gint value    = 0;
        criteria = MATCHCRITERIA_SIZE_EQUAL;
-       value = atoi($2);
-       prop = matcherprop_unquote_new(criteria, NULL, 0, NULL, value);
+       value = strtol($2, NULL, 0);
+       prop = matcherprop_new(criteria, NULL, 0, NULL, value);
 }
 | MATCHER_HEADER MATCHER_STRING
 {
@@ -745,10 +1104,10 @@ MATCHER_ALL
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
+       matcher_is_fast = FALSE;
        criteria = MATCHCRITERIA_HEADER;
        expr = $2;
-       prop = matcherprop_unquote_new(criteria, header, match_type, expr, 0);
+       prop = matcherprop_new(criteria, header, match_type, expr, 0);
        g_free(header);
 }
 | MATCHER_NOT_HEADER MATCHER_STRING
@@ -758,83 +1117,127 @@ MATCHER_ALL
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
+       matcher_is_fast = FALSE;
        criteria = MATCHCRITERIA_NOT_HEADER;
        expr = $2;
-       prop = matcherprop_unquote_new(criteria, header, match_type, expr, 0);
+       prop = matcherprop_new(criteria, header, match_type, expr, 0);
        g_free(header);
 }
 | MATCHER_HEADERS_PART match_type MATCHER_STRING
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
+       matcher_is_fast = FALSE;
        criteria = MATCHCRITERIA_HEADERS_PART;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_HEADERS_PART match_type MATCHER_STRING
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
+       matcher_is_fast = FALSE;
        criteria = MATCHCRITERIA_NOT_HEADERS_PART;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
-| MATCHER_MESSAGE match_type MATCHER_STRING
+| MATCHER_HEADERS_CONT match_type MATCHER_STRING
+{
+       gint criteria = 0;
+       gchar *expr = NULL;
+       matcher_is_fast = FALSE;
+       criteria = MATCHCRITERIA_HEADERS_CONT;
+       expr = $3;
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
+}
+| MATCHER_NOT_HEADERS_CONT match_type MATCHER_STRING
+{
+       gint criteria = 0;
+       gchar *expr = NULL;
+       matcher_is_fast = FALSE;
+       criteria = MATCHCRITERIA_NOT_HEADERS_CONT;
+       expr = $3;
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
+}
+| MATCHER_FOUND_IN_ADDRESSBOOK MATCHER_STRING
+{
+       header = g_strdup($2);
+} MATCHER_IN MATCHER_STRING
 {
        gint criteria = 0;
        gchar *expr = NULL;
 
+       criteria = MATCHCRITERIA_FOUND_IN_ADDRESSBOOK;
+       expr = $2;
+       prop = matcherprop_new(criteria, header, match_type, expr, 0);
+       g_free(header);
+}
+| MATCHER_NOT_FOUND_IN_ADDRESSBOOK MATCHER_STRING
+{
+       header = g_strdup($2);
+} MATCHER_IN MATCHER_STRING
+{
+       gint criteria = 0;
+       gchar *expr = NULL;
+
+       criteria = MATCHCRITERIA_NOT_FOUND_IN_ADDRESSBOOK;
+       expr = $2;
+       prop = matcherprop_new(criteria, header, match_type, expr, 0);
+       g_free(header);
+}
+| MATCHER_MESSAGE match_type MATCHER_STRING
+{
+       gint criteria = 0;
+       gchar *expr = NULL;
+       matcher_is_fast = FALSE;
        criteria = MATCHCRITERIA_MESSAGE;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_MESSAGE match_type MATCHER_STRING
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
+       matcher_is_fast = FALSE;
        criteria = MATCHCRITERIA_NOT_MESSAGE;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_BODY_PART match_type MATCHER_STRING
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
+       matcher_is_fast = FALSE;
        criteria = MATCHCRITERIA_BODY_PART;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
 | MATCHER_NOT_BODY_PART match_type MATCHER_STRING
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
+       matcher_is_fast = FALSE;
        criteria = MATCHCRITERIA_NOT_BODY_PART;
        expr = $3;
-       prop = matcherprop_unquote_new(criteria, NULL, match_type, expr, 0);
+       prop = matcherprop_new(criteria, NULL, match_type, expr, 0);
 }
-| MATCHER_EXECUTE MATCHER_STRING
+| MATCHER_TEST MATCHER_STRING
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
-       criteria = MATCHCRITERIA_EXECUTE;
+       matcher_is_fast = FALSE;
+       criteria = MATCHCRITERIA_TEST;
        expr = $2;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, expr, 0);
+       prop = matcherprop_new(criteria, NULL, MATCHTYPE_MATCH, expr, 0);
 }
-| MATCHER_NOT_EXECUTE MATCHER_STRING
+| MATCHER_NOT_TEST MATCHER_STRING
 {
        gint criteria = 0;
        gchar *expr = NULL;
-
-       criteria = MATCHCRITERIA_NOT_EXECUTE;
+       matcher_is_fast = FALSE;
+       criteria = MATCHCRITERIA_NOT_TEST;
        expr = $2;
-       prop = matcherprop_unquote_new(criteria, NULL, 0, expr, 0);
+       prop = matcherprop_new(criteria, NULL, MATCHTYPE_MATCH, expr, 0);
 }
 ;
 
@@ -846,7 +1249,7 @@ MATCHER_EXECUTE MATCHER_STRING
 
        action_type = MATCHACTION_EXECUTE;
        cmd = $2;
-       action = filteringaction_new(action_type, 0, cmd, 0);
+       action = filteringaction_new(action_type, 0, cmd, 0, 0, NULL);
 }
 | MATCHER_MOVE MATCHER_STRING
 {
@@ -855,7 +1258,32 @@ MATCHER_EXECUTE MATCHER_STRING
 
        action_type = MATCHACTION_MOVE;
        destination = $2;
-       action = filteringaction_new(action_type, 0, destination, 0);
+       action = filteringaction_new(action_type, 0, destination, 0, 0, NULL);
+}
+| MATCHER_SET_TAG MATCHER_STRING
+{
+       gchar *destination = NULL;
+       gint action_type = 0;
+
+       action_type = MATCHACTION_SET_TAG;
+       destination = $2;
+       action = filteringaction_new(action_type, 0, destination, 0, 0, NULL);
+}
+| MATCHER_UNSET_TAG MATCHER_STRING
+{
+       gchar *destination = NULL;
+       gint action_type = 0;
+
+       action_type = MATCHACTION_UNSET_TAG;
+       destination = $2;
+       action = filteringaction_new(action_type, 0, destination, 0, 0, NULL);
+}
+| MATCHER_CLEAR_TAGS
+{
+       gint action_type = 0;
+
+       action_type = MATCHACTION_CLEAR_TAGS;
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
 }
 | MATCHER_COPY MATCHER_STRING
 {
@@ -864,56 +1292,70 @@ MATCHER_EXECUTE MATCHER_STRING
 
        action_type = MATCHACTION_COPY;
        destination = $2;
-       action = filteringaction_new(action_type, 0, destination, 0);
+       action = filteringaction_new(action_type, 0, destination, 0, 0, NULL);
 }
 | MATCHER_DELETE
 {
        gint action_type = 0;
 
        action_type = MATCHACTION_DELETE;
-       action = filteringaction_new(action_type, 0, NULL, 0);
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
 }
 | MATCHER_MARK
 {
        gint action_type = 0;
 
        action_type = MATCHACTION_MARK;
-       action = filteringaction_new(action_type, 0, NULL, 0);
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
 }
 | MATCHER_UNMARK
 {
        gint action_type = 0;
 
        action_type = MATCHACTION_UNMARK;
-       action = filteringaction_new(action_type, 0, NULL, 0);
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
 }
 | MATCHER_LOCK
 {
        gint action_type = 0;
 
        action_type = MATCHACTION_LOCK;
-       action = filteringaction_new(action_type, 0, NULL, 0);
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
 }
 | MATCHER_UNLOCK
 {
        gint action_type = 0;
 
        action_type = MATCHACTION_UNLOCK;
-       action = filteringaction_new(action_type, 0, NULL, 0);
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
 }
 | MATCHER_MARK_AS_READ
 {
        gint action_type = 0;
 
        action_type = MATCHACTION_MARK_AS_READ;
-       action = filteringaction_new(action_type, 0, NULL, 0);
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
 }
 | MATCHER_MARK_AS_UNREAD
 {
        gint action_type = 0;
 
        action_type = MATCHACTION_MARK_AS_UNREAD;
-       action = filteringaction_new(action_type, 0, NULL, 0);
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
+}
+| MATCHER_MARK_AS_SPAM
+{
+       gint action_type = 0;
+
+       action_type = MATCHACTION_MARK_AS_SPAM;
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
+}
+| MATCHER_MARK_AS_HAM
+{
+       gint action_type = 0;
+
+       action_type = MATCHACTION_MARK_AS_HAM;
+       action = filteringaction_new(action_type, 0, NULL, 0, 0, NULL);
 }
 | MATCHER_FORWARD MATCHER_INTEGER MATCHER_STRING
 {
@@ -922,9 +1364,10 @@ MATCHER_EXECUTE MATCHER_STRING
        gint account_id = 0;
 
        action_type = MATCHACTION_FORWARD;
-       account_id = atoi($2);
+       account_id = strtol($2, NULL, 10);
        destination = $3;
-       action = filteringaction_new(action_type, account_id, destination, 0);
+       action = filteringaction_new(action_type,
+            account_id, destination, 0, 0, NULL);
 }
 | MATCHER_FORWARD_AS_ATTACHMENT MATCHER_INTEGER MATCHER_STRING
 {
@@ -933,9 +1376,10 @@ MATCHER_EXECUTE MATCHER_STRING
        gint account_id = 0;
 
        action_type = MATCHACTION_FORWARD_AS_ATTACHMENT;
-       account_id = atoi($2);
+       account_id = strtol($2, NULL, 10);
        destination = $3;
-       action = filteringaction_new(action_type, account_id, destination, 0);
+       action = filteringaction_new(action_type,
+            account_id, destination, 0, 0, NULL);
 }
 | MATCHER_REDIRECT MATCHER_INTEGER MATCHER_STRING
 {
@@ -944,9 +1388,10 @@ MATCHER_EXECUTE MATCHER_STRING
        gint account_id = 0;
 
        action_type = MATCHACTION_REDIRECT;
-       account_id = atoi($2);
+       account_id = strtol($2, NULL, 10);
        destination = $3;
-       action = filteringaction_new(action_type, account_id, destination, 0);
+       action = filteringaction_new(action_type,
+            account_id, destination, 0, 0, NULL);
 }
 | MATCHER_COLOR MATCHER_INTEGER
 {
@@ -954,35 +1399,61 @@ MATCHER_EXECUTE MATCHER_STRING
        gint color = 0;
 
        action_type = MATCHACTION_COLOR;
-       color = atoi($2);
-       action = filteringaction_new(action_type, 0, NULL, color);
-}
-| MATCHER_CHANGE_SCORE MATCHER_STRING
-{
-       gint action_type = MATCHACTION_CHANGE_SCORE;
-       char *last_tok = NULL;
-       const gchar *first_tok;
-       long int chscore;
-       int change_type; /* -1 == decrement, 0 == assign, 1 == increment */     
-
-#define is_number(x) ( ((x) == '+') || ((x) == '-') || (isdigit((x))) ) 
-       /* find start */
-       for (first_tok = $2; *first_tok && !is_number(*first_tok); first_tok++)
-               ;            
-#undef is_number               
-       
-       chscore = strtol(first_tok, &last_tok, 10);
-       if (last_tok == first_tok || *last_tok == 0) 
-               chscore = 0;
-       change_type = *first_tok == '+' ? 1 : *first_tok == '-' ? -1 : 0;
-       action = filteringaction_new(MATCHACTION_CHANGE_SCORE, change_type,
-                                    NULL, chscore);
+       color = strtol($2, NULL, 10);
+       action = filteringaction_new(action_type, 0, NULL, color, 0, NULL);
 }
-;
+| MATCHER_CHANGE_SCORE MATCHER_INTEGER
+{
+        gint score = 0;
+        
+        score = strtol($2, NULL, 10);
+       action = filteringaction_new(MATCHACTION_CHANGE_SCORE, 0,
+                                    NULL, 0, score, NULL);
+}
+/* backward compatibility */
+| MATCHER_SCORE MATCHER_INTEGER
+{
+        gint score = 0;
+        
+        score = strtol($2, NULL, 10);
+       action = filteringaction_new(MATCHACTION_CHANGE_SCORE, 0,
+                                    NULL, 0, score, NULL);
+}
+| MATCHER_SET_SCORE MATCHER_INTEGER
+{
+        gint score = 0;
+        
+        score = strtol($2, NULL, 10);
+       action = filteringaction_new(MATCHACTION_SET_SCORE, 0,
+                                    NULL, 0, score, NULL);
+}
+| MATCHER_HIDE
+{
+       action = filteringaction_new(MATCHACTION_HIDE, 0, NULL, 0, 0, NULL);
+}
+| MATCHER_IGNORE
+{
+       action = filteringaction_new(MATCHACTION_IGNORE, 0, NULL, 0, 0, NULL);
+}
+| MATCHER_WATCH
+{
+       action = filteringaction_new(MATCHACTION_WATCH, 0, NULL, 0, 0, NULL);
+}
+| MATCHER_ADD_TO_ADDRESSBOOK MATCHER_STRING
+{
+       header = g_strdup($2);
+} MATCHER_STRING
+{
+       gchar *addressbook = NULL;
+       gint action_type = 0;
 
-scoring_rule:
-MATCHER_SCORE MATCHER_INTEGER
+       action_type = MATCHACTION_ADD_TO_ADDRESSBOOK;
+       addressbook = $2;
+       action = filteringaction_new(action_type, 0, addressbook, 0, 0, header);
+       g_free(header);
+}
+| MATCHER_STOP
 {
-       score = atoi($2);
+       action = filteringaction_new(MATCHACTION_STOP, 0, NULL, 0, 0, NULL);
 }
 ;