From 87a9deceb5e320ba5cb009d8fd17fa448d6fd2e9 Mon Sep 17 00:00:00 2001 From: Ryan McGeary Date: Sat, 4 Apr 2026 08:50:35 -0600 Subject: [PATCH] Fix BloomFilter::Native#include? when multiple keys are passed --- ext/cbloomfilter/cbloomfilter.c | 4 +--- spec/native_spec.rb | 1 + 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/ext/cbloomfilter/cbloomfilter.c b/ext/cbloomfilter/cbloomfilter.c index 316bc7e..e3ad6b1 100644 --- a/ext/cbloomfilter/cbloomfilter.c +++ b/ext/cbloomfilter/cbloomfilter.c @@ -341,10 +341,8 @@ static VALUE bf_include(int argc, VALUE* argv, VALUE self) { return Qfalse; /* i.e., it is a new entry ; escape the loop */ } } - - return Qtrue; } - + return Qtrue; } static VALUE bf_to_s(VALUE self) { diff --git a/spec/native_spec.rb b/spec/native_spec.rb index ba3653c..d993bf0 100644 --- a/spec/native_spec.rb +++ b/spec/native_spec.rb @@ -29,6 +29,7 @@ expect(bf.include?("test")).to be true expect(bf.include?("abcd")).to be false expect(bf.include?("test", "test1")).to be true + expect(bf.include?("test", "bogus")).to be false end it "should work with any object's to_s" do