vmiklos marked 2 inline comments as done. vmiklos added a comment. > Would also be nice to support at least `offset + new-name` in YAML input > files, too.
Done. > Please move this block upwards [preferably right after line 42]. I consider > information about limitations and editor more important than numerous > clang-rename's terminal interface invocation examples. Makes sense, done. I wouldn't mind if rename-all wouldn't be advertised (apart from dumping the options) at all in the documentation, but then users would have to look at the source code for what is the wanted YAML format, and *that* would be bad, I guess. ;-) > https://reviews.llvm.org/D23158 introduces simpler clang-rename invocations > in tests. Please do that here, too. Done. https://reviews.llvm.org/D23198 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
