Closed
Bug 571581
Opened 15 years ago
Closed 15 years ago
Propagate SQLITE_CONSTRAINT as an nsresult
Categories
(Core :: SQLite and Embedded Database Bindings, defect)
Core
SQLite and Embedded Database Bindings
Tracking
()
RESOLVED
FIXED
mozilla2.0b1
People
(Reporter: sdwilsh, Assigned: sdwilsh)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
|
854 bytes,
patch
|
asuth
:
review+
vlad
:
superreview+
|
Details | Diff | Splinter Review |
Say NS_ERROR_STORAGE_CONSTRAINT perhaps? Asking for sr because I guess this is an API change.
Attachment #450739 -
Flags: superreview?(vladimir)
Attachment #450739 -
Flags: review?(bugmail)
Comment on attachment 450739 [details] [diff] [review]
v1.0
I guess it's an API change, I mean it still returned an error before, now it's just different :)
Attachment #450739 -
Flags: superreview?(vladimir) → superreview+
|
||
Updated•15 years ago
|
Attachment #450739 -
Flags: review?(bugmail) → review+
|
Assignee | |
Updated•15 years ago
|
Whiteboard: [needs review asuth][needs sr vlad] → [can land]
|
|
Assignee | |
Comment 2•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Keywords: dev-doc-needed
Resolution: --- → FIXED
Whiteboard: [can land]
Target Milestone: --- → mozilla1.9.3a6
|
||
Comment 3•15 years ago
|
||
I'm inclined to just document this as a note in Firefox 4 for developers that says that the SQLITE_CONSTRAINT error is now returned as an NS_ERROR_STORAGE_CONSTRAINT instead of as NS_ERROR_FAILURE. Reasonable?
|
|
Assignee | |
Comment 4•15 years ago
|
||
(In reply to comment #3)
> I'm inclined to just document this as a note in Firefox 4 for developers that
> says that the SQLITE_CONSTRAINT error is now returned as an
> NS_ERROR_STORAGE_CONSTRAINT instead of as NS_ERROR_FAILURE. Reasonable?
yes
|
||
Updated•1 year ago
|
Product: Toolkit → Core
You need to log in
before you can comment on or make changes to this bug.
Description
•