Merge pull request #1928 from jaffee/op-apply-simplify

simply setting list of values with *N methods
This commit is contained in:
Matthew Jaffee 2019-04-05 16:39:55 -04:00 committed by GitHub
commit b263d8ccdf
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -3667,15 +3667,9 @@ func (op *op) apply(b *Bitmap) (changed bool) {
case opTypeRemove:
return b.remove(op.value)
case opTypeAddBatch:
for _, v := range op.values {
nc := b.DirectAdd(v)
changed = nc || changed
}
changed = b.DirectAddN(op.values...) > 0
case opTypeRemoveBatch:
for _, v := range op.values {
nc := b.remove(v)
changed = nc || changed
}
changed = b.DirectRemoveN(op.values...) > 0
default:
panic(fmt.Sprintf("invalid op type: %d", op.typ))
}