3 files changed, 26 insertions, 14 deletions
diff --git a/src/gui/itempopup.cpp b/src/gui/itempopup.cpp
index 3b3dfcd3..966e2c92 100644
--- a/src/gui/itempopup.cpp
+++ b/src/gui/itempopup.cpp
@@ -195,13 +195,7 @@ void ItemPopup::setItem(const ItemInfo &item, bool showImage)
     mItemName->setPosition(getPadding() + space, getPadding());
 
     mItemDesc->setTextWrapped(item.getDescription(), ITEMPOPUP_WRAP_WIDTH);
-    {
-        const std::vector<std::string> &effect = item.getEffect();
-        std::string temp = "";
-        for (const auto &it : effect)
-            temp += temp.empty() ? it : "\n" + it;
-        mItemEffect->setTextWrapped(temp, ITEMPOPUP_WRAP_WIDTH);
-    }
+    mItemEffect->setTextWrapped(join(item.getEffect(), "\n"), ITEMPOPUP_WRAP_WIDTH);
     mItemWeight->setTextWrapped(strprintf(_("Weight: %s"),
                                 Units::formatWeight(item.getWeight()).c_str()),
                                 ITEMPOPUP_WRAP_WIDTH);
diff --git a/src/utils/stringutils.cpp b/src/utils/stringutils.cpp
index 5083770c..57138005 100644
--- a/src/utils/stringutils.cpp
+++ b/src/utils/stringutils.cpp
@@ -85,7 +85,7 @@ const char *ipToString(int address)
 std::string strprintf(char const *format, ...)
 {
     char buf[256];
-    va_list(args);
+    va_list args;
     va_start(args, format);
     int nb = vsnprintf(buf, 256, format, args);
     va_end(args);
@@ -191,7 +191,7 @@ bool getBoolFromString(const std::string &text, bool def)
         return def;
 }
 
-std::string autocomplete(std::vector<std::string> &candidates,
+std::string autocomplete(const std::vector<std::string> &candidates,
                          std::string base)
 {
     auto i = candidates.begin();
@@ -232,7 +232,7 @@ std::string normalize(const std::string &name)
     return toLower(trim(normalized));
 }
 
-std::string removeTrailingSymbol(const std::string& s, const char c)
+std::string removeTrailingSymbol(const std::string &s, const char c)
 {
     // Remove the trailing symblol at the end of the string
     if (!s.empty() && s.at(s.size() - 1) == c)
@@ -240,7 +240,7 @@ std::string removeTrailingSymbol(const std::string& s, const char c)
     return std::string(s);
 }
 
-std::string getHostNameFromURL(const std::string& url)
+std::string getHostNameFromURL(const std::string &url)
 {
     std::string myHostName;
 
@@ -276,3 +276,15 @@ std::string getHostNameFromURL(const std::string& url)
 
     return myHostName;
 }
+
+std::string join(const std::vector<std::string> &strings, const char *separator)
+{
+    std::string result;
+    if (auto i = strings.begin(), e = strings.end(); i != e)
+    {
+        result += *i++;
+        for (; i != e; ++i)
+            result.append(separator).append(*i);
+    }
+    return result;
+}
diff --git a/src/utils/stringutils.h b/src/utils/stringutils.h
index b83bc580..cd80b60f 100644
--- a/src/utils/stringutils.h
+++ b/src/utils/stringutils.h
@@ -141,7 +141,7 @@ bool getBoolFromString(const std::string &text, bool def = false);
 /**
  * Returns the most approaching string of base from candidates.
  */
-std::string autocomplete(std::vector<std::string> &candidates,
+std::string autocomplete(const std::vector<std::string> &candidates,
                          std::string base);
 
 /**
@@ -152,12 +152,18 @@ std::string normalize(const std::string &name);
 /**
  * Remove a potential trailing symbol from a string.
  */
-std::string removeTrailingSymbol(const std::string& s, const char c);
+std::string removeTrailingSymbol(const std::string &s, const char c);
 
 /**
  * Gets the hostname out of the given URL.
  * i.e.: http://www.manasource.org:9601 -> www.manasource.org
  */
-std::string getHostNameFromURL(const std::string& url);
+std::string getHostNameFromURL(const std::string &url);
+
+/**
+ * Joins a vector of strings into one string, separated by the given
+ * separator.
+ */
+std::string join(const std::vector<std::string> &strings, const char *separator);
 
 #endif // UTILS_STRINGUTILS_H