Improve regex to handle spaces

Author: lcartey

c/misra/src/rules/RULE-2-5/UnusedMacroDeclaration.ql

@@ -29,7 +29,7 @@ where
     bd.getHead() = m.getName()
     or
     // Covers the use of defined() to check if a macro is defined
-    m.getName() = bd.getHead().regexpCapture(".*defined\\((.*)\\).*", 1)
+    m.getName() = bd.getHead().regexpCapture(".*defined *\\(? *([^\\s()]+) *\\)?\\.*", 1)
   ) and
   // We consider a macro "used" if the name is undef-ed at some point in the same file, or a file
   // that includes the file defining the macro. This will over approximate use in the case of a

c/misra/test/rules/RULE-2-5/test.c

@@ -42,6 +42,18 @@ void test() {
 #if defined(CHECKED_MACRO_BEFORE_3)
 #endif
 
+#if defined(CHECKED_MACRO_BEFORE_4)
+#endif
+
+#if defined(CHECKED_MACRO_BEFORE_5)
+#endif
+
+#if defined(CHECKED_MACRO_BEFORE_6)
+#endif
+
 #define CHECKED_MACRO_BEFORE_1 // COMPLIANT - used in branch
 #define CHECKED_MACRO_BEFORE_2 // COMPLIANT - used in branch
-#define CHECKED_MACRO_BEFORE_3 // COMPLIANT - used in branch
+#define CHECKED_MACRO_BEFORE_3 // COMPLIANT - used in branch
+#define CHECKED_MACRO_BEFORE_4 // COMPLIANT - used in branch
+#define CHECKED_MACRO_BEFORE_5 // COMPLIANT - used in branch
+#define CHECKED_MACRO_BEFORE_6 // COMPLIANT - used in branch