On 11/03/2013 10:08, Dmitry Kovalenko wrote:
> Hi
>
> Could somebody look, test and (if all is ok) commit my changes in FB
> source code for avoid the compiler warning C4390 for code like
> "if(condition) fb_assert(false);"
>
>
IMO it's much better to just define fb_assert to "do {} while (false)"
in release, like it's done in debug.

do ... while (false) is the only right way to create macro with multiple
(or in this case, empty) commands.


Adriano


------------------------------------------------------------------------------
Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester  
Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the  
endpoint security space. For insight on selecting the right partner to 
tackle endpoint security challenges, access the full report. 
http://p.sf.net/sfu/symantec-dev2dev
Firebird-Devel mailing list, web interface at 
https://lists.sourceforge.net/lists/listinfo/firebird-devel

Reply via email to