From cc5500f7de954e461c843237346527220bbce0ba Mon Sep 17 00:00:00 2001 From: MewPurPur Date: Sat, 8 Jul 2023 15:54:05 +0300 Subject: [PATCH] Fix erroneous pad_zeros warning --- core/string/ustring.cpp | 11 ++++++----- tests/core/string/test_string.h | 2 ++ 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/core/string/ustring.cpp b/core/string/ustring.cpp index 49c7ead4234..12e64237248 100644 --- a/core/string/ustring.cpp +++ b/core/string/ustring.cpp @@ -4281,12 +4281,13 @@ String String::pad_zeros(int p_digits) const { begin++; } - if (begin >= end) { - return s; - } - int zeros_to_add = p_digits - (end - begin); - return s.insert(begin, String("0").repeat(zeros_to_add)); + + if (zeros_to_add <= 0) { + return s; + } else { + return s.insert(begin, String("0").repeat(zeros_to_add)); + } } String String::trim_prefix(const String &p_prefix) const { diff --git a/tests/core/string/test_string.h b/tests/core/string/test_string.h index 5a2f8714305..93b237788db 100644 --- a/tests/core/string/test_string.h +++ b/tests/core/string/test_string.h @@ -1095,7 +1095,9 @@ TEST_CASE("[String] pad") { s = String("10.10"); CHECK(s.pad_decimals(4) == U"10.1000"); + CHECK(s.pad_decimals(1) == U"10.1"); CHECK(s.pad_zeros(4) == U"0010.10"); + CHECK(s.pad_zeros(1) == U"10.10"); } TEST_CASE("[String] is_subsequence_of") {