<div dir="ltr">Okay - so you're not aware then of any cases where we build ObjectFileELF files incrementally and might need to re-parse the headers to incrementally pick up anything new? (I'm not aware of it but just raise it as "Do we do this anywhere?").</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 11, 2014 at 8:01 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 11 July 2014 10:50, Todd Fiala <<a href="mailto:tfiala@google.com">tfiala@google.com</a>> wrote:<br>
><br>
> 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?<br>
<br>
</div>That's correct - I was specifically talking about the first patch.<br>
<br>
I do not believe the second patch will break anything for me -- if we<br>
have an ELF file with no sections then m_section_headers.empty() will<br>
be true, and we'll just have an additional ParseSectionHeaders() call,<br>
as we do today.  Either way I think the first patch is correct and<br>
that we should apply it, regardless of whether we also apply the<br>
second patch.  If m_section_headers is not empty() then we must have<br>
already parsed the section headers so should not fall back to the<br>
defaults.<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>