Closed Bug 735536 Opened 9 years ago Closed 9 years ago
B2G SMS DB: Use new fangled Indexed
DB string constants
See bug 735094.
Also c.f. bug 735357 for inspiration (not that the fix is that complicated...)
Sorry, the patch was corrupted and wouldn´t apply.
Attachment #606244 - Attachment is patch: true
Comment on attachment 606244 [details] [diff] [review] Fix v1 I would actually prefer if we const'ed those values, e.g.: const READONLY = "readonly"; etc. Looks good otherwise!
Attachment #606244 - Flags: review?(philipp) → feedback+
Yes, I thought about the constants, but the ContactDB patch didn´t use constants, so I chose to also don´t use it. Anyway, I also prefer this way.
Attachment #606669 - Flags: review?(philipp) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
You need to log in before you can comment on or make changes to this bug.