Mercurial > ldforge / changelog

also removed the if() case mentioned in the prior commit from addObjectDialog.cpp. :P

• Tue, 22 Oct 2013 13:49:55 +0300
• by Santeri Piippo <crimsondusk64@gmail.com> [Tue, 22 Oct 2013 13:49:55 +0300] rev 524
• also removed the if() case mentioned in the prior commit from addObjectDialog.cpp. :P

enclosed the assert replacement in braces. In release mode, the assert is replaced with nothing and there's a little bit of code in addObjectDialog which uses it as a statement in an one-line if(). With the assert disappearing in release builds this triggered a GCC warning, as the if would be left without a body.

• Tue, 22 Oct 2013 13:47:41 +0300
• by Santeri Piippo <crimsondusk64@gmail.com> [Tue, 22 Oct 2013 13:47:41 +0300] rev 523
• enclosed the assert replacement in braces. In release mode, the assert is replaced with nothing and there's a little bit of code in addObjectDialog which uses it as a statement in an one-line if(). With the assert disappearing in release builds this triggered a GCC warning, as the if would be left without a body.