Closed Bug 394241 Opened 12 years ago Closed 12 years ago
Add a close
Database() method to moz Storage
Given all the warnings about how the database can be corrupted if you open it from multiple threads, it's a little scary that there's no way to explicitly close the database. Since you're at the mercy of the semi-random nature of the garbage collector to get the database closed, this could be dangerous.
Assignee: nobody → comrade693+bmo
Status: NEW → ASSIGNED
Attachment #278893 - Flags: review?(sspitzer)
Target Milestone: --- → mozilla1.9 M8
Version: unspecified → Trunk
Comment on attachment 278893 [details] [diff] [review] v1.0 r=sspitzer
Attachment #278893 - Flags: review?(sspitzer) → review+
Checking in storage/public/mozIStorageConnection.idl; new revision: 1.12; previous revision: 1.11 Checking in storage/src/mozStorageConnection.cpp; new revision: 1.29; previous revision: 1.28 Checking in storage/test/unit/test_storage_connection.js; new revision: 1.6; previous revision: 1.5
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Now *that's* what I call service. Thanks! :)
Removing doc needed keyword in favor of a new bug that covers the fact that in general we actually need proper documentation for Storage; that bug (bug 399535) is set as blocked by this one.
You need to log in before you can comment on or make changes to this bug.