zdoom
/
gzdoom-gles
mirror of https://github.com/ZDoom/gzdoom-gles.git
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

- fixed: An 'if' statement with only one branch may never considered to be always returning.

This commit is contained in:
Christoph Oelckers 2016-11-06 14:14:54 +01:00
parent 6414fd301b
commit 647fd53a15
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/scripting/codegeneration/codegen.cpp
View File

@ -7441,9 +7441,9 @@ ExpEmit FxIfStatement::Emit(VMFunctionBuilder *build)
bool FxIfStatement::CheckReturn()
{
//An if statement returns if both branches return, if present.
return (WhenTrue == nullptr || WhenTrue->CheckReturn()) &&
(WhenFalse == nullptr || WhenFalse->CheckReturn());
//An if statement returns if both branches return. Both branches must be present.
return WhenTrue != nullptr && WhenTrue->CheckReturn() &&
WhenFalse != nullptr && WhenFalse->CheckReturn();
}
//==========================================================================