igrigorik-bloomfilter 0.1.1 → 0.1.2

data/ext/sbloomfilter.c

@@ -80,42 +80,27 @@ static VALUE bf_s_new(int argc, VALUE *argv, VALUE self) {
 
     obj = Data_Make_Struct(self, struct BloomFilter, NULL, bits_free, bf);
 
-    if (argc == 5) {
-        arg1 = argv[0];
-        arg2 = argv[1];
-        arg3 = argv[2];
-        arg4 = argv[3];
-        arg5 = argv[4];
-    } else if (argc == 4) {
-        arg1 = argv[0];
-        arg2 = argv[1];
-        arg3 = argv[2];
-        arg4 = argv[3];
-        arg5 = 0;
-    } else if (argc == 3) {
-        arg1 = argv[0];
-        arg2 = argv[1];
-        arg3 = argv[2];
-        arg4 = INT2FIX(1);
-        arg5 = 0;
-    } else if (argc == 2) {
-        arg1 = argv[0];
-        arg2 = argv[1];
-        arg3 = INT2FIX(0);
-        arg4 = INT2FIX(1);
-        arg5 = 0;
-    } else if (argc == 1) {
-        arg1 = argv[0];
-        arg2 = INT2FIX(4);
-        arg3 = INT2FIX(0);
-        arg4 = INT2FIX(1);
-        arg5 = 0;
-    } else { /* default = Fugou approach :-) */
-        arg1 = INT2FIX(100000000);
-        arg2 = INT2FIX(4);
-        arg3 = INT2FIX(0);
-        arg4 = INT2FIX(1);
-        arg5 = 0;
+    /* default = Fugou approach :-) */
+    arg1 = INT2FIX(100000000);
+    arg2 = INT2FIX(4);
+    arg3 = INT2FIX(0);
+    arg4 = INT2FIX(1);
+    arg5 = INT2FIX(0);
+
+    switch (argc) {
+        case 5:
+	    if (argv[4] == Qtrue) {
+		    arg5 = INT2FIX(1);
+	    }
+        case 4:
+	    arg4 = argv[3];
+        case 3:
+	    arg3 = argv[2];
+        case 2:
+	    arg2 = argv[1];
+        case 1:
+	    arg1 = argv[0];
+	    break;
     }
 
     m = FIX2INT(arg1);

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: igrigorik-bloomfilter
 version: !ruby/object:Gem::Version 
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors: 
 - Ilya Grigorik