diff options
| author | Juan Linietsky <reduzio@gmail.com> | 2018-11-01 12:54:51 -0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-11-01 12:54:51 -0300 |
| commit | c8765b459add8524116b2fdca8e53e5943d6e0d6 (patch) | |
| tree | e049fe04040f38111f2af6a5271d73d6150ca41b | |
| parent | 18d06b971b7d04cc4dc93607dd90720f8e6c5624 (diff) | |
| parent | 62848f94eef289d951370b8cd071821b588b21ea (diff) | |
Merge pull request #23159 from DavidSichma/string-escape
Fix: String::format Variant to String conversion
| -rw-r--r-- | core/ustring.cpp | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/core/ustring.cpp b/core/ustring.cpp index 1195cd0719..2191bb5e23 100644 --- a/core/ustring.cpp +++ b/core/ustring.cpp @@ -2774,16 +2774,13 @@ String String::format(const Variant &values, String placeholder) const { if (value_arr.size() == 2) { Variant v_key = value_arr[0]; - String key; - - key = v_key.get_construct_string(); + String key = v_key; if (key.left(1) == "\"" && key.right(key.length() - 1) == "\"") { key = key.substr(1, key.length() - 2); } Variant v_val = value_arr[1]; - String val; - val = v_val.get_construct_string(); + String val = v_val; if (val.left(1) == "\"" && val.right(val.length() - 1) == "\"") { val = val.substr(1, val.length() - 2); @@ -2795,8 +2792,7 @@ String String::format(const Variant &values, String placeholder) const { } } else { //Array structure ["RobotGuy","Logis","rookie"] Variant v_val = values_arr[i]; - String val; - val = v_val.get_construct_string(); + String val = v_val; if (val.left(1) == "\"" && val.right(val.length() - 1) == "\"") { val = val.substr(1, val.length() - 2); @@ -2815,8 +2811,8 @@ String String::format(const Variant &values, String placeholder) const { d.get_key_list(&keys); for (List<Variant>::Element *E = keys.front(); E; E = E->next()) { - String key = E->get().get_construct_string(); - String val = d[E->get()].get_construct_string(); + String key = E->get(); + String val = d[E->get()]; if (key.left(1) == "\"" && key.right(key.length() - 1) == "\"") { key = key.substr(1, key.length() - 2); |