Closed
Bug 518440
Opened 16 years ago
Closed 16 years ago
Deprecate mozIStorageStatementWrapper & mozIStorageStatement.step() in IDL
Categories
(Core :: SQLite and Embedded Database Bindings, defect)
Core
SQLite and Embedded Database Bindings
Tracking
()
RESOLVED
FIXED
mozilla1.9.3a1
People
(Reporter: zpao, Assigned: zpao)
References
Details
Attachments
(1 file)
|
2.92 KB,
patch
|
sdwilsh
:
review+
|
Details | Diff | Splinter Review |
Need to make it known in the IDL that step() is deprecated in favor of executeStep()
|
Assignee | |
Comment 1•16 years ago
|
||
Looking around at other IDLs, it seems like just @deprecated is used instead of [deprecated]. Maybe we want to take this a step further and manually print deprecated warnings?
Assignee: nobody → paul
|
|
Assignee | |
Updated•16 years ago
|
Attachment #402627 -
Flags: review?(sdwilsh)
|
|
Assignee | |
Updated•16 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
Summary: Deprecate mozIStorageStatement.step() in IDL → Deprecate mozIStorageStatementWrapper & mozIStorageStatement.step() in IDL
|
||
Updated•16 years ago
|
Attachment #402627 -
Flags: review?(sdwilsh) → review+
|
|
||
Comment 2•16 years ago
|
||
Comment on attachment 402627 [details] [diff] [review]
Patch v0.1
r=sdwilsh
|
|
Assignee | |
Comment 3•16 years ago
|
||
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
|
|
Assignee | |
Comment 4•16 years ago
|
||
Comment on attachment 402627 [details] [diff] [review]
Patch v0.1
Shawn wants this on 1.9.2 to officially kick off the deprecation (and the comments say as of 1.9.2)
Attachment #402627 -
Flags: approval1.9.2?
|
||
Comment 5•16 years ago
|
||
Comment on attachment 402627 [details] [diff] [review]
Patch v0.1
too late for this on 1.9.2, I think
Attachment #402627 -
Flags: approval1.9.2?
|
||
Updated•1 year ago
|
Product: Toolkit → Core
You need to log in
before you can comment on or make changes to this bug.
Description
•