-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Content Launcher feature map shows up as indexes instead of bitmask #33444
Comments
bzbarsky-apple
changed the title
Content Launcer feature map shows up as indexes instead of bitmask
Content Launcher feature map shows up as indexes instead of bitmask
May 14, 2024
Looks like the buggy XML was added in #30691 @lazarkov @hawk248 @Brian-Resideo |
These are not actually used in the code today, but this should be fixed to prevent future issues in case developers reference the output from these. |
They're shipped in the Android and Python APIs, no? (And Darwin, but we made sure to not expose this to API consumers so far....) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Definition of
is wrong because masks should be
0x01, 0x02, 0x04, 0x08, 0x10, ...
instead of using indexes. This prevented us from migrating to feature bits for these features.If we use these, there may be sideffects. Need to update the data type and validate if there are any sideffects in code (that thinks these are indexes instead of bitmasks).
The text was updated successfully, but these errors were encountered: