Merge remote-tracking branch 'gleenn/master' into release

This commit is contained in:
Jason Davies 2012-07-09 10:54:33 +01:00
commit 762d969a94
2 changed files with 11 additions and 5 deletions

View File

@ -42,7 +42,7 @@
};
BloomFilter.prototype.add = function(v) {
var l = this.locations(v),
var l = this.locations(v + ""),
i = -1,
k = this.k,
buckets = this.buckets;
@ -50,7 +50,7 @@
};
BloomFilter.prototype.test = function(v) {
var l = this.locations(v),
var l = this.locations(v + ""),
i = -1,
k = this.k,
b,

View File

@ -30,9 +30,9 @@ suite.addBatch({
},
"basic uint32": function() {
var f = new BloomFilter(1000, 4),
n1 = "\u100",
n2 = "\u101",
n3 = "\u103";
n1 = "\u0100",
n2 = "\u0101",
n3 = "\u0103";
f.add(n1);
assert.equal(f.test(n1), true);
assert.equal(f.test(n2), false);
@ -42,6 +42,12 @@ suite.addBatch({
var f = new BloomFilter(20, 10);
f.add("abc");
assert.equal(f.test("wtf"), false);
},
"works with integer types": function() {
var f = new BloomFilter(1000, 4);
f.add(1);
assert.equal(f.test(1), true);
assert.equal(f.test(2), false);
}
}
});