<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Oct 25, 2016 at 11:08 AM, Eugene Leviant <span dir="ltr"><<a href="mailto:evgeny.leviant@gmail.com" target="_blank">evgeny.leviant@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">evgeny777 added inline comments.<br>
<span class="gmail-"><br>
<br>
================<br>
Comment at: ELF/InputFiles.cpp:56<br>
+<br>
+template <class ELFT> DIHelper<ELFT>::~DIHelper() { delete DwarfLine; }<br>
+<br>
----------------<br>
</span><span class="gmail-">ruiu wrote:<br>
> Use unique_ptr to remove this dtor.<br>
</span>It's not possible unless you include DWARFContext.h to InputFiles.h or override unique_ptr deleter (second template parameter). Is this better?<br></blockquote><div><br></div><div>You can have an empty dtor in InputFiles.cpp like `template <class ELFT> DIHelper<ELFT>::~DIHelper() {}` to write a definition of the dtor in the cpp file instead of in the header so that you don't need to include the DWARFContext.h, can't you?</div></div></div></div>