From ea214727a54f16821474fec977bed7aa29a5a1db Mon Sep 17 00:00:00 2001
From: Mark Olesen <Mark.Olesen@esi-group.com>
Date: Mon, 11 Nov 2019 17:07:30 +0100
Subject: [PATCH] ENH: 'set' instead of 'insert' std::initializer_list to
 HashTable

- Duplicate entries are handled by overwriting, which corresponds more
  closely to the notion of assignment
---
 src/OpenFOAM/containers/HashTables/HashTable/HashTable.C | 4 ++--
 src/OpenFOAM/containers/HashTables/HashTable/HashTable.H | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/OpenFOAM/containers/HashTables/HashTable/HashTable.C b/src/OpenFOAM/containers/HashTables/HashTable/HashTable.C
index 17d0121bfc..694a8e9442 100644
--- a/src/OpenFOAM/containers/HashTables/HashTable/HashTable.C
+++ b/src/OpenFOAM/containers/HashTables/HashTable/HashTable.C
@@ -97,7 +97,7 @@ Foam::HashTable<T, Key, Hash>::HashTable
 {
     for (const auto& keyval : list)
     {
-        insert(keyval.first, keyval.second);
+        set(keyval.first, keyval.second);
     }
 }
 
@@ -809,7 +809,7 @@ void Foam::HashTable<T, Key, Hash>::operator=
 
     for (const auto& keyval : rhs)
     {
-        insert(keyval.first, keyval.second);
+        set(keyval.first, keyval.second);
     }
 }
 
diff --git a/src/OpenFOAM/containers/HashTables/HashTable/HashTable.H b/src/OpenFOAM/containers/HashTables/HashTable/HashTable.H
index 85b98c4410..9fc07ea5e8 100644
--- a/src/OpenFOAM/containers/HashTables/HashTable/HashTable.H
+++ b/src/OpenFOAM/containers/HashTables/HashTable/HashTable.H
@@ -222,6 +222,7 @@ public:
         HashTable(this_type&& rhs);
 
         //- Construct from an initializer list
+        //  Duplicate entries are handled by overwriting
         HashTable(std::initializer_list<std::pair<Key, T>> list);
 
 
@@ -518,6 +519,7 @@ public:
         void operator=(const this_type& rhs);
 
         //- Copy assign from an initializer list
+        //  Duplicate entries are handled by overwriting
         void operator=(std::initializer_list<std::pair<Key, T>> rhs);
 
         //- Move assign