Remove symbol 'b'/'B' from locale data before loaded by j.t.SimpleDateFormat
Reason:
- Symbol 'b'/'B' are not supported in the OpenJDK
SimpleDateFormat. The CL https://r.android.com/547859 silently
drop the symbol 'b'/'B' during formatting and parsing, but however,
it's still visible via the API SimpleDateFormat.toPattern(), and thus
caused the user, e.g. the desugaring library, to crash because it does
not expect the symbol.
- However, the locale data comes from ICU has a symbol 'b'/'B'. We need
to remove it before consumed by SimpleDateFormat.
The removal is applied when the SimpleDateFormat instance is
obtained from DateFormat.get*Instance().
The example locale is my-MM.
The fix only removes symbol 'b'/'B' when 24-hour 'H' exists. It prevents
causing more bugs when removing the symbol blindly, e.g. 'b'/'B' with
12-hour 'h'. The new test cases in DateFormatTest will ensure that
'b'/'B' symbols are not there.
Bug: 175349773
Test: CtsLibcoreTestCases
Change-Id: Ic7774daef2bee1efd6fd6e40009759446d8f57c2
3 files changed