godotengine/godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #7723 from lonesurvivor/gdscript-range-fix

Browse Source 
Fix parsing bug which causes range(variable) to crash the engine
This commit is contained in:
Rémi Verschelde 2017-02-12 23:13:08 +01:00 committed by GitHub
commit 530de920d4
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
modules/gdscript/gd_parser.cpp
View File

@ -2577,7 +2577,7 @@ void GDParser::_parse_block(BlockNode *p_block,bool p_static) {
Vector<Node*> args; Vector<Node*> args;
Vector<double> constants; Vector<double> constants;
bool constant=true; bool constant=false;
for(int i=1;i<op->arguments.size();i++) { for(int i=1;i<op->arguments.size();i++) {
args.push_back(op->arguments[i]); args.push_back(op->arguments[i]);
@ -2585,13 +2585,12 @@ void GDParser::_parse_block(BlockNode *p_block,bool p_static) {
ConstantNode *c = static_cast<ConstantNode*>(op->arguments[i]); ConstantNode *c = static_cast<ConstantNode*>(op->arguments[i]);
if (c->value.get_type()==Variant::REAL || c->value.get_type()==Variant::INT) { if (c->value.get_type()==Variant::REAL || c->value.get_type()==Variant::INT) {
constants.push_back(c->value); constants.push_back(c->value);
} else { constant=true;
constant=false;
} }
} }
} }
if (args.size()>0 || args.size()<4) { if (args.size()>0 && args.size()<4) {
if (constant) { if (constant) {