<div dir="rtl"><div dir="ltr">r242150<br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-07-14 17:56 GMT+03:00 İsmail Dönmez <span dir="ltr"><<a href="mailto:ismail@donmez.ws" target="_blank">ismail@donmez.ws</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Works fine out of the box, cheers!<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Jul 14, 2015 at 3:49 PM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>
> Here is updated diff with all comments addressed.<br>
> ismail: could you check this one last time on openSUSE before I commit?<br>
><br>
><br>
> 2015-07-14 14:33 GMT+03:00 Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>>:<br>
>><br>
>> 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>
>> 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>
>><br>
>> Joerg<br>
>> _______________________________________________<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>
><br>
><br>
><br>
> _______________________________________________<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>
><br>
</div></div></blockquote></div><br></div>