Add error highlighting for duplicate variables/constants
This commit is contained in:
parent
d93bec61c3
commit
ef2aeca5df
|
@ -1535,7 +1535,7 @@ GDScriptParser::SuiteNode *GDScriptParser::parse_suite(const String &p_context,
|
||||||
VariableNode *variable = static_cast<VariableNode *>(statement);
|
VariableNode *variable = static_cast<VariableNode *>(statement);
|
||||||
const SuiteNode::Local &local = current_suite->get_local(variable->identifier->name);
|
const SuiteNode::Local &local = current_suite->get_local(variable->identifier->name);
|
||||||
if (local.type != SuiteNode::Local::UNDEFINED) {
|
if (local.type != SuiteNode::Local::UNDEFINED) {
|
||||||
push_error(vformat(R"(There is already a %s named "%s" declared in this scope.)", local.get_name(), variable->identifier->name));
|
push_error(vformat(R"(There is already a %s named "%s" declared in this scope.)", local.get_name(), variable->identifier->name), variable->identifier);
|
||||||
}
|
}
|
||||||
current_suite->add_local(variable, current_function);
|
current_suite->add_local(variable, current_function);
|
||||||
break;
|
break;
|
||||||
|
@ -1550,7 +1550,7 @@ GDScriptParser::SuiteNode *GDScriptParser::parse_suite(const String &p_context,
|
||||||
} else {
|
} else {
|
||||||
name = "variable";
|
name = "variable";
|
||||||
}
|
}
|
||||||
push_error(vformat(R"(There is already a %s named "%s" declared in this scope.)", name, constant->identifier->name));
|
push_error(vformat(R"(There is already a %s named "%s" declared in this scope.)", name, constant->identifier->name), constant->identifier);
|
||||||
}
|
}
|
||||||
current_suite->add_local(constant, current_function);
|
current_suite->add_local(constant, current_function);
|
||||||
break;
|
break;
|
||||||
|
|
Loading…
Reference in New Issue