From 6b3165d3cca2ab44bd967b348c9d9d11fc86eedc Mon Sep 17 00:00:00 2001
From: Bartosz Taudul <wolf.pld@gmail.com>
Date: Mon, 20 Jan 2020 23:34:48 +0100
Subject: [PATCH] Perform map lookup in one place.

---
 server/TracyView.cpp | 28 +++++++++++++---------------
 1 file changed, 13 insertions(+), 15 deletions(-)

diff --git a/server/TracyView.cpp b/server/TracyView.cpp
index c7d3621a..1ee518ca 100644
--- a/server/TracyView.cpp
+++ b/server/TracyView.cpp
@@ -9175,41 +9175,39 @@ void View::DrawFindZone()
             }
 
             processed++;
-            FindZone::Group* group;
+            uint64_t gid = 0;
             switch( groupBy )
             {
             case FindZone::GroupBy::Thread:
-                group = &m_findZone.groups[ev.Thread()];
+                gid = ev.Thread();
                 break;
             case FindZone::GroupBy::UserText:
-                group = &m_findZone.groups[ev.Zone()->text.Active() ? ev.Zone()->text.Idx() : std::numeric_limits<uint64_t>::max()];
+                gid = ev.Zone()->text.Active() ? ev.Zone()->text.Idx() : std::numeric_limits<uint64_t>::max();
                 break;
             case FindZone::GroupBy::Callstack:
-                group = &m_findZone.groups[ev.Zone()->callstack.Val()];
+                gid = ev.Zone()->callstack.Val();
                 break;
             case FindZone::GroupBy::Parent:
             {
                 const auto parent = GetZoneParent( *ev.Zone(), m_worker.DecompressThread( ev.Thread() ) );
-                if( parent )
-                {
-                    group = &m_findZone.groups[uint64_t( uint16_t( parent->SrcLoc() ) )];
-                }
-                else
-                {
-                    group = &m_findZone.groups[0];
-                }
+                if( parent ) gid = uint64_t( uint16_t( parent->SrcLoc() ) );
                 break;
             }
             case FindZone::GroupBy::NoGrouping:
-                group = &m_findZone.groups[0];
                 break;
             default:
-                group = nullptr;
                 assert( false );
                 break;
             }
+            auto it = m_findZone.groups.find( gid );
+            if( it == m_findZone.groups.end() )
+            {
+                it = m_findZone.groups.emplace( gid, FindZone::Group {} ).first;
+                it->second.zones.reserve( 1024 );
+            }
+            FindZone::Group* group = &it->second;
             group->time += timespan;
-            group->zones.push_back( ev.Zone() );
+            group->zones.push_back_non_empty( ev.Zone() );
         }
         m_findZone.processed = processed;