From bd085e0a21b938708cf0c5b5477ba4cd399f9f71 Mon Sep 17 00:00:00 2001 From: Matt Jaffee Date: Mon, 1 Apr 2019 20:23:34 -0500 Subject: [PATCH] simply setting list of values with *N methods --- roaring/roaring.go | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/roaring/roaring.go b/roaring/roaring.go index fa2d09bcb..a8ccd81e2 100644 --- a/roaring/roaring.go +++ b/roaring/roaring.go @@ -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)) }