1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Simplify code, remove redundant condition

Browse Source 
'clazz || (!clazz && isCached)' is equivalent to
'clazz || isCached'

Done-with: Tang Peng <tangpeng@uniontech.com>
Pick-to: 6.1
Change-Id: Ie9eab4a94a61be2b360f64980c4666a622f3a209
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
This commit is contained in:
Wang Yicun 2021-03-04 17:19:55 +08:00 committed by Volker Hilsheimer
parent c1c61c3bb4
commit a913002f13
2 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/corelib/kernel/qjni.cpp
View File

@ -280,9 +280,7 @@ jclass QJNIEnvironmentPrivate::findClass(const char *className, JNIEnv *env)
bool isCached = false;
jclass clazz = getCachedClass(classDotEnc, &isCached);
const bool found = (clazz != 0) || (clazz == 0 && isCached);
if (found)
if (clazz || isCached)
return clazz;
const QLatin1String key(classDotEnc);

4
src/corelib/kernel/qjniobject.cpp
View File

@ -467,9 +467,7 @@ jclass QtAndroidPrivate::findClass(const char *className, JNIEnv *env)
bool isCached = false;
jclass clazz = getCachedClass(classDotEnc, &isCached);
const bool found = clazz || (!clazz && isCached);
if (found)
if (clazz || isCached)
return clazz;
const QLatin1String key(classDotEnc);