Simplify event filtering logic in server_audit plugin

Replace if-else chain with a single bitwise AND check when filtering query
events by type (DDL, DML, DCL, etc.).

The removes the need for the goto statements.

Additionally, we remove the orig_query variable as it served no purpose.

All new code of the whole pull request, including one or several files
that are either new files or modified ones, are contributed under the
BSD-new license

Author: tonychen2001

plugin/server_audit/server_audit.c

@@ -1686,20 +1686,10 @@ static int log_statement_ex(const struct connection_info *cn,
   if (query && !(events & EVENT_QUERY_ALL) &&
       (events & EVENT_QUERY && !cn->log_always))
   {
-    const char *orig_query= query;
-
-    if (events & EVENT_QUERY_DDL && cmdtype & EVENT_QUERY_DDL)
-      goto do_log_query;
-    if (events & EVENT_QUERY_DML && cmdtype & EVENT_QUERY_DML)
-      goto do_log_query;
-    if (events & EVENT_QUERY_DML_NO_SELECT && cmdtype & EVENT_QUERY_DML_NO_SELECT)
-      goto do_log_query;
-    if (events & EVENT_QUERY_DCL && cmdtype & EVENT_QUERY_DCL)
-      goto do_log_query;
-
-    return 0;
-do_log_query:
-    query= orig_query;
+    if (!(events & cmdtype &
+          (EVENT_QUERY_DDL | EVENT_QUERY_DML | EVENT_QUERY_DML_NO_SELECT |
+           EVENT_QUERY_DCL)))
+      return 0;
   }
 
   csize= log_header(message, message_size-1, &ev_time,