Closed
Bug 1569516
Opened 5 years ago
Closed 5 years ago
Remove very old calendar storage migration code
Categories
(Calendar :: Provider: Local Storage, task)
Calendar
Provider: Local Storage
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 70.0
People
(Reporter: darktrojan, Assigned: darktrojan)
Details
Attachments
(1 file)
|
21.74 KB,
patch
|
Fallen
:
review+
|
Details | Diff | Splinter Review |
It's highly unlikely that any user would still be using a storage.sdb database, and it's unreasonable to expect that we could still migrate from it to the current database.
Removing the migration code also prevents us from synchronously opening the file at start-up. We never check whether the file exists, so even if it doesn't we create it and then try to read it.
|
Assignee | |
Comment 1•5 years ago
|
||
Attachment #9081179 -
Flags: review?(philipp)
|
||
Comment 2•5 years ago
|
||
Comment on attachment 9081179 [details] [diff] [review] 1569516-remove-old-schema-1.diff Review of attachment 9081179 [details] [diff] [review]: ----------------------------------------------------------------- \o/ to removing old code!
Attachment #9081179 -
Flags: review?(philipp) → review+
|
||
Updated•5 years ago
|
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/580987860caa
Remove very old calendar storage migration code. r=philipp
|
|
||
Updated•5 years ago
|
Target Milestone: --- → 70
You need to log in
before you can comment on or make changes to this bug.
Description
•