Use in-class initializer for class members

[platform/core/api/audio-io.git] / src / cpp / CAudioInput.cpp

diff --git a/src/cpp/CAudioInput.cpp b/src/cpp/CAudioInput.cpp
index d930137..201585f 100644 (file)
--- a/src/cpp/CAudioInput.cpp
+++ b/src/cpp/CAudioInput.cpp
@@ -31,9 +31,7 @@ using namespace tizen_media_audio;
  * class CAudioInput inherited by CAudioIO
  */
 CAudioInput::CAudioInput(CAudioInfo& info) :
-    CAudioIO(info),
-    __mIsUsedSyncRead(true),
-    __mIsInit(false) {
+    CAudioIO(info) {
     mDirection = CAudioInfo::EAudioDirection::AUDIO_DIRECTION_IN;
 }
 
@@ -83,27 +81,34 @@ void CAudioInput::initialize() {
         CAudioIO::initialize();
         __setInit(true);
     } catch (const CAudioError& e) {
+//LCOV_EXCL_START
         finalize();
         throw;
+//LCOV_EXCL_STOP
     }
 
+    CAudioIO::setState(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
     CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
 }
 
 void CAudioInput::finalize() {
     if (!__IsInit()) {
+//LCOV_EXCL_START
         AUDIO_IO_LOGD("Did not initialize");
         return;
+//LCOV_EXCL_STOP
     }
 
     CAudioIO::finalize();
-
     __setInit(false);
+    __mVolume = CAudioInfo::DEFAULT_RECORD_VOLUME;
 }
 
 void CAudioInput::prepare() {
+    CAudioTimedLocker locker(__func__, mMutex);
+
     if (!__IsInit())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput"); //LCOV_EXCL_LINE
 
     if (__IsReady()) {
         AUDIO_IO_LOGD("Already prepared CAudioInput");
@@ -128,33 +133,31 @@ void CAudioInput::prepare() {
 
         CPulseStreamSpec spec(streamSpec, mAudioInfo);
 
-        internalLock();
         mpPulseAudioClient = new CPulseAudioClient(CPulseAudioClient::EStreamDirection::STREAM_DIRECTION_RECORD, spec, this);
         mpPulseAudioClient->initialize();
+        mpPulseAudioClient->applyRecordVolume(__mVolume);
 #ifndef DISABLE_MOBILE_BACK_COMP
         /* Uncork stream which is created with CORKED flag */
         mpPulseAudioClient->cork(false);
 #endif
-        internalUnlock();
-
         CAudioIO::prepare();
     } catch (const CAudioError& e) {
+//LCOV_EXCL_START
         SAFE_FINALIZE(mpPulseAudioClient);
         SAFE_DELETE(mpPulseAudioClient);
-        internalUnlock();
         throw;
+//LCOV_EXCL_STOP
     } catch (const std::bad_alloc&) {
-//LCOV_EXCL_START
-        internalUnlock();
         THROW_ERROR_MSG(CAudioError::EError::ERROR_OUT_OF_MEMORY, "Failed to allocate CPulseAudioClient object");
-//LCOV_EXCL_STOP
     }
 }
 
 void CAudioInput::unprepare() {
+    CAudioTimedLocker locker(__func__, mMutex);
+
     if (!__IsInit())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
-                        "Did not initialize CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, //LCOV_EXCL_LINE
+                        "Did not initialize CAudioInput");          //LCOV_EXCL_LINE
 
     if (!__IsReady()) {
         AUDIO_IO_LOGD("Already unprepared");
@@ -163,64 +166,76 @@ void CAudioInput::unprepare() {
 
     CAudioIO::unprepare();
 
-    try {
-        internalLock();
-        if (mpPulseAudioClient && mpPulseAudioClient->isInThread())
-            THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't unprepare inside pulseaudio thread");
-        SAFE_FINALIZE(mpPulseAudioClient);
-        SAFE_DELETE(mpPulseAudioClient);
-        internalUnlock();
-    } catch (const CAudioError& e) {
-        internalUnlock();
-        throw;
-    }
+    if (mpPulseAudioClient && mpPulseAudioClient->isInThread())
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't unprepare inside pulseaudio thread");
+
+    SAFE_FINALIZE(mpPulseAudioClient);
+    SAFE_DELETE(mpPulseAudioClient);
+
+    CAudioIO::setState(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
+
+    locker.unlock();
 
     CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
 }
 
 void CAudioInput::pause() {
+   CAudioTimedLocker locker(__func__, mMutex);
+
     if (!__IsInit() || !__IsReady())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
-                        "Did not initialize or prepare CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,   //LCOV_EXCL_LINE
+                        "Did not initialize or prepare CAudioInput"); //LCOV_EXCL_LINE
 
     if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_STATE,
                         "Can't pause if not in Running state");
 
     if (mpPulseAudioClient->isInThread())
-        THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't pause in thread");
+        THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't pause in thread"); //LCOV_EXCL_LINE
 
     CAudioIO::pause();
+    CAudioIO::setState(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED);
+
+    locker.unlock();
+
     CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED);
 }
 
 void CAudioInput::resume() {
+    CAudioTimedLocker locker(__func__, mMutex);
+
     if (!__IsInit() || !__IsReady())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
-                        "Did not initialize or prepare CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,   //LCOV_EXCL_LINE
+                        "Did not initialize or prepare CAudioInput"); //LCOV_EXCL_LINE
 
     if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_STATE,
                         "Can't resume if not in Paused state");
 
     if (mpPulseAudioClient->isInThread())
