google
diff --git a/‎googlemock/include/gmock/gmock-matchers.h
Copy file name to clipboardExpand all lines: googlemock/include/gmock/gmock-matchers.h
+6-1Lines changed: 6 additions & 1 deletion b/‎googlemock/include/gmock/gmock-matchers.h
Copy file name to clipboardExpand all lines: googlemock/include/gmock/gmock-matchers.h
+6-1Lines changed: 6 additions & 1 deletion
@@ -4098,7 +4098,12 @@ class ArgsMatcherImpl : public MatcherInterface<ArgsTuple> {
     const char* sep = "";
     // Workaround spurious C4189 on MSVC<=15.7 when k is empty.
     (void)sep;
-    const char* dummy[] = {"", (*os << sep << "#" << k, sep = ", ")...};
+    // The static_cast to void is needed to silence Clang's -Wcomma warning.
+    // This pattern looks suspiciously like we may have mismatched parentheses
+    // and may have been trying to use the first operation of the comma operator
+    // as a member of the array, so Clang warns that we may have made a mistake.
+    const char* dummy[] = {
+        "", (static_cast<void>(*os << sep << "#" << k), sep = ", ")...};
     (void)dummy;
     *os << ") ";
   }