search_expr.c: Remove unused search_op_name

Due to the way search_expr.c is linked in with cunit tests, this variable causes an odr violation that's detected by asan.
ksmurchison · Feb 12, 2025 · f115538 · f115538
1 parent e9a3955
commit f115538
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 18 deletions.
diff --git a/imap/search_expr.c b/imap/search_expr.c
@@ -72,22 +72,6 @@ static search_expr_t **the_rootp;
 static search_expr_t *the_focus;
 #endif
 
-/* keep this in sync with enum search_op in search_expr.h */
-EXPORTED const char *search_op_name[] = {
-    "SEOP_UNKNOWN",
-    "SEOP_TRUE",
-    "SEOP_FALSE",
-    "SEOP_LT",
-    "SEOP_LE",
-    "SEOP_GT",
-    "SEOP_GE",
-    "SEOP_MATCH",
-    "SEOP_FUZZYMATCH",
-    "SEOP_AND",
-    "SEOP_OR",
-    "SEOP_NOT",
-};
-
 static void split(search_expr_t *e,
                   void (*cb)(const char *, search_expr_t *, search_expr_t *, void *),
                   void *rock);

diff --git a/imap/search_expr.h b/imap/search_expr.h
@@ -51,7 +51,6 @@
 struct protstream;
 struct index_state;
 
-/* keep this in sync with search_op_name in search_expr.c */
 enum search_op {
     SEOP_UNKNOWN,
     SEOP_TRUE,
@@ -71,7 +70,6 @@ enum search_op {
     SEOP_OR,
     SEOP_NOT,
 };
-extern const char *search_op_name[];
 
 union search_value {
     time_t t;