commit 2c08dc66c2980a50e90befe84f836b7ebf17fe94
author Eino-Ville Talvala <etalvala@google.com> Fri Jun 15 12:49:21 2012 -0700
committer Android (Google) Code Review <android-gerrit@google.com> Mon Jun 18 08:40:58 2012 -0700
tree 2044dfd736a94c1a24020eece437a132ac0f88ba
parent 74cfa5b792fc17d7afd3e833d552fa0ec45ea977

Camera2: Move per-frame verbose logs to separate log enable

Bug: 6243944
Change-Id: I5850f8ceb2e603aa555cc71328f64d6a928694a0

services/camera/libcameraservice/Camera2Device.cpp

diff --git a/services/camera/libcameraservice/Camera2Device.cpp b/services/camera/libcameraservice/Camera2Device.cpp
index 4761263..5a37c8d 100644
--- a/services/camera/libcameraservice/Camera2Device.cpp
+++ b/services/camera/libcameraservice/Camera2Device.cpp
@@ -16,6 +16,13 @@
 
 #define LOG_TAG "Camera2Device"
 //#define LOG_NDEBUG 0
+//#define LOG_NNDEBUG 0  // Per-frame verbose logging
+
+#ifdef LOG_NNDEBUG
+#define ALOGVV(...) ALOGV(__VA_ARGS__)
+#else
+#define ALOGVV(...) ((void)0)
+#endif
 
 #include <utils/Log.h>
 #include "Camera2Device.h"
@@ -324,7 +331,7 @@
 
 // Real interfaces
 status_t Camera2Device::MetadataQueue::enqueue(camera_metadata_t *buf) {
-    ALOGV("%s: E", __FUNCTION__);
+    ALOGVV("%s: E", __FUNCTION__);
     Mutex::Autolock l(mMutex);
 
     mCount++;
@@ -344,18 +351,18 @@
 status_t Camera2Device::MetadataQueue::dequeue(camera_metadata_t **buf,
         bool incrementCount)
 {
-    ALOGV("%s: E", __FUNCTION__);
+    ALOGVV("%s: E", __FUNCTION__);
     status_t res;
     Mutex::Autolock l(mMutex);
 
     if (mCount == 0) {
         if (mStreamSlotCount == 0) {
-            ALOGV("%s: Empty", __FUNCTION__);
+            ALOGVV("%s: Empty", __FUNCTION__);
             *buf = NULL;
             mSignalConsumer = true;
             return OK;
         }
-        ALOGV("%s: Streaming %d frames to queue", __FUNCTION__,
+        ALOGVV("%s: Streaming %d frames to queue", __FUNCTION__,
               mStreamSlotCount);
 
         for (List<camera_metadata_t*>::iterator slotEntry = mStreamSlot.begin();
@@ -371,7 +378,7 @@
         }
         mCount = mStreamSlotCount;
     }
-    ALOGV("MetadataQueue: deque (%d buffers)", mCount);
+    ALOGVV("MetadataQueue: deque (%d buffers)", mCount);
     camera_metadata_t *b = *(mEntries.begin());
     mEntries.erase(mEntries.begin());
 
@@ -879,7 +886,7 @@
     *buffer = &(anb->handle);
     stream->mActiveBuffers++;
 
-    ALOGV("%s: Buffer %p", __FUNCTION__, *buffer);
+    ALOGVV("%s: Buffer %p", __FUNCTION__, *buffer);
     return res;
 }
 
@@ -888,7 +895,7 @@
         buffer_handle_t* buffer) {
     StreamAdapter *stream =
             const_cast<StreamAdapter*>(static_cast<const StreamAdapter*>(w));
-    ALOGV("%s: Stream %d: Buffer %p captured at %lld ns",
+    ALOGVV("%s: Stream %d: Buffer %p captured at %lld ns",
             __FUNCTION__, stream->mId, buffer, timestamp);
     int state = stream->mState;
     if (state != ACTIVE) {