[PATCH][analyzer][Review request] Eliminate memory leak in BugReporter::emitReport()

David Blaikie dblaikie at gmail.com
Fri Mar 7 09:27:28 PST 2014


Please prefer unique_ptr - a migration of existing OwningPtrs is
underway & it'd be nice not to save Ahmed the extra work/iteration on
further uses.

(and bonus points if you can push ownership a bit further with
unique_ptr (eg: take unique_ptr by value as a parameter to this
function, etc) if owners are willing)

On Fri, Mar 7, 2014 at 6:41 AM, Anton Yartsev <anton.yartsev at gmail.com> wrote:
> Hi all!
>
> Attached is the patch that eliminates memory leak in
> BugReporter::emitReport() found by alpha.cplusplus.NewDeleteLeaks ran over
> LLVM codebase. Please review.
>
> --
> Anton
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



More information about the cfe-commits mailing list