-        THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't resume in thread");
+        THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't resume in thread"); //LCOV_EXCL_LINE
 
     CAudioIO::resume();
+    CAudioIO::setState(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING);
+
+    locker.unlock();
+
     CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING);
 }
 
 void CAudioInput::flush() {
+    CAudioTimedLocker locker(__func__, mMutex);
+
     if (!__IsInit() || !__IsReady())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
-                        "Did not initialize or prepare CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,   //LCOV_EXCL_LINE
+                        "Did not initialize or prepare CAudioInput"); //LCOV_EXCL_LINE
 
     CAudioIO::flush();
 }
 
 int CAudioInput::getBufferSize() {
     if (!__IsInit())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput"); //LCOV_EXCL_LINE
 
     /* FIXME : return calculated size here to satisfy backward compatibility */
     return (mAudioInfo.getSampleRate() * DEFAULT_PERIOD_SIZE) / 1000 * mAudioInfo.getSampleSize();
@@ -228,7 +243,7 @@ int CAudioInput::getBufferSize() {
 
 void CAudioInput::setStreamCallback(SStreamCallback callback) {
     if (!__IsInit())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput"); //LCOV_EXCL_LINE
 
     __mIsUsedSyncRead = (callback.onStream == nullptr);
 
@@ -238,9 +253,11 @@ void CAudioInput::setStreamCallback(SStreamCallback callback) {
 }
 
 size_t CAudioInput::read(void* buffer, size_t length) {
+    CAudioTimedLocker locker(__func__, mMutex);
+
     if (!__IsInit() || !__IsReady())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
-                        "Did not initialize or prepare CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,   //LCOV_EXCL_LINE
+                        "Did not initialize or prepare CAudioInput"); //LCOV_EXCL_LINE
 
     if (!buffer)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,
@@ -257,35 +274,28 @@ size_t CAudioInput::read(void* buffer, size_t length) {
 
     int ret = 0;
 
-    try {
-        internalLock();
-
-        // If another thread did call unprepare, do not read
-        if (!mpPulseAudioClient)
-            THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
-                            "Did not initialize CPulseAudioClient");
+    // If another thread did call unprepare, do not read
+    if (!mpPulseAudioClient)
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, //LCOV_EXCL_LINE
+                        "Did not initialize CPulseAudioClient");    //LCOV_EXCL_LINE
 
-        // Block until read done
-        ret = mpPulseAudioClient->read(buffer, length);
-        internalUnlock();
+    // Block until read done
+    ret = mpPulseAudioClient->read(buffer, length);
+    locker.unlock();
 
-        sched_yield();
-    } catch (const CAudioError& e) {
-        internalUnlock();
-        throw;
-    }
+    sched_yield();
 
     return ret;
 }
 
 int CAudioInput::peek(const void** buffer, size_t* length) {
     if (!__IsInit() || !__IsReady())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
-                        "Did not initialize or prepare CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,   //LCOV_EXCL_LINE
+                        "Did not initialize or prepare CAudioInput"); //LCOV_EXCL_LINE
 
     if (buffer == nullptr || length == nullptr)
-        THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,
-                               "Parameters are NULL buffer:%p, length:%p", buffer, length);
+        THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,                 //LCOV_EXCL_LINE
+                               "Parameters are NULL buffer:%p, length:%p", buffer, length); //LCOV_EXCL_LINE
 
     /* Checks synchronous flag */
     if (__mIsUsedSyncRead)
@@ -297,8 +307,8 @@ int CAudioInput::peek(const void** buffer, size_t* length) {
 
 int CAudioInput::drop() {
     if (!__IsInit() || !__IsReady())
-        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
-                        "Did not initialize or prepare CAudioInput");
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,   //LCOV_EXCL_LINE
+                        "Did not initialize or prepare CAudioInput"); //LCOV_EXCL_LINE
 
     /* Checks synchronous flag */
     if (__mIsUsedSyncRead)
@@ -307,3 +317,25 @@ int CAudioInput::drop() {
 
     return mpPulseAudioClient->drop();
 }
+
+void CAudioInput::setVolume(double volume) {
+    if (!__IsInit())
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Not initialized"); //LCOV_EXCL_LINE
+
+    if (__IsReady()) {
+        CAudioTimedLocker locker(__func__);
+        if (!mpPulseAudioClient->isInThread())
+            locker.lock(mMutex, mutex_timeout_s);
+
+        mpPulseAudioClient->applyRecordVolume(volume);
+    }
+
+    __mVolume = volume;
+}
+
+double CAudioInput::getVolume() {
+    if (!__IsInit())
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Not initialized"); //LCOV_EXCL_LINE
+
+    return __mVolume;
+}