diff options
| author | 2024-06-12 09:47:41 -0700 | |
|---|---|---|
| committer | 2024-06-12 09:47:41 -0700 | |
| commit | e95076af2b804f35972c273967e82cb38e9c0f8a (patch) | |
| tree | b5f30c0a24e3c0b1255bc0363b0329645f569e5d /libs/androidfw/StringPool.cpp | |
| parent | 825c7a1adf96b923eb8f32134c37776cc9859782 (diff) | |
Address confusions around "subtypeId" in InputMethodSettings
This CL aims to clarify confusions around "subtypeId" in
InputMethodSettings without changing any observable behavior.
Due to historical confusion around InputMethodUtils#NOT_A_SUBTYPE_ID,
which happens to be -1, we have somehow used "-1" as an invalid data
when dealing with subtypeHashCode saved in the Secure Settings.
Strictly speaking, however, InputMethodUtils#NOT_A_SUBTYPE_ID, is an
invalid "array index", while the Secure Settings deal with invalid
subtype hashcode. To avoid future confusion, this CL introduces a new
constant
* INVALID_SUBTYPE_HASHCODE
* INVALID_SUBTYPE_HASHCODE_STR
to effectively decouple them from InputMethodUtils#NOT_A_SUBTYPE_ID
with keeping the current data format in the Secure Settings.
Bug: 346662143
Test: presubmit
Flag: EXEMPT refactor
Change-Id: Ic4968449ea89debc3bc9e519a724c16222851235
Diffstat (limited to 'libs/androidfw/StringPool.cpp')
0 files changed, 0 insertions, 0 deletions