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