From 185320db23cc1f9c50f61548d3b75fe08c890edc Mon Sep 17 00:00:00 2001 From: David Vacca Date: Wed, 6 Feb 2019 09:20:56 -0800 Subject: [PATCH] Initialize internals of ReadableNativeMap with the correct initial capacity Summary: This diff initializes internals maps of ReadableNativeMap with the correct initial capacity Reviewed By: kathryngray Differential Revision: D13970989 fbshipit-source-id: d104fcbaf938f377a0138b839375c4bc146f1732 --- .../com/facebook/react/bridge/ReadableNativeMap.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/bridge/ReadableNativeMap.java b/ReactAndroid/src/main/java/com/facebook/react/bridge/ReadableNativeMap.java index 1ece8d666..505486650 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/bridge/ReadableNativeMap.java +++ b/ReactAndroid/src/main/java/com/facebook/react/bridge/ReadableNativeMap.java @@ -54,8 +54,9 @@ public class ReadableNativeMap extends NativeMap implements ReadableMap { if (mLocalMap == null) { Object[] values = Assertions.assertNotNull(importValues()); mJniCallCounter++; - mLocalMap = new HashMap<>(); - for(int i = 0; i< mKeys.length; i++) { + int length = mKeys.length; + mLocalMap = new HashMap<>(length); + for(int i = 0; i< length; i++) { mLocalMap.put(mKeys[i], values[i]); } } @@ -80,8 +81,9 @@ public class ReadableNativeMap extends NativeMap implements ReadableMap { if (mLocalTypeMap == null) { Object[] types = Assertions.assertNotNull(importTypes()); mJniCallCounter++; - mLocalTypeMap = new HashMap<>(); - for(int i = 0; i< mKeys.length; i++) { + int length = mKeys.length; + mLocalTypeMap = new HashMap<>(length); + for(int i = 0; i< length; i++) { mLocalTypeMap.put(mKeys[i], (ReadableType) types[i]); } }