Skip to content

Commit

Permalink
Merge pull request #68 from nodec-project/feature/fix-observers
Browse files Browse the repository at this point in the history
fix: Fix issue that calls the removed observer
  • Loading branch information
ContentsViewer authored Oct 31, 2024
2 parents abaa1f5 + b866ede commit 33d8dc5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion include/nodec/observers.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -56,12 +56,13 @@ class Observers {

// Remove all empty slots.
std::size_t sz = 0;
for (std::size_t i = 0; i < n; ++i) {
for (std::size_t i = 0; i < observers_.size(); ++i) {
if (observers_[i] == nullptr) continue;
observers_[sz] = observers_[i];
observer_indices_[observers_[i]] = sz;
++sz;
}
observers_.resize(sz);
}

std::size_t size() const {
Expand Down

0 comments on commit 33d8dc5

Please sign in to comment.