commit 0cb67a606cb1f3f6a3fd3f376d2d236f465e8d85
author Phil Burk <philburk@google.com> Mon Mar 26 09:42:22 2018 -0700
committer Phil Burk <philburk@google.com> Mon Mar 26 09:42:22 2018 -0700
tree 7e186cdb23b8e0cd889983f532dfb58060b6b91a
parent dac7687db5b400d361745e5954f9441e2c1797c9

oboe: cleanup OpenSL ES input

Init imRecordInterface to nullptr.
Better logs.

include/oboe/AudioStream.h

diff --git a/include/oboe/AudioStream.h b/include/oboe/AudioStream.h
index 8317ad4..6de4fb8 100644
--- a/include/oboe/AudioStream.h
+++ b/include/oboe/AudioStream.h
@@ -186,7 +186,6 @@
         return static_cast<int32_t>(Result::ErrorUnimplemented);
     }
 
-
     /**
      *
      * @return the API that this stream uses
@@ -228,8 +227,8 @@
      *   or ErrorTimeout.
      */
     virtual Result waitForStateTransition(StreamState startingState,
-                                              StreamState endingState,
-                                              int64_t timeoutNanoseconds);
+                                          StreamState endingState,
+                                          int64_t timeoutNanoseconds);
 
     /**
      * Override this to provide a default for when the application did not specify a callback.

src/common/AudioStream.cpp

diff --git a/src/common/AudioStream.cpp b/src/common/AudioStream.cpp
index 56fd9bd..1c468dd 100644
--- a/src/common/AudioStream.cpp
+++ b/src/common/AudioStream.cpp
@@ -61,8 +61,8 @@
 }
 
 Result AudioStream::waitForStateTransition(StreamState startingState,
-                                               StreamState endingState,
-                                               int64_t timeoutNanoseconds)
+                                           StreamState endingState,
+                                           int64_t timeoutNanoseconds)
 {
     StreamState state = getState();
     StreamState nextState = state;

src/opensles/AudioInputStreamOpenSLES.cpp

diff --git a/src/opensles/AudioInputStreamOpenSLES.cpp b/src/opensles/AudioInputStreamOpenSLES.cpp
index dfe8513..f80d2de 100644
--- a/src/opensles/AudioInputStreamOpenSLES.cpp
+++ b/src/opensles/AudioInputStreamOpenSLES.cpp
@@ -156,12 +156,13 @@
 Result AudioInputStreamOpenSLES::setRecordState(SLuint32 newState) {
     Result result = Result::OK;
     LOGD("AudioInputStreamOpenSLES::setRecordState(%d)", newState);
-    if (mRecordInterface == NULL) {
+    if (mRecordInterface == nullptr) {
+        LOGE("AudioInputStreamOpenSLES::SetRecordState() mRecordInterface is null");
         return Result::ErrorInvalidState;
     }
     SLresult slResult = (*mRecordInterface)->SetRecordState(mRecordInterface, newState);
     if(SL_RESULT_SUCCESS != slResult) {
-        LOGD("AudioInputStreamOpenSLES::setPlayState() returned %s", getSLErrStr(slResult));
+        LOGE("AudioInputStreamOpenSLES::SetRecordState() returned %s", getSLErrStr(slResult));
         result = Result::ErrorInvalidState; // TODO review
     } else {
         setState(StreamState::Pausing);
@@ -173,9 +174,7 @@
 {
     LOGD("AudioInputStreamOpenSLES::requestStart()");
     Result result = setRecordState(SL_RECORDSTATE_RECORDING);
-    if(result != Result::OK) {
-        result = Result::ErrorInvalidState; // TODO review
-    } else {
+    if(result == Result::OK) {
         // Enqueue the first buffer so that we have data ready in the callback.
         enqueueCallbackBuffer(mSimpleBufferQueueInterface);
         setState(StreamState::Starting);

src/opensles/AudioInputStreamOpenSLES.h

diff --git a/src/opensles/AudioInputStreamOpenSLES.h b/src/opensles/AudioInputStreamOpenSLES.h
index 89c03d0..8fb2b8b 100644
--- a/src/opensles/AudioInputStreamOpenSLES.h
+++ b/src/opensles/AudioInputStreamOpenSLES.h
@@ -55,7 +55,7 @@
 
     Result setRecordState(SLuint32 newState);
 
-    SLRecordItf mRecordInterface;
+    SLRecordItf mRecordInterface = nullptr;
 };
 
 } // namespace oboe