Closed Bug 462434 Opened 17 years ago Closed 17 years ago

Transaction semantics are bad

Categories

(Core :: SQLite and Embedded Database Bindings, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.1b2

People

(Reporter: sdwilsh, Assigned: sdwilsh)

Details

Attachments

(1 file)

How it currently works, we end up breaking assumptions that mozStorageTransaction holds, causing badness.
Attached patch v1.0Splinter Review
Attachment #345579 - Flags: review?(bugmail)
Comment on attachment 345579 [details] [diff] [review] v1.0 Looks good. mozStorageTransaction should no longer be misled about what is happening with transactions, and mozStorageConnection's understanding of whether one is active should be consistent with the underlying sqlite state.
Attachment #345579 - Flags: review?(bugmail) → review+
Whiteboard: [has patch][has review][can land]
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: [has patch][has review][can land]
Product: Toolkit → Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: