Status

()

RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: brettw, Assigned: brettw)

Tracking

({fixed1.8.1})

1.8 Branch
fixed1.8.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

13 years ago
 
(Assignee)

Comment 1

13 years ago
Created attachment 220672 [details] [diff] [review]
Patch
Attachment #220672 - Flags: first-review?(darin)

Comment 2

13 years ago
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+
(Assignee)

Comment 3

13 years ago
On branch and trunk.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED

Comment 4

13 years ago
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.

Comment 7

13 years ago
(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.