vmiklos marked 2 inline comments as done.
vmiklos added a comment.

> rename-at isn't necessary here anymore since it's going to be default 
> behavior IIUC


Indeed, it can be changed back now, done.

> docs should be fixed correspondingly; i.e. prefer to write clang-rename 
> -offset=42.... over clang-rename -rename-at


Done.

> here and later, too


`clang-rename rename-at` is now only mentioned by the document when it comes to
`clang-rename rename-at -help`, which is still necessary, as `clang-rename
-help` talks about the subcommands only. I've updated `clang-rename -help` so
that it points out that `rename-at` is the default, though.


https://reviews.llvm.org/D21814



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

Reply via email to