<div dir="ltr">Hi Mehdi,<div><br></div><div>Thanks for the +1, and for offering to help out with reviews. I really appreciate that. :)</div><div><br></div><div>- Lang.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 10, 2016 at 8:07 PM, Mehdi Amini <span dir="ltr"><<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On Feb 10, 2016, at 7:19 PM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
><br>
>> In the long-term I expect many parts of LLVM may want to migrate from<br>
>> std::error_code to TypedError, but that's ultimately a decision for the<br>
>> maintainers of the respective parts. I'll be very happy to provide input and<br>
>> support, but I don't want to pre-empt anyone else's decision on whether to<br>
>> adopt this.<br>
><br>
> That means we will forever have an incomplete transition.<br>
><br>
> We still haven't finished moving to the "new" naming style. Moving to<br>
> the current path handling took from Nov 2009 to Jun 2013.<br>
><br>
> I am still not convinced that the new system is better than a<br>
> asserting wrapper over std::error_code with diag handlers, but I don't<br>
> have the time to change Orc to use to show it. Given that I also<br>
> failed to convince you otherwise, we will probably have to go with the<br>
> view of who can actually code this.<br>
><br>
> But *please*, make an effort to use it everywhere. We still have a mix<br>
> of bool+std::string, std::error_code and ErrorOr. Are each of these<br>
> better than what you are proposing in one case or the other? Probably.<br>
> Is it worth it having unique snow flakes?  I don't think so.<br>
><br>
> In particular, please don't change just the "MachO-specific part of<br>
> X". In the past I have made an effort to keep MachO bits current when<br>
> working in MC and lib/Object. Please do the corresponding effort. For<br>
> example, if you change llvm-nm, please change all of it<br>
<br>
</span>FWIW, I support the effort Lang is leading, so it is a +1 on my side to move on!<br>
And I also agree with Rafael that it would be really better to limit as much as possible the mix of multiple reporting inside a component.<br>
<br>
Lang: please CC me on the reviews, I'll be happy to follow the development.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Mehdi<br>
<br>
<br>
<br>
</font></span></blockquote></div><br></div>