<div dir="ltr"><div class="gmail_default"><font face="arial, helvetica, sans-serif">committed as r204898.</font><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-03-21 18:13 GMT+08:00 Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 21 March 2014 03:00, Jiangning Liu <<a href="mailto:liujiangning1@gmail.com">liujiangning1@gmail.com</a>> wrote:<br>

> Hi Saleem,<br>
><br>
> Your patch r203762 is reverted by r204427, because the fix is even worse<br>
> than before, and it introduced silent failure.<br>
><br>
> BTW, should we follow review process before committing any code to trunk?<br>
<br>
</div>Hi Jiangning, Saleem,<br>
<br>
Yes, I agree we should still review IAS patches before commit, as they<br>
tend to wash with the baby water on my stream and the IAS is not yet<br>
stable or well tested to warrant quick changes.<br>
<br>
cheers,<br>
--renato<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><font face="courier new, monospace">Thanks,</font><div><font face="courier new, monospace">-Jiangning</font></div></div>
</div>