Closed Bug 758343 Opened 13 years ago Closed 6 years ago

Move Storage asyncClose spinning earlier, before exit(0)

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED INACTIVE

People

(Reporter: mak, Unassigned)

References

Details

(In reply to Rafael Ávila de Espíndola (:espindola) from comment #26)
> Your proposal, if I understand it correctly, is to change the semantics to
> say that a db user is only responsible for calling AsyncClose. Firefox will
> guarantee that all async statements not explicitly marked as cancellable
> will be executed.
Summary: Move Storage asyncClose spinning earlier (before exit(0) → Move Storage asyncClose spinning earlier, before exit(0)
where "your proposal" is my proposal, actually.
As pointed out on the original bug, this is an API change, but on that would make it less error prone. One cannot just forget to make sure an async statement is executed, he has to explicitly make it cancelable.

In summary, I like the idea.

CCing Taras to see what he things and cc others he thinks might have an opinion on this.
This is ok by me.
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
Product: Toolkit → Core
You need to log in before you can comment on or make changes to this bug.