trrespass icon indicating copy to clipboard operation
trrespass copied to clipboard

"verify_sets" does not remove conflicting elements

Open its-luca opened this issue 3 years ago • 0 comments

In the verify_sets(std::vector<set_t>& sets, uint64_t threshold, size_t rounds) functions the outermost loop should be for (auto& s: sets) {...} instead of or (auto s: sets) {...}. Otherwise the s.erase(...) call inside the for loop will not modify the set_t elements contained in the sets argument, i.e. the conflicting elements are not removed.

its-luca avatar Jan 12 '22 15:01 its-luca