[PATCH] D24183: A clang tool for changing surrouding namespaces of class/function definitions.
Kirill Bobyrev via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 13 03:40:57 PDT 2016
omtcyfz added inline comments.
================
Comment at: change-namespace/ChangeNamespace.cpp:359
@@ +358,3 @@
+ End, tok::semi, *Result.SourceManager, Result.Context->getLangOpts(),
+ /*SkipTrailingWhitespaceAndNewLine=*/true);
+ if (AfterSemi.isValid())
----------------
The indentation seems a bit off. Semicolon is on the next line for some reason and I think `/*SkipTrailingWhitespaceAndNewLine=*/` should be `/* SkipTrailingWhitespaceAndNewLine */`.
UPD: Semicolon came back :)
https://reviews.llvm.org/D24183
More information about the cfe-commits
mailing list