Tbh I felt dirty calling it Ex, so thanks for calling me out on it :)<br><br>I'll whip up some changes later<br><div class="gmail_quote"><div dir="ltr">On Thu, Mar 2, 2017 at 6:09 PM Jim Ingham via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jingham added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Is it horrible of me to ask that we choose a name that is more descriptive than DataExtractorEx?  That "Ex" extension to a class name is just such a punt, and I'd really rather not encourage it.<br class="gmail_msg">
<br class="gmail_msg">
The subclass you made adds the ability to:<br class="gmail_msg">
<br class="gmail_msg">
(a) Dump itself<br class="gmail_msg">
(b) Extract some Dwarf EH Frame specific bits.<br class="gmail_msg">
<br class="gmail_msg">
And (b) is only used in DwarfCallFrameInfo.cpp,<br class="gmail_msg">
<br class="gmail_msg">
So it would be clearer to make a class that just does (a) and call it DataExtractorDumpable.<br class="gmail_msg">
<br class="gmail_msg">
Then you could have a local class override (like DataExtractorEH) to DwarfCallFrameInfo that adds the GetGNUEHPointer.  That latter seems awfully specific, and probably doesn't need to be sitting out in the general world.  It certainly doesn't need to be riding along all the dumpable data extractors.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D30560" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D30560</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>