Fix C++ compiler warning (#989)

Remove parentheses around `(name)` because gcc8-c++ complains that they're unnecessary.

Unfortunately, after removal clang-tidy complains that we're not putting parentheses around a macro argument. I choose to silence clang-tidy because it's easier to silence an optional tool than figure out how to silence a whole family of compilers.

include/aws/common/string.h

@@ -291,7 +291,10 @@ int aws_array_list_comparator_string(const void *a, const void *b);
         const size_t len;                                                                                              \
         const uint8_t bytes[sizeof(literal)];                                                                          \
     } name##_s = {NULL, sizeof(literal) - 1, literal};                                                                 \
-    static const struct aws_string *(name) = (struct aws_string *)(&name##_s)
+    static const struct aws_string *name = (struct aws_string *)(&name##_s) /* NOLINT(bugprone-macro-parentheses) */
+
+/* NOLINT above is because clang-tidy complains that (name) isn't in parentheses,
+ * but gcc8-c++ complains that the parentheses are unnecessary */
 
 /*
  * A related macro that declares the string pointer without static, allowing it to be externed as a global constant