use structured binding instead of std::tie

To make sure vertices can be some class that is not default constructable.
reticula-network · Nov 19, 2023 · 88f8eb7 · 88f8eb7
1 parent c208666
commit 88f8eb7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/static_edges.tpp b/src/static_edges.tpp
@@ -147,7 +147,9 @@ namespace reticula {
 
   template <network_vertex VertexType>
   undirected_edge<VertexType>::undirected_edge(VertexType v1, VertexType v2) {
-    std::tie(_v1, _v2) = std::minmax(v1, v2);
+    auto [i, j] = std::minmax(v1, v2);
+    _v1 = i;
+    _v2 = j;
   }
 
   template <network_vertex VertexType>

diff --git a/src/temporal_edges.tpp b/src/temporal_edges.tpp
@@ -345,7 +345,9 @@ namespace reticula {
   template <network_vertex VertexType, typename TimeType>
   undirected_temporal_edge<VertexType, TimeType>::undirected_temporal_edge(
       VertexType v1, VertexType v2, TimeType time) : _time(time) {
-    std::tie(_v1, _v2) = std::minmax(v1, v2);
+    auto [i, j] = std::minmax(v1, v2);
+    _v1 = i;
+    _v2 = j;
   }
 
   template <network_vertex VertexType, typename TimeType>