Size() was broken for Union()

gaissmai / bart

The Balanced Routing Table is an adaptation of D. Knuth's ART algorithm combined with popcount level compression and backtracking. It is somewhat slower than ART, but requires considerably less memory.

MIT License

33 stars 3 forks source link

Size() was broken for Union() #44

Closed gaissmai closed 5 months ago

gaissmai commented 5 months ago

It makes the algorithm for union unnecessarily complex to keep this up to date as a struct member, so I decided to use the calculation.

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9075245115

Details

23 of 23 (100.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 95.177%

Totals
Change from base Build 9070347900:	0.2%
Covered Lines:	1046
Relevant Lines:	1099

gaissmai / bart

Size() was broken for Union() #44

Pull Request Test Coverage Report for Build 9075245115

Details

💛 - Coveralls