At the moment this method override is not used by the clang-refactor tool, so I don't think I can add a test for it.
On 16 October 2017 at 10:11, David Blaikie <dblai...@gmail.com> wrote: > Is there a test that could be added to cover this new code? > > On Fri, Oct 13, 2017 at 2:15 PM Alex Lorenz via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: arphaman >> Date: Fri Oct 13 14:15:25 2017 >> New Revision: 315755 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=315755&view=rev >> Log: >> Fix -Woverloaded-virtual warning in clang-refactor >> >> Modified: >> cfe/trunk/tools/clang-refactor/ClangRefactor.cpp >> >> Modified: cfe/trunk/tools/clang-refactor/ClangRefactor.cpp >> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/clang- >> refactor/ClangRefactor.cpp?rev=315755&r1=315754&r2=315755&view=diff >> ============================================================ >> ================== >> --- cfe/trunk/tools/clang-refactor/ClangRefactor.cpp (original) >> +++ cfe/trunk/tools/clang-refactor/ClangRefactor.cpp Fri Oct 13 14:15:25 >> 2017 >> @@ -314,6 +314,10 @@ public: >> SourceChanges.insert(SourceChanges.begin(), Changes.begin(), >> Changes.end()); >> } >> >> + void handle(SymbolOccurrences Occurrences) override { >> + RefactoringResultConsumer::handle(std::move(Occurrences)); >> + } >> + >> const AtomicChanges &getSourceChanges() const { return SourceChanges; } >> >> private: >> >> >> _______________________________________________ >> cfe-commits mailing list >> cfe-commits@lists.llvm.org >> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >> >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits