13 years ago
Created attachment 220672 [details] [diff] [review] Patch
Attachment #220672 - Flags: first-review?(darin)
Comment on attachment 220672 [details] [diff] [review] Patch Looks fine. My only comment is that it might make sense to document the initialization requirements of the mozStorageService implementation of mozIStorageService separately from the interface. Interfaces could be re-used in general, so documentation about a particular ContractID is better placed next to the declaration of the ContractID ;-) r=darin
Attachment #220672 - Flags: first-review?(darin) → first-review+
On branch and trunk.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
as expected, this change breaks BeOS builds due to lack of pthreads support in BeOS and lack of bthreads support in sqlite. Bezilla team has filed a bug with sqlite to document this lack. Is it possible to disable "Places" until this problem is resolved? --disable-places no longer seems to work.
Places is disabled by default, but mozStorage is a core part of the platform. The best thing to do would be to simply implement the bthreads threading backend for sqlite.
(the sqlite bug in question seems to be http://www.sqlite.org/cvstrac/tktview?tn=1791)
(In reply to comment #5) > Places is disabled by default, but mozStorage is a core part of the platform. > The best thing to do would be to simply implement the bthreads threading > backend for sqlite. > Because BeOS uses its own native threads (not pthreads), this is not really a workable solution. Instead, we're going to need to implement bthreads under sqlite but this isn't quite so simple a task. It would be great to find some other workaround in the meantime and any ideas would be appreciated.
You need to log in before you can comment on or make changes to this bug.