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

[Jordan Rose]

Ha, I was thinking about the "pushing ownership a bit further" and decided it would be unfair to ask that right now. Yes, that's the direction I'd like to go, too.

Jordan


On Mar 7, 2014, at 9:27 , David Blaikie <dblaikie at gmail.com> wrote:

> 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
>> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits