klimek added inline comments. ================ Comment at: lib/Format/Format.cpp:1395 @@ -1394,2 +1394,3 @@ +// If any replacements in \p Replaces fails to apply, this returns \p Replaces. template <typename T> ---------------- ioeric wrote: > klimek wrote: > > Why would we want to return the same replacements? > this is just a temp solution to keep the current interfaces of > processReplacements and formatReplacements...should I add a FIXME or can I > change them in this patch as well? Both are good options :) Choose one.
http://reviews.llvm.org/D21601 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits