react-native icon indicating copy to clipboard operation
react-native copied to clipboard

[Android] MapBuffer long support

Open ryanlntn opened this issue 2 years ago • 4 comments

Summary:

This adds support for 64 bit integer (long) values to MapBuffer. Per the wide gamut color RFC Android encodes wide gamut colors as long values so we need to update MapBuffer to support 64 bit integers as well.

Changelog:

[ANDROID] [ADDED] - Add 64 bit integer (long) value support to MapBuffer

Test Plan:

I've added a test to the MapBuffer test suite. This new API is otherwise currently unused but will be used in subsequent PRs as part of wide gamut color support changes.

ryanlntn avatar Feb 14 '24 18:02 ryanlntn

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 17,832,429 -102,649
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 21,186,796 -106,487
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 4ecf57ead2a32294cb9ed19088e7cce59702794e Branch: main

analysis-bot avatar Feb 14 '24 21:02 analysis-bot

@NickGerleman has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Feb 17 '24 03:02 facebook-github-bot

@NickGerleman I dropped the bridge changes to keep this completely focused on MapBuffer.

ryanlntn avatar Feb 20 '24 19:02 ryanlntn

@NickGerleman has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Feb 22 '24 18:02 facebook-github-bot

@NickGerleman merged this pull request in facebook/react-native@57ed0fb30931979742634a1faa9a4d3b5261e50d.

facebook-github-bot avatar Feb 28 '24 01:02 facebook-github-bot