From 052eacc3b9345ce68048e31657e518241c5b004d Mon Sep 17 00:00:00 2001
From: Lioncash
Date: Wed, 30 Sep 2015 11:51:06 -0400
Subject: symbols: Simplify GetSymbol

---
 src/common/symbols.cpp | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

(limited to 'src/common/symbols.cpp')

diff --git a/src/common/symbols.cpp b/src/common/symbols.cpp
index f23e51c9d..2115dc062 100644
--- a/src/common/symbols.cpp
+++ b/src/common/symbols.cpp
@@ -29,17 +29,14 @@ namespace Symbols
 
     TSymbol GetSymbol(u32 _address)
     {
-        TSymbolsMap::iterator foundSymbolItr;
-        TSymbol symbol;
+        const auto iter = g_symbols.find(_address);
 
-        foundSymbolItr = g_symbols.find(_address);
-        if (foundSymbolItr != g_symbols.end())
-        {
-            symbol = (*foundSymbolItr).second;
-        }
+        if (iter != g_symbols.end())
+            return iter->second;
 
-        return symbol;
+        return {};
     }
+
     const std::string GetName(u32 _address)
     {
         return GetSymbol(_address).name;
-- 
cgit v1.2.3


From 65d7a82db4d6b0236627baef1064a23695b12841 Mon Sep 17 00:00:00 2001
From: Lioncash
Date: Wed, 30 Sep 2015 11:54:44 -0400
Subject: symbols: Get rid of initial underscores in variable names

---
 src/common/symbols.cpp | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

(limited to 'src/common/symbols.cpp')

diff --git a/src/common/symbols.cpp b/src/common/symbols.cpp
index 2115dc062..0d7f0324a 100644
--- a/src/common/symbols.cpp
+++ b/src/common/symbols.cpp
@@ -8,28 +8,28 @@ TSymbolsMap g_symbols;
 
 namespace Symbols
 {
-    bool HasSymbol(u32 _address)
+    bool HasSymbol(u32 address)
     {
-        return g_symbols.find(_address) != g_symbols.end();
+        return g_symbols.find(address) != g_symbols.end();
     }
 
-    void Add(u32 _address, const std::string& _name, u32 _size, u32 _type)
+    void Add(u32 address, const std::string& name, u32 size, u32 type)
     {
-        if (!HasSymbol(_address))
+        if (!HasSymbol(address))
         {
             TSymbol symbol;
-            symbol.address = _address;
-            symbol.name = _name;
-            symbol.size = _size;
-            symbol.type = _type;
+            symbol.address = address;
+            symbol.name = name;
+            symbol.size = size;
+            symbol.type = type;
 
-            g_symbols.insert(TSymbolsPair(_address, symbol));
+            g_symbols.insert(TSymbolsPair(address, symbol));
         }
     }
 
-    TSymbol GetSymbol(u32 _address)
+    TSymbol GetSymbol(u32 address)
     {
-        const auto iter = g_symbols.find(_address);
+        const auto iter = g_symbols.find(address);
 
         if (iter != g_symbols.end())
             return iter->second;
@@ -37,14 +37,14 @@ namespace Symbols
         return {};
     }
 
-    const std::string GetName(u32 _address)
+    const std::string GetName(u32 address)
     {
-        return GetSymbol(_address).name;
+        return GetSymbol(address).name;
     }
 
-    void Remove(u32 _address)
+    void Remove(u32 address)
     {
-        g_symbols.erase(_address);
+        g_symbols.erase(address);
     }
 
     void Clear()
-- 
cgit v1.2.3


From 8cb2bc748d56cb0f0cb0f432e17fc6db73311696 Mon Sep 17 00:00:00 2001
From: Lioncash
Date: Wed, 30 Sep 2015 11:55:58 -0400
Subject: symbols: Replace an insert call with emplace

---
 src/common/symbols.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/common/symbols.cpp')

diff --git a/src/common/symbols.cpp b/src/common/symbols.cpp
index 0d7f0324a..db8340043 100644
--- a/src/common/symbols.cpp
+++ b/src/common/symbols.cpp
@@ -23,7 +23,7 @@ namespace Symbols
             symbol.size = size;
             symbol.type = type;
 
-            g_symbols.insert(TSymbolsPair(address, symbol));
+            g_symbols.emplace(address, symbol);
         }
     }
 
-- 
cgit v1.2.3