Closed Bug 518440 Opened 11 years ago Closed 11 years ago

Deprecate mozIStorageStatementWrapper & mozIStorageStatement.step() in IDL

Categories

(Toolkit :: Storage, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9.3a1

People

(Reporter: zpao, Assigned: zpao)

References

Details

Attachments

(1 file)

Need to make it known in the IDL that step() is deprecated in favor of executeStep()
Attached patch Patch v0.1Splinter Review
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
OS: Mac OS X → All
Hardware: x86 → All
Summary: Deprecate mozIStorageStatement.step() in IDL → Deprecate mozIStorageStatementWrapper & mozIStorageStatement.step() in IDL
Attachment #402627 - Flags: review?(sdwilsh) → review+
Comment on attachment 402627 [details] [diff] [review]
Patch v0.1

r=sdwilsh
Pushed http://hg.mozilla.org/mozilla-central/rev/5e6a413226d2
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
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 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?
Depends on: 589032
You need to log in before you can comment on or make changes to this bug.