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