<div dir="rtl"><div dir="ltr">Here is updated diff with all comments addressed. <br></div><div dir="ltr">ismail: could you check this one last time on openSUSE before I commit?<br></div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-07-14 14:33 GMT+03:00 Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Jul 13, 2015 at 11:53:47PM +0300, Yaron Keren wrote:<br>
> Re-reading, the problem is with using #ifdef inside the conditional or<br>
> using the #ifdef at all?<br>
<br>
</span>the problem is adding #else blocks that themselve that with else.<br>
Can you restructure the branches to avoid that? A single trailing<br>
conditional block is fine as is a single leading conditional block.<br>
But having #if #else #endif where both start with a else is bad.<br>
<span class="HOEnZb"><font color="#888888"><br>
Joerg<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>