From bbce79d7b3e9d7cff25ddf98a432ab3bc75b9ced Mon Sep 17 00:00:00 2001
From: PKEuS <philipp.kloke@web.de>
Date: Tue, 28 Aug 2012 12:57:11 +0200
Subject: [PATCH] Small refactorization: Handle std::map more efficient in
 setVarIdStructMembers()

---
 lib/tokenize.cpp | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

diff --git a/lib/tokenize.cpp b/lib/tokenize.cpp
index 29cf6d099..bd69a3f43 100644
--- a/lib/tokenize.cpp
+++ b/lib/tokenize.cpp
@@ -2539,23 +2539,12 @@ static void setVarIdStructMembers(Token **tok1,
         if (TemplateSimplifier::templateParameters(tok->next()) > 0)
             break;
 
-        std::map<unsigned int, std::map<std::string,unsigned int> >::iterator structIterator;
-        structIterator = structMembers->find(struct_varid);
-        if (structIterator == structMembers->end()) {
-            std::map<std::string,unsigned int> members;
-            members[tok->str()] = ++ (*_varId);
-            (*structMembers)[struct_varid] = members;
+        std::map<std::string,unsigned int>& members = (*structMembers)[struct_varid];
+        if (members.empty() || members.find(tok->str()) == members.end()) {
+            members[tok->str()] = ++(*_varId);
             tok->varId(*_varId);
         } else {
-            std::map<std::string,unsigned int> &members = structIterator->second;
-            std::map<std::string,unsigned int>::const_iterator memberIterator;
-            memberIterator = members.find(tok->str());
-            if (memberIterator == members.end()) {
-                members[tok->str()] = ++(*_varId);
-                tok->varId(*_varId);
-            } else {
-                tok->varId(memberIterator->second);
-            }
+            tok->varId(members[tok->str()]);
         }
     }
     if (tok)