From d85a03b5610ed1dec8cdf64b4f17a511668bf383 Mon Sep 17 00:00:00 2001 From: Artur Wieczorek Date: Wed, 26 Dec 2018 10:02:10 +0100 Subject: [PATCH] Don't make unnecessary calls to the member function Use already obtained reference to the list of selected properties instead of retrieving the list by calling GetSelectedProperties() function. --- src/propgrid/propgrid.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/propgrid/propgrid.cpp b/src/propgrid/propgrid.cpp index 53b193aedd..a2e4ba2cab 100644 --- a/src/propgrid/propgrid.cpp +++ b/src/propgrid/propgrid.cpp @@ -841,7 +841,7 @@ bool wxPropertyGrid::AddToSelectionFromInputEvent( wxPGProperty* prop, { // Allow right-click for context menu without // disturbing the selection. - if ( GetSelectedProperties().size() <= 1 || + if ( selection.size() <= 1 || !alreadySelected ) return DoSelectAndEdit(prop, colIndex, selFlags); return true; @@ -870,7 +870,7 @@ bool wxPropertyGrid::AddToSelectionFromInputEvent( wxPGProperty* prop, { res = DoAddToSelection(prop, selFlags); } - else if ( GetSelectedProperties().size() > 1 ) + else if ( selection.size() > 1 ) { res = DoRemoveFromSelection(prop, selFlags); }