kbobyrev added a comment.

Thanks Aaron!

Yes, fair enough, that looks good to me, probably no need to move a sanity 
check to the front of `main()` in this case.

Also, just as a note: I don't know how many people use `clang-rename` anymore 
(especially with `clangd` being present), I rewrote a lot of what clang-rename 
has to offer for `clangd` with better precision and results. I think the tool 
is still useful, but not as much as before.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148439/new/

https://reviews.llvm.org/D148439

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

Reply via email to