Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

am 1d4cd879: am 11cd57ca: Merge "Avoid unnecessary FindClass("java/lang/String")."

Browse Source 
* commit '1d4cd879044578cf7e2b37d45ef29abaabf7e662':
  Avoid unnecessary FindClass("java/lang/String").
This commit is contained in:
Dianne Hackborn
2013-06-17 18:28:58 -07:00
committed by Android Git Automerger
parent 8e8f5c119e 1d4cd87904
commit 2f3f86c113
1 changed files with 3 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
core/jni/android_util_AssetManager.cpp
View File

@@ -293,17 +293,10 @@ static jobjectArray android_content_AssetManager_list(JNIEnv* env, jobject clazz
return NULL;
}
jclass cls = env->FindClass("java/lang/String");
LOG_FATAL_IF(cls == NULL, "No string class?!?");
if (cls == NULL) {
delete dir;
return NULL;
}
size_t N = dir->getFileCount();
jobjectArray array = env->NewObjectArray(dir->getFileCount(),
cls, NULL);
g_stringClass, NULL);
if (array == NULL) {
delete dir;
return NULL;
@@ -1459,19 +1452,13 @@ static jobjectArray android_content_AssetManager_getArrayStringResource(JNIEnv*
}
const ResTable& res(am->getResources());
jclass cls = env->FindClass("java/lang/String");
LOG_FATAL_IF(cls == NULL, "No string class?!?");
if (cls == NULL) {
return NULL;
}
const ResTable::bag_entry* startOfBag;
const ssize_t N = res.lockBag(arrayResId, &startOfBag);
if (N < 0) {
return NULL;
}
jobjectArray array = env->NewObjectArray(N, cls, NULL);
jobjectArray array = env->NewObjectArray(N, g_stringClass, NULL);
if (env->ExceptionCheck()) {
res.unlockBag(startOfBag);
return NULL;
@@ -1768,6 +1755,7 @@ int register_android_content_AssetManager(JNIEnv* env)
jclass stringClass = env->FindClass("java/lang/String");
LOG_FATAL_IF(stringClass == NULL, "Unable to find class java/lang/String");
g_stringClass = (jclass)env->NewGlobalRef(stringClass);
LOG_FATAL_IF(g_stringClass == NULL, "Unable to create global reference for class java/lang/String");
return AndroidRuntime::registerNativeMethods(env,
"android/content/res/AssetManager", gAssetManagerMethods, NELEM(gAssetManagerMethods));