<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 23 January 2015 at 17:14, Kevin Enderby <span dir="ltr"><<a href="mailto:enderby@apple.com" target="_blank">enderby@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Added Lang to the To: line and he maybe better to comment.<div><br></div><div>While the change looks cleaner I’m not sure what the error handling should be for malformed Mach-O files in llvm’s libObject.  If malformed Mach-O files cause asserts it is difficult to build tools to print and diagnose bad files.  For example in Mac OS otool(1) is used both to display information about good Mach-O files and do its best to display what it can with warning about  malformed Mach-O files.  For the guy producing Mach-O files which some times are broken this is a valuable approach for have otool(1) be able so show what it can instead of just stopping.</div></div></blockquote><div><br></div><div>I agree that it should not assert.</div><div><br></div><div>The intention was to point out that the original change was (and still is) untested.</div><div><br></div><div>Cheers,</div><div>Rafael</div><div><br></div></div></div></div>