From b83380b2ba459736587d4f0583063b3aeed80317 Mon Sep 17 00:00:00 2001
From: Jeff Young <jeff@rokeby.ie>
Date: Thu, 8 Apr 2021 23:10:09 +0100
Subject: [PATCH] Performance: don't look up map entry twice.

Fixes https://gitlab.com/kicad/code/kicad/issues/8157
---
 common/gal/opengl/opengl_gal.cpp | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/common/gal/opengl/opengl_gal.cpp b/common/gal/opengl/opengl_gal.cpp
index e16f43e9d3..8949e2590d 100644
--- a/common/gal/opengl/opengl_gal.cpp
+++ b/common/gal/opengl/opengl_gal.cpp
@@ -1623,22 +1623,28 @@ void OPENGL_GAL::EndGroup()
 
 void OPENGL_GAL::DrawGroup( int aGroupNumber )
 {
-    if( m_groups[aGroupNumber] )
-        m_cachedManager->DrawItem( *m_groups[aGroupNumber] );
+    auto group = m_groups.find( aGroupNumber );
+
+    if( group != m_groups.end() )
+        m_cachedManager->DrawItem( *group->second );
 }
 
 
 void OPENGL_GAL::ChangeGroupColor( int aGroupNumber, const COLOR4D& aNewColor )
 {
-    if( m_groups[aGroupNumber] )
-        m_cachedManager->ChangeItemColor( *m_groups[aGroupNumber], aNewColor );
+    auto group = m_groups.find( aGroupNumber );
+
+    if( group != m_groups.end() )
+        m_cachedManager->ChangeItemColor( *group->second, aNewColor );
 }
 
 
 void OPENGL_GAL::ChangeGroupDepth( int aGroupNumber, int aDepth )
 {
-    if( m_groups[aGroupNumber] )
-        m_cachedManager->ChangeItemDepth( *m_groups[aGroupNumber], aDepth );
+    auto group = m_groups.find( aGroupNumber );
+
+    if( group != m_groups.end() )
+        m_cachedManager->ChangeItemDepth( *group->second, aDepth );
 }