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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to