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

Merge "Refine DeleteLocalRef invocation"

Browse Source
This commit is contained in:
Ray Chin
2022-11-18 09:34:11 +00:00
committed by Android (Google) Code Review
parent 40f628798e 0b8d90d23e
commit 677a47590f
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
media/jni/android_media_tv_Tuner.cpp
View File

@@ -380,6 +380,7 @@ void LnbClientCallbackImpl::onDiseqcMessage(const vector<uint8_t> &diseqcMessage
gFields.onLnbDiseqcMessageID,
array);
env->DeleteLocalRef(lnb);
env->DeleteLocalRef(array);
} else {
ALOGE("LnbClientCallbackImpl::onDiseqcMessage:"
"Lnb object has been freed. Ignoring callback.");
@@ -848,6 +849,7 @@ void FilterClientCallbackImpl::getRestartEvent(jobjectArray &arr, const int size
const int32_t &startId = event.get<DemuxFilterEvent::Tag::startId>();
jobject obj = env->NewObject(eventClazz, eventInit, startId);
env->SetObjectArrayElement(arr, size, obj);
env->DeleteLocalRef(obj);
}
void FilterClientCallbackImpl::onFilterEvent(const vector<DemuxFilterEvent> &events) {
@@ -2181,7 +2183,6 @@ jobject JTuner::getFrontendStatus(jintArray types) {
env->NewObject(longClazz, initLong,
static_cast<long>(s.get<FrontendStatus::Tag::innerFec>()));
env->SetObjectField(statusObj, field, newLongObj);
env->DeleteLocalRef(longClazz);
env->DeleteLocalRef(newLongObj);
break;
}
@@ -2347,7 +2348,6 @@ jobject JTuner::getFrontendStatus(jintArray types) {
}
env->SetObjectField(statusObj, field, valObj);
env->DeleteLocalRef(plpClazz);
env->DeleteLocalRef(valObj);
break;
}
@@ -2758,7 +2758,6 @@ jobject JTuner::getFrontendStatus(jintArray types) {
}
env->SetObjectField(statusObj, field, valObj);
env->DeleteLocalRef(plpClazz);
env->DeleteLocalRef(valObj);
break;
}