status-im/react-native
2
0
Fork 0
mirror of https://github.com/status-im/react-native.git synced 2025-02-27 16:40:38 +00:00
Code Issues Projects Releases Wiki Activity

Android: Decrease cost of reflection

Browse Source 
Summary:
This change suppresses access checking during reflection which makes reflection faster by decreasing its overhead.

**Test plan (required)**

My team uses this change in our app.

Adam Comella
Microsoft Corp.
Closes https://github.com/facebook/react-native/pull/11204

Differential Revision: D4250790

Pulled By: astreet

fbshipit-source-id: 0ee2f40dcadccc695980fcae14fafe1050acb52f
This commit is contained in:
Adam Comella 2016-11-30 03:57:58 -08:00 committed by Facebook Github Bot
parent 528a3c776a
commit 911c05a89b
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ReactAndroid/src/main/java/com/facebook/react/bridge/BaseJavaModule.java
View File

@ -169,6 +169,7 @@ public abstract class BaseJavaModule implements NativeModule {
public JavaMethod(Method method) {
mMethod = method;
mMethod.setAccessible(true);
Class[] parameterTypes = method.getParameterTypes();
mArgumentExtractors = buildArgumentExtractors(parameterTypes);
mSignature = buildSignature(parameterTypes);
@ -354,6 +355,7 @@ public abstract class BaseJavaModule implements NativeModule {
public SyncJavaHook(Method method) {
mMethod = method;
mMethod.setAccessible(true);
mSignature = buildSignature(method);
}