<div dir="ltr">Hi Rafael,<div><br></div><div>"<span style="font-size:13px">But it is important to </span><span style="font-size:13px">stress that we are not the common case. We should not make objdump </span><span style="font-size:13px">harder to maintain because a handful of developers sometimes find it </span><span style="font-size:13px">useful to see more than one error printed on a file they just created."</span><div><br></div><div>No comment on the rest of the patch (I haven't read it), but regarding error handling schemes I'm not sure I understand the objection. Most users would be unaffected by any change in error handling since errors in objects files are uncommon. Some users care a lot, but they're the same people volunteering to do the development and maintenance (e.g. Kevin and Pete). The changes shouldn't place undue burden on anyone else.</div><div><br></div><div>If it's a matter of avoiding polluting ELF and COFF with error schemes that you don't want to maintain, maybe we can confine the richer error-checking to the mach-o specific parts, and have the top-level mach-o specific function error out with fatal_error if any errors are reported.   </div><div><br></div><div>Cheers,</div><div>Lang.</div></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 14, 2015 at 3:29 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> For me it is extremely common to see broken .o files and that is what I do for a living.  And I write these tools for me to do my job.  If others find them useful great!  But they are needed to look at good and bad .o files.<br>
><br>
<br>
</span>I see files like that for the same reason. But it is important to<br>
stress that we are not the common case. We should not make objdump<br>
harder to maintain because a handful of developers sometimes find it<br>
useful to see more than one error printed on a file they just created.<br>
<br>
And given the option of multiple errors with a good chance of only the<br>
first one being relevant or a single error that is far easier to put a<br>
breakpoint on I would always take the single error.<br>
<div class="HOEnZb"><div class="h5"><br>
Cheers,<br>
Rafael<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>