Glean Android uses `SimpleDateFormat` for which there may be a better choice in `DateTimeFormatter`
Categories
(Data Platform and Tools :: Glean: SDK, enhancement, P2)
Tracking
(Not tracked)
People
(Reporter: travis_, Assigned: travis_)
Details
According to this stackoverflow comment that I stumbled upon while looking into how we were using dates in the MPS for Android, SimpleDateFormat is not threadsafe and looking at DateTimeFormatter it appears to have a built-in format for ISO strings that looks like we wouldn't have to do the string manipulation with that we do here
|
||
Comment 1•4 years ago
|
||
Fly-by: Note that the reason why we didn't go with DateTimeFormatter is that it needs API level 26, while A-C compatibility requires us to support API level 21. Not sure if we're still required to stick to 21 though..
|
Assignee | |
Comment 2•4 years ago
|
||
It does appear that SDK 21 is still the minimum support level for A-C and there aren't any other components that are using DateTimeFormatter, they all appear to be using SimpleTimeFormat yet. Might as well close this as won't fix until A-C updates minSdk.
Description
•