Update reverse_purge_hash_map copy constructor

jmalkin · jmalkin · commit 5a3cd54b9bdc · 2022-09-27T16:04:25.000-07:00
I think reverse_purge_hash_map copy constructor should do this here, the original result is not being able to copy all the values
diff --git a/fi/include/reverse_purge_hash_map_impl.hpp b/fi/include/reverse_purge_hash_map_impl.hpp
@@ -71,8 +71,8 @@ states_(nullptr)
       if (other.states_[i] > 0) {
         new (&keys_[i]) K(other.keys_[i]);
         values_[i] = other.values_[i];
+         if (--num == 0) break;
       }
-      if (--num == 0) break;
     }
   }
   std::copy(other.states_, other.states_ + size, states_);

Original file line number	Diff line number	Diff line change
`@@ -71,8 +71,8 @@ states_(nullptr)`
`71`	`71`	`if (other.states_[i] > 0) {`
`72`	`72`	`new (&keys_[i]) K(other.keys_[i]);`
`73`	`73`	`values_[i] = other.values_[i];`
	`74`	`+ if (--num == 0) break;`
`74`	`75`	`}`
`75`		`- if (--num == 0) break;`
`76`	`76`	`}`
`77`	`77`	`}`
`78`	`78`	`std::copy(other.states_, other.states_ + size, states_);`