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

Code Editor: Fix regression with using doc comments for code regions

This commit is contained in:
Danil Alexeev 2023-10-12 23:20:40 +03:00
parent b1371806ad
commit 881fe67d94
No known key found for this signature in database
GPG Key ID: 124453E157DA8DC7
3 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/gdscript/editor/gdscript_highlighter.cpp
View File

@ -907,6 +907,8 @@ void GDScriptSyntaxHighlighter::add_color_region(const String &p_start_key, cons
ERR_FAIL_COND_MSG(color_regions[i].start_key == p_start_key, "color region with start key '" + p_start_key + "' already exists."); ERR_FAIL_COND_MSG(color_regions[i].start_key == p_start_key, "color region with start key '" + p_start_key + "' already exists.");
if (p_start_key.length() < color_regions[i].start_key.length()) { if (p_start_key.length() < color_regions[i].start_key.length()) {
at++; at++;
} else {
break;
} }
} }

5
scene/gui/code_edit.cpp
View File

@ -2862,7 +2862,8 @@ void CodeEdit::_update_code_region_tags() {
return; return;
} }
for (int i = 0; i < delimiters.size(); i++) { // A shorter delimiter has higher priority.
for (int i = delimiters.size() - 1; i >= 0; i--) {
if (delimiters[i].type != DelimiterType::TYPE_COMMENT) { if (delimiters[i].type != DelimiterType::TYPE_COMMENT) {
continue; continue;
} }
@ -3104,6 +3105,8 @@ void CodeEdit::_add_delimiter(const String &p_start_key, const String &p_end_key
ERR_FAIL_COND_MSG(delimiters[i].start_key == p_start_key, "delimiter with start key '" + p_start_key + "' already exists."); ERR_FAIL_COND_MSG(delimiters[i].start_key == p_start_key, "delimiter with start key '" + p_start_key + "' already exists.");
if (p_start_key.length() < delimiters[i].start_key.length()) { if (p_start_key.length() < delimiters[i].start_key.length()) {
at++; at++;
} else {
break;
} }
} }

6
tests/scene/test_code_edit.h
View File

@ -2944,15 +2944,15 @@ TEST_CASE("[SceneTree][CodeEdit] region folding") {
CHECK(code_edit->is_line_code_region_end(2)); CHECK(code_edit->is_line_code_region_end(2));
// Update code region delimiter when removing comment delimiter. // Update code region delimiter when removing comment delimiter.
code_edit->set_text("//region region_name\nline2\n//endregion\n#region region_name\nline2\n#endregion"); code_edit->set_text("#region region_name\nline2\n#endregion\n//region region_name\nline2\n//endregion");
code_edit->clear_comment_delimiters(); code_edit->clear_comment_delimiters();
code_edit->add_comment_delimiter("//", ""); code_edit->add_comment_delimiter("//", "");
code_edit->add_comment_delimiter("#", ""); code_edit->add_comment_delimiter("#", ""); // A shorter delimiter has higher priority.
CHECK(code_edit->is_line_code_region_start(0)); CHECK(code_edit->is_line_code_region_start(0));
CHECK(code_edit->is_line_code_region_end(2)); CHECK(code_edit->is_line_code_region_end(2));
CHECK_FALSE(code_edit->is_line_code_region_start(3)); CHECK_FALSE(code_edit->is_line_code_region_start(3));
CHECK_FALSE(code_edit->is_line_code_region_end(5)); CHECK_FALSE(code_edit->is_line_code_region_end(5));
code_edit->remove_comment_delimiter("//"); code_edit->remove_comment_delimiter("#");
CHECK_FALSE(code_edit->is_line_code_region_start(0)); CHECK_FALSE(code_edit->is_line_code_region_start(0));
CHECK_FALSE(code_edit->is_line_code_region_end(2)); CHECK_FALSE(code_edit->is_line_code_region_end(2));
CHECK(code_edit->is_line_code_region_start(3)); CHECK(code_edit->is_line_code_region_start(3));