ioeric added a comment.

Lg with one nit.



================
Comment at: clang-move/ClangMove.cpp:280
       assert(It < CurrentNamespaces.rend());
-      NewCode += "} // namespace " + *It + "\n";
+      NewCode += "} // namespace " + *It + "\n\n";
     }
----------------
Wouldn't this create something like:
```
namespace x {
namespace y {
...
} // namespace y
  <----------------------------extra \n.
} // namespace x
```


https://reviews.llvm.org/D26493



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

Reply via email to