search

aviggiano / redis-roaring

Roaring Bitmaps for Redis
MIT License
348 stars 56 forks source link

flip a bitmap by range #77

Closed jiangtao244 closed 4 years ago

jiangtao244 commented 4 years ago

fix this: https://github.com/aviggiano/redis-roaring/issues/73

jiangtao244 commented 4 years ago

Test added.

Valgrid find some memory leak, is this matter?

aviggiano commented 4 years ago

I believe this was already there (related to #2)

I'll merge this PR and create a separate issue to investigate the memory leaks.

jiangtao244 commented 4 years ago

@aviggiano ,
Sorry, I missed a test case when lastOffset < Max

Example:

R.SETINTARRAY src 1 3 5 R.BITOP NOT dst src 3 R.GETINTARRAY dst

Get: 0 2 5 Expect: 0 2

Need to cut the bits out of range.