search

Level / memdown

In-memory abstract-leveldown store for Node.js and browsers.
MIT License
287 stars 37 forks source link

Implement destroy(), use global db store #23

Closed nolanlawson closed 9 years ago

nolanlawson commented 10 years ago

Fixes #22.

There's now a global store that keeps track of all memdown databases, keyed by location. so if you do:

var db1 = new MemDOWN('foo')
var db2 = new MemDOWN('foo')

then both db1 and db2 will contain the same data.

I also added a destroy test here, rather than in abstract-leveldown, because memdown isn't even passing the latest abstract-leveldown tests. If you want to see these tests pass at 100%, we're going to need to peg abstract-leveldown to v0.12.3 due to the snapshot test.

nolanlawson commented 10 years ago

yeah yeah, tests are broken. I say we peg abstract-leveldown

nolanlawson commented 10 years ago

rebased, mebbe it'll be green now

nolanlawson commented 10 years ago

updated

nolanlawson commented 10 years ago

Missed some more semicolons. When in Rome...

calvinmetcalf commented 10 years ago

NO SEMICOLONS WE ARE IN ROME AND THEY SHOOT SEMICOLONS IN ROME

rvagg commented 10 years ago

fine by me with minor style tweaks

nolanlawson commented 9 years ago

sorry for the delay. de-Crockfordized the code and ready to push now

nolanlawson commented 9 years ago

fe7c13ee722d6bd7b83f17598ea06945c2fe0ebb