From 808ee11bfc250f7d2cef8377c1f9fc474a51443c Mon Sep 17 00:00:00 2001
From: Ilya Grigorik <ilya@igvita.com>
Date: Fri, 16 Mar 2012 18:27:47 -0700
Subject: [PATCH] remove num_set, replace with "set_bits"

---
 ext/cbloomfilter/cbloomfilter.c | 15 ++-------------
 lib/bloomfilter/native.rb       |  4 ++--
 spec/native_spec.rb             |  6 ++++--
 3 files changed, 8 insertions(+), 17 deletions(-)

diff --git a/ext/cbloomfilter/cbloomfilter.c b/ext/cbloomfilter/cbloomfilter.c
index dfdeb68..750f7da 100644
--- a/ext/cbloomfilter/cbloomfilter.c
+++ b/ext/cbloomfilter/cbloomfilter.c
@@ -19,7 +19,6 @@ struct BloomFilter {
     int k; /* # of hash functions */
     int s; /* # seed of hash functions */
     int r; /* # raise on bucket overflow? */
-    int num_set; /* # of set bits */
     unsigned char *ptr; /* bits data */
     int bytes; /* size of byte data */
 };
@@ -127,7 +126,6 @@ static VALUE bf_s_new(int argc, VALUE *argv, VALUE self) {
     bf->k = k;
     bf->s = s;
     bf->r = r;
-    bf->num_set = 0;
 
     bf->bytes = ((m * b) + 15) / 8;
     bf->ptr = ALLOC_N(unsigned char, bf->bytes);
@@ -170,13 +168,7 @@ static VALUE bf_r(VALUE self) {
     return bf->r == 0 ? Qfalse : Qtrue;
 }
 
-static VALUE bf_num_set(VALUE self) {
-    struct BloomFilter *bf;
-    Data_Get_Struct(self, struct BloomFilter, bf);
-    return INT2FIX(bf->num_set);
-}
-
-static VALUE bf_num_bit(VALUE self){
+static VALUE bf_set_bits(VALUE self){
     struct BloomFilter *bf;
     int i,j,count = 0;
     Data_Get_Struct(self, struct BloomFilter, bf);
@@ -215,7 +207,6 @@ static VALUE bf_insert(VALUE self, VALUE key) {
         bucket_set(bf, index);
     }
 
-    bf->num_set += 1;
     return Qnil;
 }
 
@@ -257,7 +248,6 @@ static VALUE bf_delete(VALUE self, VALUE key) {
         bucket_unset(bf, index);
     }
 
-    bf->num_set += 1;
     return Qnil;
 }
 
@@ -352,8 +342,7 @@ void Init_cbloomfilter(void) {
     rb_define_method(cBloomFilter, "k", bf_k, 0);
     rb_define_method(cBloomFilter, "b", bf_b, 0);
     rb_define_method(cBloomFilter, "r", bf_r, 0);
-    rb_define_method(cBloomFilter, "num_set", bf_num_set, 0);
-    rb_define_method(cBloomFilter, "num_bit", bf_num_bit, 0);
+    rb_define_method(cBloomFilter, "set_bits", bf_set_bits, 0);
     rb_define_method(cBloomFilter, "insert", bf_insert, 1);
     rb_define_method(cBloomFilter, "delete", bf_delete, 1);
     rb_define_method(cBloomFilter, "include?", bf_include, -1);
diff --git a/lib/bloomfilter/native.rb b/lib/bloomfilter/native.rb
index d78a6dd..0eb9184 100644
--- a/lib/bloomfilter/native.rb
+++ b/lib/bloomfilter/native.rb
@@ -37,8 +37,8 @@ def size; @bf.num_set; end
     def merge!(o); @bf.merge!(o.bf); end
 
     # Returns the number of bits that are set to 1 in the filter.
-    def bits_count
-      @bf.num_bit
+    def set_bits
+      @bf.set_bits
     end
 
     def bitmap
diff --git a/spec/native_spec.rb b/spec/native_spec.rb
index 645c10b..f219d02 100644
--- a/spec/native_spec.rb
+++ b/spec/native_spec.rb
@@ -47,11 +47,13 @@
     it "should return the number of bits set to 1" do
       bf = Native.new(:hashes => 4)
       bf.insert("test")
-      bf.bits_count.should == 4
+      bf.set_bits.should == 4
+      bf.delete("test")
+      bf.set_bits.should == 0
 
       bf = Native.new(:hashes => 1)
       bf.insert("test")
-      bf.bits_count.should == 1
+      bf.set_bits.should == 1
     end
   end