zturner added inline comments.

> labath wrote in Args.cpp:207
> I think we don't need to call Clear() here, as all memory owned by the object 
> will be correctly destroyed anyway.

Thanks for pointing that out.  Originally I wasn't using `unique_ptr` but 
rather `new[]` and `delete[]`.  After I changed to `unique_ptr<char[]>` this 
became unnecessary and I forgot to remove it.  Thanks for pointing it out.

https://reviews.llvm.org/D25099



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to