[clang] [clang-format] Option to ignore PP directives (PR #70338)

Björn Schäpers via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 12 11:44:27 PST 2023


================
@@ -1134,6 +1134,14 @@ void UnwrappedLineParser::parsePPDefine() {
     return;
   }
 
+  if (Style.IgnorePPDefinitions) {
+    do {
+      nextToken();
+    } while (!eof());
----------------
HazardyKnusperkeks wrote:

I assume you have tested this on real code. Can you explain why it is okay to check for `eof()`? I really don't know this code, and am a bit surprised.

https://github.com/llvm/llvm-project/pull/70338


More information about the cfe-commits mailing list