Some slight logic cleanups in the shortcut classes, as well as some

style cleanups.

TODO: Either abstract out all of the shared information between these
two classes to a parent class, or combine the two if it can be done
without doing it clumsily.

Signed-off-by: Tametomo <irarice@gmail.com>

1 files changed, 1 insertions, 8 deletions

diff --git a/src/emoteshortcut.cpp b/src/emoteshortcut.cpp
index 80376983..50879e4f 100644
--- a/src/emoteshortcut.cpp
+++ b/src/emoteshortcut.cpp
@@ -31,10 +31,6 @@ EmoteShortcut *emoteShortcut;
 EmoteShortcut::EmoteShortcut():
     mEmoteSelected(0)
 {
-    for (int i = 0; i < SHORTCUT_EMOTES; i++)
-    {
-        mEmotes[i] = i + 1;
-    }
     load();
 }
 
@@ -49,10 +45,7 @@ void EmoteShortcut::load()
     {
         int emoteId = (int) config.getValue("emoteshortcut" + toString(i), i + 1);
 
-        if (emoteId)
-        {
-            mEmotes[i] = emoteId;
-        }
+        mEmotes[i] = emoteId;
     }
 }