From 5d124c4a8f86176e8adafab3d3703f28e597cb12 Mon Sep 17 00:00:00 2001 From: Hugo Locurcio Date: Sun, 25 Apr 2021 16:40:58 +0200 Subject: Remove uses of `auto` for better readability and online code reviews The current code style guidelines forbid the use of `auto`. Some uses of `auto` are still present, such as in UWP code (which can't be currently tested) and macros (where removing `auto` isn't easy). --- modules/gdscript/tests/gdscript_test_runner.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'modules/gdscript') diff --git a/modules/gdscript/tests/gdscript_test_runner.cpp b/modules/gdscript/tests/gdscript_test_runner.cpp index f53c3046e6..76ae43e792 100644 --- a/modules/gdscript/tests/gdscript_test_runner.cpp +++ b/modules/gdscript/tests/gdscript_test_runner.cpp @@ -441,7 +441,7 @@ GDScriptTest::TestResult GDScriptTest::execute_test_code(bool p_is_generating) { result.output = get_text_for_status(result.status) + "\n"; const List &errors = parser.get_errors(); - for (auto *E = errors.front(); E; E = E->next()) { + for (const List::Element *E = errors.front(); E; E = E->next()) { result.output += E->get().message + "\n"; // TODO: line, column? break; // Only the first error since the following might be cascading. } @@ -460,7 +460,7 @@ GDScriptTest::TestResult GDScriptTest::execute_test_code(bool p_is_generating) { result.output = get_text_for_status(result.status) + "\n"; const List &errors = parser.get_errors(); - for (auto *E = errors.front(); E; E = E->next()) { + for (const List::Element *E = errors.front(); E; E = E->next()) { result.output += E->get().message + "\n"; // TODO: line, column? break; // Only the first error since the following might be cascading. } -- cgit v1.2.3