On Thu, Nov 3, 2016 at 10:52 AM Chandler Carruth via cfe-commits < cfe-commits@lists.llvm.org> wrote:
> Author: chandlerc > Date: Thu Nov 3 12:42:32 2016 > New Revision: 285946 > > URL: http://llvm.org/viewvc/llvm-project?rev=285946&view=rev > Log: > Using release to free memory is at best confusing -- one would expect > that its result is in fact used. Instead, use reset. > Perhaps I'm missing something/perhaps you could provide some more context/detail: release() wouldn't've free'd memory, it would've leaked it, right? (seems more than just confusing, but actually wrong - so I'm confused by your description) > > This was pointed out by PVS-Studio. > > Modified: > cfe/trunk/lib/Frontend/VerifyDiagnosticConsumer.cpp > > Modified: cfe/trunk/lib/Frontend/VerifyDiagnosticConsumer.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/VerifyDiagnosticConsumer.cpp?rev=285946&r1=285945&r2=285946&view=diff > > ============================================================================== > --- cfe/trunk/lib/Frontend/VerifyDiagnosticConsumer.cpp (original) > +++ cfe/trunk/lib/Frontend/VerifyDiagnosticConsumer.cpp Thu Nov 3 > 12:42:32 2016 > @@ -43,7 +43,7 @@ VerifyDiagnosticConsumer::~VerifyDiagnos > assert(!CurrentPreprocessor && "CurrentPreprocessor should be > invalid!"); > SrcManager = nullptr; > CheckDiagnostics(); > - Diags.takeClient().release(); > + Diags.takeClient().reset(); > } > > #ifndef NDEBUG > > > _______________________________________________ > 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
