LegalizeAdulthood added inline comments. Herald added a reviewer: njames93. Herald added a subscriber: PiotrZSL.
================ Comment at: clang-tools-extra/test/clang-tidy/checkers/modernize/redundant-void-arg.cpp:596 + +// Explicitly specifying `(void)` is a way to sidestep -Wvexing-parse, so we +// should not warn about it here. ---------------- This is not the only way to eliminate the vexing parse warning. For instance, if the intention is to declare `foo` as a function not in this translation unit, then: ``` void most_vexing_parse() { extern int foo(); } ``` does the trick. Why assume that the only way to fix vexing-parse is to add `(void)`? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126735/new/ https://reviews.llvm.org/D126735 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits