rework sort order reference (#7)

markisus · web-flow · commit c55f34ade3cf · 2022-02-13T16:25:51.000-05:00
diff --git a/vapid/soa.h b/vapid/soa.h
@@ -156,7 +156,6 @@ namespace vapid {
         template <typename... Xs>
         void insert(Xs... xs) {
             insert_impl(std::index_sequence_for<Ts...>{}, std::forward_as_tuple(xs...));
-            sort_order_reference_.push_back(size() - 1);
         }
 
         auto operator[](size_t idx) const {
@@ -277,6 +276,7 @@ namespace vapid {
         }
 
         void reset_sort_reference() {
+            sort_order_reference_.resize(size());
             for (size_t i = 0; i < size(); ++i) {
                 sort_order_reference_[i] = i;
             }

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,6 @@ namespace vapid {`
`156`	`156`	`template <typename... Xs>`
`157`	`157`	`void insert(Xs... xs) {`
`158`	`158`	`insert_impl(std::index_sequence_for<Ts...>{}, std::forward_as_tuple(xs...));`
`159`		`- sort_order_reference_.push_back(size() - 1);`
`160`	`159`	`}`
`161`	`160`
`162`	`161`	`auto operator[](size_t idx) const {`
`@@ -277,6 +276,7 @@ namespace vapid {`
`277`	`276`	`}`
`278`	`277`
`279`	`278`	`void reset_sort_reference() {`
	`279`	`+ sort_order_reference_.resize(size());`
`280`	`280`	`for (size_t i = 0; i < size(); ++i) {`
`281`	`281`	`sort_order_reference_[i] = i;`
`282`	`282`	`}`