[PATCH] D138780: [include-cleaner] Minor fixes to parseIWYUPragma:

2022-11-28 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. Thanks! The change is LG, I think it would be great to have some tests. Comment at: clang-tools-extra/include-cleaner/lib/Record.cpp:146 // to share the code? static llvm::Optional parseIWYUPragma(const char *Text) { + // Skip the comment start, // or

[PATCH] D138780: [include-cleaner] Minor fixes to parseIWYUPragma:

2022-11-28 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: hokein. Herald added a subscriber: kadircet. Herald added a project: All. sammccall requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. - remove assert that can fail for i