r315755 - Fix -Woverloaded-virtual warning in clang-refactor

David Blaikie via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 16 10:11:25 PDT 2017


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 at 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 at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171016/2e244d1e/attachment.html>


More information about the cfe-commits mailing list