aboutsummaryrefslogtreecommitdiff
path: root/common
diff options
context:
space:
mode:
authorJohannes Schickel2011-10-23 20:02:35 +0200
committerJohannes Schickel2011-10-23 20:04:03 +0200
commit3a196478c69b635c7c1c2d121b30d9710f8bd1e5 (patch)
tree2fdf78e3a0b0756279164b9f7abc683c13cab619 /common
parenta41d72a44a660c72fdadbc3a8ef580e5e03cb890 (diff)
downloadscummvm-rg350-3a196478c69b635c7c1c2d121b30d9710f8bd1e5.tar.gz
scummvm-rg350-3a196478c69b635c7c1c2d121b30d9710f8bd1e5.tar.bz2
scummvm-rg350-3a196478c69b635c7c1c2d121b30d9710f8bd1e5.zip
ALL: Reduce assignment of "" to Common::String.
When clearing an existant object clear() should be used. When constructing objects (or using default values for parameters) the constructor of String without any argument should be used. This changes only a few instances I noticed while looking over some recent commit logs.
Diffstat (limited to 'common')
-rw-r--r--common/util.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/common/util.cpp b/common/util.cpp
index bd2ebc4937..1f1b8a1440 100644
--- a/common/util.cpp
+++ b/common/util.cpp
@@ -381,7 +381,7 @@ String parseGameGUIOptions(const String &str) {
}
const String getGameGUIOptionsDescription(const String &options) {
- String res = "";
+ String res;
for (int i = 0; g_gameOptions[i].desc; i++)
if (options.contains(g_gameOptions[i].option[0]))