Re: [PATCH] D20296: clang-rename: avoid StringRef members in USRLocFindingASTVisitor

2016-05-17 Thread Miklos Vajna via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL269796: clang-rename: avoid StringRef members in USRLocFindingASTVisitor (authored by vmiklos). Changed prior to commit: http://reviews.llvm.org/D20296?vs=57385&id=57503#toc Repository: rL LLVM http

Re: [PATCH] D20296: clang-rename: avoid StringRef members in USRLocFindingASTVisitor

2016-05-17 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. LG http://reviews.llvm.org/D20296 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

[PATCH] D20296: clang-rename: avoid StringRef members in USRLocFindingASTVisitor

2016-05-16 Thread Miklos Vajna via cfe-commits
vmiklos created this revision. vmiklos added reviewers: klimek, cfe-commits. Even if this is defined in the .cpp file and only used as part of the function (so here it's safe), usually storing StringRefs in the class is dangerous, so don't do so. http://reviews.llvm.org/D20296 Files: clang-ren