[std-map] do not use operator[], but emplace() to deal with non-default-constructible case

ManifoldFR · ManifoldFR · commit a91e57cb48e1 · 2024-09-17T18:08:52.000+02:00
diff --git a/include/eigenpy/std-map.hpp b/include/eigenpy/std-map.hpp
@@ -142,7 +142,7 @@ struct dict_to_map {
         bp::throw_error_already_set();
       }
       typename Container::mapped_type val = valproxy();
-      map[key] = val;
+      map.emplace(key, val);
     }
 
     // remember the location for later

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ struct dict_to_map {`
`142`	`142`	`bp::throw_error_already_set();`
`143`	`143`	`}`
`144`	`144`	`typename Container::mapped_type val = valproxy();`
`145`		`- map[key] = val;`
	`145`	`+ map.emplace(key, val);`
`146`	`146`	`}`
`147`	`147`
`148`	`148`	`// remember the location for later`