<div dir="ltr">Yeah that was the one piece I was thinking of, Ed.  (You'll recall I broke one of your scenarios when I did that very thing - expecting 1+ section headers before I'd return a proper architecture for the elf).<div>
<br></div><div>In Matthew's first patch, all it did was say "hey if you do have section headers already parsed, we know we've done the note parsing too and short circuit the bits we've already run.  I think the only time we'd get in trouble with this is if we're somehow building and extending an ObjectFileELF in stages and really do need to parse them multiple times.</div>
<div><br></div><div>I was about to give his second patch a look now.  When you looked at it, you were looking at his first one, is that correct?  Any thoughts on the second one?</div><div><br></div><div>-Todd</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Jul 11, 2014 at 7:24 AM, Ed Maste <span dir="ltr"><<a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On 10 July 2014 21:48, Todd Fiala <<a href="mailto:tfiala@google.com">tfiala@google.com</a>> wrote:<br>
><br>
> Matthew,<br>
><br>
> Your patch looks good to me. I would like Greg and/or Ed to have a look at it too. If they don’t object, I can submit it.<br>
><br>
> I ran tests on Linux and MacOSX with the patch applied<br>
<br>
</div>The change LGTM and it passes on FreeBSD.<br>
<br>
One little point to keep in mind is that there are ELF files that have<br>
no section headers.  We should avoid (further) conflating<br>
m_section_headers.empty() with having not yet parsed the header.<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>