Skip to content
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

Reduce visibility of PerformanceCounter #39746

Closed
wants to merge 9 commits into from

Conversation

mdvacca
Copy link
Contributor

@mdvacca mdvacca commented Oct 1, 2023

Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of PerformanceCounter

Reviewed By: arushikesarwani94

Differential Revision: D49803283

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Oct 1, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49803283

@analysis-bot
Copy link

analysis-bot commented Oct 1, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 16,902,901 -13
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 20,381,659 -4
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 294c0fa
Branch: main

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Warnings
⚠️ One hour and a half have passed and the E2E jobs haven't finished yet.

Generated by 🚫 dangerJS against 02775de

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49803283

mdvacca added a commit to mdvacca/react-native that referenced this pull request Oct 1, 2023
Summary:
Pull Request resolved: facebook#39746

In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of PerformanceCounter

Reviewed By: arushikesarwani94

Differential Revision: D49803283

fbshipit-source-id: 39aff598ddfd56a3fde75499099a5f58ce6ae496
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of DynamicFromArray

Differential Revision: D49752134

fbshipit-source-id: 0766ade364e3cdfe9997a5947e3cea7a17dc1c0b
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of DynamicFromMap

Differential Revision: D49752137

fbshipit-source-id: 663b8b951780896a01ec737753ef684264082b0c
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of JSCJavaScriptExecutorFactory

Differential Revision: D49752141

fbshipit-source-id: 4f2cc3e96116e9544851564a687d0f0e1b8fbe76
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of JSIModuleHolder

Differential Revision: D49752136

fbshipit-source-id: 4cdf407833a354c06d92b95a4ec84b5e33eb2b87
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of JSIModuleRegistry.

Differential Revision: D49752138

fbshipit-source-id: ca2b4fe7ad0a1027c86592676d3d76f6ec933e29
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of JSInstance

Differential Revision: D49752139

fbshipit-source-id: 47e038562349b2596a30209381f21d584bed18a0
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of NativeArgumentsParseException

Differential Revision: D49752145

fbshipit-source-id: 708ea1fbcb48f646fd03edd51a0ad04a2bf7f82f
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of NativeArrayInterface

Differential Revision: D49803296

fbshipit-source-id: 2715ae89f7019f9c0666a7203143f3273039cad5
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49803283

Summary:
Pull Request resolved: facebook#39746

In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.

If you are using this class or interface please comment in this PR and we will restate the public access.

changelog: [Android][Changed] Reducing visibility of PerformanceCounter

Reviewed By: arushikesarwani94

Differential Revision: D49803283

fbshipit-source-id: 69f0d4049e8180286963de73c9c891453c81c40b
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Oct 2, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 614afb5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants