<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 16, 2013 at 2:37 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank" class="cremed">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> I agree with you in general but in this case consistency looks more<br>
> important than that. Look at the other methods, such as the above two.<br>
> getCOFFHeader() and getPE32Header() will never fail and will always return<br>
> object_error::success. Other methods in the other files in the same<br>
> directory are written in the similar manner. If we change the signature of<br>
> this function, it won't fix the API, but it'd make the API cluttered.<br>
><br>
> I'd think we should change all the other methods in the directory not to<br>
> return error_code, if we think it's the right thing to do. That should be<br>
> done in another patch.<br>
<br>
</div>I think we should change it all, yes. If you don't want to have the<br>
methods temporarily in a mixed state, then please change the existing<br>
ones first.</blockquote></div><br>Rafael, I don't think that's necessary. This is all code that Rui has written and been maintaining. If he would prefer to do the API rework in a follow-up patch that seems totally fine.</div>
</div>