count_ones() gives wrong result for BitSlice about bitvec HOT 3 CLOSED

Hi! Thanks so much for your report. The bug here is pretty embarrassing in how it got overlooked. The diff here is just inserting the correct .filter call in the sole-element case; I'd managed to forget to do that in both count_ones and count_zeros, and never noticed.

I've published 0.10.1 to https://crates.io/crates/bitvec; you should be able to get the fix with a cargo update command.

from bitvec.

@geq1t, I've been looking through the git history of this bug and I want to take a minute to say how extremely thankful I am not only for opening this issue but for the information you provided. The test case you gave reproduced, and your sentence,

Interestingly, it does work across when operating across byte boundaries.

Told me exactly what the error was while I was reading the email, before I'd even opened my editor. This was a great report; thanks for using bitvec and I hope it's working well for you!

from bitvec.

Thanks for fixing this so quickly! I'm glad the bug report was useful. I'll give it a try again next week, but other duties call right now.

from bitvec.