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
