Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a85ddc54 authored by Jayant Chowdhary's avatar Jayant Chowdhary Committed by Android (Google) Code Review
Browse files

Merge "camera2 jni: nativeReadValues() can use camera_metadata_ro_entry... 

Merge "camera2 jni: nativeReadValues() can use camera_metadata_ro_entry instead of camera_metadata_entry" into main
parents e7ca1681 3d5cfaac

core/jni/android_hardware_camera2_CameraMetadata.cpp

+8 −10
Original line number Diff line number Diff line
@@ -336,7 +336,7 @@ static void CameraMetadata_swap(JNIEnv *env, jclass thiz, jlong ptr, jlong other 
static jbyteArray CameraMetadata_readValues(JNIEnv *env, jclass thiz, jint tag, jlong ptr) {

    ALOGV("%s (tag = %d)", __FUNCTION__, tag);



    CameraMetadata* metadata = CameraMetadata_getPointerThrow(env, ptr);

    const CameraMetadata *metadata = CameraMetadata_getPointerThrow(env, ptr);

    if (metadata == NULL) return NULL;



    const camera_metadata_t *metaBuffer = metadata->getAndLock();
@@ -349,16 +349,14 @@ static jbyteArray CameraMetadata_readValues(JNIEnv *env, jclass thiz, jint tag, 
    }

    size_t tagSize = Helpers::getTypeSize(tagType);



    camera_metadata_entry entry = metadata->find(tag);

    camera_metadata_ro_entry entry = metadata->find(tag);

    if (entry.count == 0) {

        if (!metadata->exists(tag)) {

            ALOGV("%s: Tag %d does not have any entries", __FUNCTION__, tag);

             return NULL;

        } else {

             // OK: we will return a 0-sized array.

             ALOGV("%s: Tag %d had an entry, but it had 0 data", __FUNCTION__,

                   tag);

            ALOGV("%s: Tag %d had an entry, but it had 0 data", __FUNCTION__, tag);

        }

        return NULL;

    }



    jsize byteCount = entry.count * tagSize;