From 6770260f83f5576b041b7b3a2536d082a631f9b0 Mon Sep 17 00:00:00 2001 From: Ruben Brunk Date: Sat, 3 May 2014 00:28:33 +0000 Subject: Revert "camera3: Update CameraMetadata vendor tag handling." This reverts commit 4764324d2704f1b08bb7407014bf203924ff2363. Change-Id: I86d7fcc11423502b73fab2a82af6aa89804f0b76 --- camera/Android.mk | 5 - camera/VendorTagDescriptor.cpp | 162 ++------------------- include/camera/VendorTagDescriptor.h | 23 +-- services/camera/libcameraservice/CameraService.cpp | 15 +- 4 files changed, 14 insertions(+), 191 deletions(-) diff --git a/camera/Android.mk b/camera/Android.mk index 18800b4..5774b6f 100644 --- a/camera/Android.mk +++ b/camera/Android.mk @@ -52,11 +52,6 @@ LOCAL_C_INCLUDES += \ system/media/camera/include \ system/media/private/camera/include -## Enable asserts for 'eng' builds -ifeq ($(TARGET_BUILD_VARIANT),eng) -LOCAL_CFLAGS += -UNDEBUG -endif - LOCAL_MODULE:= libcamera_client include $(BUILD_SHARED_LIBRARY) diff --git a/camera/VendorTagDescriptor.cpp b/camera/VendorTagDescriptor.cpp index d9a7a54..a0a6a51 100644 --- a/camera/VendorTagDescriptor.cpp +++ b/camera/VendorTagDescriptor.cpp @@ -21,13 +21,11 @@ #include #include #include -#include #include #include #include "camera/VendorTagDescriptor.h" -#include #include namespace android { @@ -47,13 +45,7 @@ static Mutex sLock; static sp sGlobalVendorTagDescriptor; VendorTagDescriptor::VendorTagDescriptor() {} - -VendorTagDescriptor::~VendorTagDescriptor() { - size_t len = mReverseMapping.size(); - for (size_t i = 0; i < len; ++i) { - delete mReverseMapping[i]; - } -} +VendorTagDescriptor::~VendorTagDescriptor() {} status_t VendorTagDescriptor::createDescriptorFromOps(const vendor_tag_ops_t* vOps, /*out*/ @@ -78,9 +70,6 @@ status_t VendorTagDescriptor::createDescriptorFromOps(const vendor_tag_ops_t* vO sp desc = new VendorTagDescriptor(); desc->mTagCount = tagCount; - SortedVector sections; - KeyedVector tagToSectionMap; - for (size_t i = 0; i < static_cast(tagCount); ++i) { uint32_t tag = tagArray[i]; if (tag < CAMERA_METADATA_VENDOR_TAG_BOUNDARY) { @@ -98,12 +87,7 @@ status_t VendorTagDescriptor::createDescriptorFromOps(const vendor_tag_ops_t* vO ALOGE("%s: no section name defined for vendor tag %d.", __FUNCTION__, tag); return BAD_VALUE; } - - String8 sectionString(sectionName); - - sections.add(sectionString); - tagToSectionMap.add(tag, sectionString); - + desc->mTagToSectionMap.add(tag, String8(sectionName)); int tagType = vOps->get_tag_type(vOps, tag); if (tagType < 0 || tagType >= NUM_TYPES) { ALOGE("%s: tag type %d from vendor ops does not exist.", __FUNCTION__, tagType); @@ -111,27 +95,6 @@ status_t VendorTagDescriptor::createDescriptorFromOps(const vendor_tag_ops_t* vO } desc->mTagToTypeMap.add(tag, tagType); } - - desc->mSections = sections; - - for (size_t i = 0; i < static_cast(tagCount); ++i) { - uint32_t tag = tagArray[i]; - String8 sectionString = tagToSectionMap.valueFor(tag); - - // Set up tag to section index map - ssize_t index = sections.indexOf(sectionString); - assert(index >= 0); - desc->mTagToSectionMap.add(tag, static_cast(index)); - - // Set up reverse mapping - ssize_t reverseIndex = -1; - if ((reverseIndex = desc->mReverseMapping.indexOfKey(sectionString)) < 0) { - KeyedVector* nameMapper = new KeyedVector(); - reverseIndex = desc->mReverseMapping.add(sectionString, nameMapper); - } - desc->mReverseMapping[reverseIndex]->add(desc->mTagToNameMap[tag], tag); - } - descriptor = desc; return OK; } @@ -159,10 +122,8 @@ status_t VendorTagDescriptor::createFromParcel(const Parcel* parcel, sp desc = new VendorTagDescriptor(); desc->mTagCount = tagCount; - uint32_t tag, sectionIndex; - uint32_t maxSectionIndex = 0; + uint32_t tag; int32_t tagType; - Vector allTags; for (int32_t i = 0; i < tagCount; ++i) { if ((res = parcel->readInt32(reinterpret_cast(&tag))) != OK) { ALOGE("%s: could not read tag id from parcel for index %d", __FUNCTION__, i); @@ -188,17 +149,15 @@ status_t VendorTagDescriptor::createFromParcel(const Parcel* parcel, res = NOT_ENOUGH_DATA; break; } - - if ((res = parcel->readInt32(reinterpret_cast(§ionIndex))) != OK) { - ALOGE("%s: could not read section index for tag %d.", __FUNCTION__, tag); + String8 sectionName = parcel->readString8(); + if (sectionName.isEmpty()) { + ALOGE("%s: parcel section name was NULL for tag %d.", __FUNCTION__, tag); + res = NOT_ENOUGH_DATA; break; } - maxSectionIndex = (maxSectionIndex >= sectionIndex) ? maxSectionIndex : sectionIndex; - - allTags.add(tag); desc->mTagToNameMap.add(tag, tagName); - desc->mTagToSectionMap.add(tag, sectionIndex); + desc->mTagToSectionMap.add(tag, sectionName); desc->mTagToTypeMap.add(tag, tagType); } @@ -206,42 +165,6 @@ status_t VendorTagDescriptor::createFromParcel(const Parcel* parcel, return res; } - size_t sectionCount; - if (tagCount > 0) { - if ((res = parcel->readInt32(reinterpret_cast(§ionCount))) != OK) { - ALOGE("%s: could not read section count for.", __FUNCTION__); - return res; - } - if (sectionCount < (maxSectionIndex + 1)) { - ALOGE("%s: Incorrect number of sections defined, received %d, needs %d.", - __FUNCTION__, sectionCount, (maxSectionIndex + 1)); - return BAD_VALUE; - } - assert(desc->mSections.setCapacity(sectionCount) > 0); - for (size_t i = 0; i < sectionCount; ++i) { - String8 sectionName = parcel->readString8(); - if (sectionName.isEmpty()) { - ALOGE("%s: parcel section name was NULL for section %d.", __FUNCTION__, i); - return NOT_ENOUGH_DATA; - } - desc->mSections.add(sectionName); - } - } - - assert(tagCount == allTags.size()); - // Set up reverse mapping - for (size_t i = 0; i < static_cast(tagCount); ++i) { - uint32_t tag = allTags[i]; - String8 sectionString = desc->mSections[desc->mTagToSectionMap.valueFor(tag)]; - - ssize_t reverseIndex = -1; - if ((reverseIndex = desc->mReverseMapping.indexOfKey(sectionString)) < 0) { - KeyedVector* nameMapper = new KeyedVector(); - reverseIndex = desc->mReverseMapping.add(sectionString, nameMapper); - } - desc->mReverseMapping[reverseIndex]->add(desc->mTagToNameMap[tag], tag); - } - descriptor = desc; return res; } @@ -266,7 +189,7 @@ const char* VendorTagDescriptor::getSectionName(uint32_t tag) const { if (index < 0) { return VENDOR_SECTION_NAME_ERR; } - return mSections[mTagToSectionMap.valueAt(index)].string(); + return mTagToSectionMap.valueAt(index).string(); } const char* VendorTagDescriptor::getTagName(uint32_t tag) const { @@ -297,83 +220,22 @@ status_t VendorTagDescriptor::writeToParcel(Parcel* parcel) const { } size_t size = mTagToNameMap.size(); - uint32_t tag, sectionIndex; + uint32_t tag; int32_t tagType; for (size_t i = 0; i < size; ++i) { tag = mTagToNameMap.keyAt(i); String8 tagName = mTagToNameMap[i]; - sectionIndex = mTagToSectionMap.valueFor(tag); + String8 sectionName = mTagToSectionMap.valueFor(tag); tagType = mTagToTypeMap.valueFor(tag); if ((res = parcel->writeInt32(tag)) != OK) break; if ((res = parcel->writeInt32(tagType)) != OK) break; if ((res = parcel->writeString8(tagName)) != OK) break; - if ((res = parcel->writeInt32(sectionIndex)) != OK) break; - } - - size_t numSections = mSections.size(); - if (numSections > 0) { - if ((res = parcel->writeInt32(numSections)) != OK) return res; - for (size_t i = 0; i < numSections; ++i) { - if ((res = parcel->writeString8(mSections[i])) != OK) return res; - } + if ((res = parcel->writeString8(sectionName)) != OK) break; } return res; } -SortedVector VendorTagDescriptor::getAllSectionNames() const { - return mSections; -} - -status_t VendorTagDescriptor::lookupTag(String8 name, String8 section, /*out*/uint32_t* tag) const { - ssize_t index = mReverseMapping.indexOfKey(section); - if (index < 0) { - ALOGE("%s: Section '%s' does not exist.", __FUNCTION__, section.string()); - return BAD_VALUE; - } - - ssize_t nameIndex = mReverseMapping[index]->indexOfKey(name); - if (nameIndex < 0) { - ALOGE("%s: Tag name '%s' does not exist.", __FUNCTION__, name.string()); - return BAD_VALUE; - } - - if (tag != NULL) { - *tag = mReverseMapping[index]->valueAt(nameIndex); - } - return OK; -} - -void VendorTagDescriptor::dump(int fd, int verbosity, int indentation) const { - - size_t size = mTagToNameMap.size(); - if (size == 0) { - fdprintf(fd, "%*sDumping configured vendor tag descriptors: None set\n", - indentation, ""); - return; - } - - fdprintf(fd, "%*sDumping configured vendor tag descriptors: %zu entries\n", - indentation, "", size); - for (size_t i = 0; i < size; ++i) { - uint32_t tag = mTagToNameMap.keyAt(i); - - if (verbosity < 1) { - fdprintf(fd, "%*s0x%x\n", indentation + 2, "", tag); - continue; - } - String8 name = mTagToNameMap.valueAt(i); - uint32_t sectionId = mTagToSectionMap.valueFor(tag); - String8 sectionName = mSections[sectionId]; - int type = mTagToTypeMap.valueFor(tag); - const char* typeName = (type >= 0 && type < NUM_TYPES) ? - camera_metadata_type_names[type] : "UNKNOWN"; - fdprintf(fd, "%*s0x%x (%s) with type %d (%s) defined in section %s\n", indentation + 2, - "", tag, name.string(), type, typeName, sectionName.string()); - } - -} - status_t VendorTagDescriptor::setAsGlobalVendorTagDescriptor(const sp& desc) { status_t res = OK; Mutex::Autolock al(sLock); diff --git a/include/camera/VendorTagDescriptor.h b/include/camera/VendorTagDescriptor.h index 1758acf..ea21d31 100644 --- a/include/camera/VendorTagDescriptor.h +++ b/include/camera/VendorTagDescriptor.h @@ -16,7 +16,6 @@ #ifndef VENDOR_TAG_DESCRIPTOR_H -#include #include #include #include @@ -68,24 +67,6 @@ class VendorTagDescriptor /*out*/ Parcel* parcel) const; - /** - * Convenience method to get a vector containing all vendor tag - * sections, or an empty vector if none are defined. - */ - SortedVector getAllSectionNames() const; - - /** - * Lookup the tag id for a given tag name and section. - * - * Returns OK on success, or a negative error code. - */ - status_t lookupTag(String8 name, String8 section, /*out*/uint32_t* tag) const; - - /** - * Dump the currently configured vendor tags to a file descriptor. - */ - void dump(int fd, int verbosity, int indentation) const; - // Static methods: /** @@ -128,11 +109,9 @@ class VendorTagDescriptor static sp getGlobalVendorTagDescriptor(); protected: VendorTagDescriptor(); - KeyedVector*> mReverseMapping; KeyedVector mTagToNameMap; - KeyedVector mTagToSectionMap; // Value is offset in mSections + KeyedVector mTagToSectionMap; KeyedVector mTagToTypeMap; - SortedVector mSections; // must be int32_t to be compatible with Parcel::writeInt32 int32_t mTagCount; private: diff --git a/services/camera/libcameraservice/CameraService.cpp b/services/camera/libcameraservice/CameraService.cpp index 9001b3a..02bca1f 100644 --- a/services/camera/libcameraservice/CameraService.cpp +++ b/services/camera/libcameraservice/CameraService.cpp @@ -1257,20 +1257,7 @@ status_t CameraService::dump(int fd, const Vector& args) { result.appendFormat("Camera module author: %s\n", mModule->common.author); result.appendFormat("Number of camera devices: %d\n\n", mNumberOfCameras); - - sp desc = VendorTagDescriptor::getGlobalVendorTagDescriptor(); - if (desc == NULL) { - result.appendFormat("Vendor tags left unimplemented.\n"); - } else { - result.appendFormat("Vendor tag definitions:\n"); - } - write(fd, result.string(), result.size()); - - if (desc != NULL) { - desc->dump(fd, /*verbosity*/2, /*indentation*/4); - } - for (int i = 0; i < mNumberOfCameras; i++) { result = String8::format("Camera %d static information:\n", i); camera_info info; @@ -1295,7 +1282,7 @@ status_t CameraService::dump(int fd, const Vector& args) { result.appendFormat(" Device static metadata:\n"); write(fd, result.string(), result.size()); dump_indented_camera_metadata(info.static_camera_characteristics, - fd, /*verbosity*/2, /*indentation*/4); + fd, 2, 4); } else { write(fd, result.string(), result.size()); } -- cgit v1.1