<div dir="ltr">It looks like although it's not mentioned in the coding standard, nobody is opposing to invert it for this case, so I'll do it. I agree with Michael. Returning true on success seems natural to me. However, I believe the most important thing is to maintain consistency of boolean return value at least within LLD whether it should be true or false.</div>

<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 24, 2013 at 1:51 PM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.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="im">On 9/24/2013 2:33 PM, Rui Ueyama wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi LLD developers,<br>
<br>
I'm about to make a change to invert the return value of Driver::parse() to return true on success. Currently it returns false on success.<br>
</blockquote></div>
You have to be a bit careful here, as the return from parse is used as a exit code from lld.<br>
<br>
The Unix shell interprets 0 (false as success) as 1(true as failure).<br>
<br>
It has to be consistent throughout for this reason.<br>
<br>
As Bigcheese mentioned,  if its in the Coding standard this would really help.<br>
<br>
Thanks<br>
<br>
Shankar Easwaran<div class="HOEnZb"><div class="h5"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
In many other functions, we return true to indicate success and false to indicate failure. The inconsistency is confusing, and fixing it should improve code readability.<br>
<br>
</blockquote>
<br>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
<br>
</font></span></blockquote></div><br></div>