optimise find/insert pattern

if we're doing a find/insert on a set or a map, it is better to just do
a conditional insert/emplace operation than triggering two lookups.

Change-Id: I80da5097f5a89fe30fa348ce5b6e747c34287a8d
Reviewed-on: https://gerrit.libreoffice.org/70937
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 1 insertions, 2 deletions

diff --git a/cui/source/dialogs/FontFeaturesDialog.cxx b/cui/source/dialogs/FontFeaturesDialog.cxx
index 27b7b18620d2..d4facba2bf7f 100644
--- a/cui/source/dialogs/FontFeaturesDialog.cxx
+++ b/cui/source/dialogs/FontFeaturesDialog.cxx
@@ -71,9 +71,8 @@ void FontFeaturesDialog::initialize()
     for (vcl::font::Feature const& rFontFeature : rFontFeatures)
     {
         sal_uInt32 nFontFeatureCode = rFontFeature.m_aID.m_aFeatureCode;
-        if (aDoneFeatures.find(nFontFeatureCode) != aDoneFeatures.end())
+        if (!aDoneFeatures.insert(nFontFeatureCode).second)
             continue;
-        aDoneFeatures.insert(nFontFeatureCode);
         rFilteredFontFeatures.push_back(rFontFeature);
     }