DSLOG: remove new lines 04/241604/1
authorSooChan Lim <sc1.lim@samsung.com>
Fri, 10 Jul 2020 04:00:20 +0000 (13:00 +0900)
committerSung-Jin Park <sj76.park@samsung.com>
Thu, 20 Aug 2020 09:46:23 +0000 (18:46 +0900)
Change-Id: I027f45c33f5e56f7ee120079c5d72fc0fcd0155c

src/DSBuffer/DSBufferQueueTBMImpl.cpp
src/DSBuffer/DSBufferTBMImpl.cpp
src/DSDisplayDevice/DSDisplayDeviceHWCTDMImpl.cpp
src/DSDisplayDevice/DSDisplayDeviceOutputTDMImpl.cpp
src/DSDisplayDevice/DSDisplayDeviceTDMImpl.cpp
src/DSRender/DSRenderEngineEcoreEvasImpl.cpp
tests/DSProperty-test.cpp
tests/DSRefBase-test.cpp
tests/DSSignal-test.cpp

index d4e35d8..08eb77e 100644 (file)
@@ -14,7 +14,7 @@ DSBufferQueueTBMImpl::DSBufferQueueTBMImpl(int slotSize, int bufferWidth, int bu
 {
        __tqueue = tbm_surface_queue_create(__slotSize, __bufferWidth, __bufferHeight, DSBufferTBMImpl::getTBMFormat(__format), TBM_BO_SCANOUT);
        if (!__tqueue) {
-               DSLOG_ERR("DSBufferTBM", "tbm_surface_queue_create fails.\n");
+               DSLOG_ERR("DSBufferTBM", "tbm_surface_queue_create fails.");
        }
 }
 
@@ -54,7 +54,7 @@ std::shared_ptr<IDSBuffer> DSBufferQueueTBMImpl::dequeueBuffer()
 
        terror = tbm_surface_queue_dequeue(__tqueue, &tsurface);
        if (terror != TBM_SURFACE_QUEUE_ERROR_NONE) {
-               DSLOG_ERR("BuffeQueueTBM", "tbm_surface_queue_dequeue fails.\n");
+               DSLOG_ERR("BuffeQueueTBM", "tbm_surface_queue_dequeue fails.");
                return nullptr;
        }
 
index 9a3dc9c..bf17ae6 100644 (file)
@@ -11,7 +11,7 @@ DSBufferTBMImpl::DSBufferTBMImpl(int width, int height, IDSBuffer::Format format
 {
        __tsurface = tbm_surface_create(__width, __height, getTBMFormat(__format));
        if (!__tsurface) {
-               DSLOG_ERR("DSBufferTBM", "tbm_surface_create fails.\n");
+               DSLOG_ERR("DSBufferTBM", "tbm_surface_create fails.");
        }
 }
 
@@ -41,7 +41,7 @@ tbm_format DSBufferTBMImpl::getTBMFormat(const IDSBuffer::Format &format)
                        tformat = TBM_FORMAT_ARGB8888;
                        break;
                default:
-                       DSLOG_ERR("DSBufferTBM", "(%d) format is not supported.\n", format);
+                       DSLOG_ERR("DSBufferTBM", "(%d) format is not supported.", format);
        }
 
        return tformat;
@@ -56,7 +56,7 @@ IDSBuffer::Format DSBufferTBMImpl::getBufferFormat(const tbm_format &tformat)
                        format = IDSBuffer::Format::FORMAT_ARGB8888 ;
                        break;
                default:
-                       DSLOG_ERR("DSBufferTBM", "(%d) format is not supported.\n", tformat);
+                       DSLOG_ERR("DSBufferTBM", "(%d) format is not supported.", tformat);
        }
 
        return format;
index f7b1cce..fea17fd 100644 (file)
@@ -29,7 +29,7 @@ std::shared_ptr<IDSBufferQueue> DSDisplayDeviceHWCTDMImpl::getTargetBufferQueue(
 
        tqueue = tdm_hwc_get_client_target_buffer_queue(__thwc, &terror);
        if (terror != TDM_ERROR_NONE) {
-               DSLOG_ERR("HWCTDM", "tdm_hwc_get_client_target_buffer_queue fails.\n");
+               DSLOG_ERR("HWCTDM", "tdm_hwc_get_client_target_buffer_queue fails.");
        }
 
        __bufferQueue = std::make_shared<DSBufferQueueTBMImpl>(tqueue);
@@ -60,13 +60,13 @@ std::shared_ptr<IDSDisplayDeviceHWCWindow> DSDisplayDeviceHWCTDMImpl::makeHWCWin
 
        twindow = tdm_hwc_create_window(__thwc, &terror);
        if (!twindow) {
-               DSLOG_ERR("HWCTDM", "tdm_hwc_create_window fails.\n");
+               DSLOG_ERR("HWCTDM", "tdm_hwc_create_window fails.");
                return nullptr;
        }
 
        deviceHWCWindow = std::make_shared<DSDisplayDeviceHWCWindowTDMImpl>(twindow);
        if (!deviceHWCWindow) {
-               DSLOG_ERR("HWCTDM", "new DSDisplayDeviceHWCWindowTDMImpl fails.\n");
+               DSLOG_ERR("HWCTDM", "new DSDisplayDeviceHWCWindowTDMImpl fails.");
                return nullptr;
        }
 
@@ -98,19 +98,19 @@ bool DSDisplayDeviceHWCTDMImpl::commit()
        uint32_t numChanges;
 
        if (!__validate(numChanges)) {
-               DSLOG_ERR("HWCTDM", "__validate fails.\n");
+               DSLOG_ERR("HWCTDM", "__validate fails.");
                return false;
        }
 
        if (numChanges > 0) {
                if (!__updateChanges(numChanges)) {
-                       DSLOG_ERR("HWCTDM", "__updateChanges fails.\n");
+                       DSLOG_ERR("HWCTDM", "__updateChanges fails.");
                        return false;
                }
        }
 
        if (!__acceptValidation()) {
-               DSLOG_ERR("HWCTDM", "__acceptValidation fails.\n");
+               DSLOG_ERR("HWCTDM", "__acceptValidation fails.");
                return false;
        }
 
@@ -202,7 +202,7 @@ bool DSDisplayDeviceHWCTDMImpl::__acceptValidation()
        // TODO: accept_validation is depending on the transitions.
        terror =  tdm_hwc_accept_validation(__thwc);
        if (terror != TDM_ERROR_NONE) {
-               DSLOG_ERR("TDM_HWC", "tdm_hwc_accept_validation fails.\n");
+               DSLOG_ERR("TDM_HWC", "tdm_hwc_accept_validation fails.");
                return false;
        }
 
index 21d1e3e..4ef1821 100644 (file)
@@ -98,7 +98,7 @@ bool DSDisplayDeviceOutputTDMImpl::setMode(std::shared_ptr<IDSDisplayDeviceOutpu
 
        terror = tdm_output_set_mode(__toutput, ttmode);
        if (terror != TDM_ERROR_NONE) {
-               DSLOG_ERR("TDM OUTPUT", "tdm_output_set_mode fails.\n");
+               DSLOG_ERR("TDM OUTPUT", "tdm_output_set_mode fails.");
                return false;
        }
 
@@ -134,7 +134,7 @@ void DSDisplayDeviceOutputTDMImpl::__initializeConnectorType()
 
        terror = tdm_output_get_output_type(__toutput, &ttype);
        if (terror != TDM_ERROR_NONE) {
-               DSLOG_ERR("TDM OUTPUT", "tdm_output_get_output_type fails.\n");
+               DSLOG_ERR("TDM OUTPUT", "tdm_output_get_output_type fails.");
                return;
        }
 
@@ -183,7 +183,7 @@ void DSDisplayDeviceOutputTDMImpl::__updateConnectState()
 
        terror = tdm_output_get_conn_status(__toutput, &tstatus);
        if (terror != TDM_ERROR_NONE) {
-               DSLOG_ERR("TDM OUTPUT", "tdm_output_get_conn_status fails.\n");
+               DSLOG_ERR("TDM OUTPUT", "tdm_output_get_conn_status fails.");
                return;
        }
 
@@ -204,7 +204,7 @@ void DSDisplayDeviceOutputTDMImpl::__updatePhysicalMMSize()
 
        terror = tdm_output_get_physical_size(__toutput, &__physicalMMWidth, &__physicalMMHeight);
        if (terror != TDM_ERROR_NONE) {
-               DSLOG_ERR("TDM OUTPUT", "tdm_output_get_physical_size fails.\n");
+               DSLOG_ERR("TDM OUTPUT", "tdm_output_get_physical_size fails.");
                __physicalMMWidth = 0;
                __physicalMMHeight = 0;
        }
@@ -226,7 +226,7 @@ void DSDisplayDeviceOutputTDMImpl::__updateAvailableModeList()
 
        terror = tdm_output_get_available_modes(__toutput, &tmodes, &count);
        if (terror != TDM_ERROR_NONE) {
-               DSLOG_ERR("TDM OUTPUT", "tdm_output_get_available_modes fails.\n");
+               DSLOG_ERR("TDM OUTPUT", "tdm_output_get_available_modes fails.");
                return;
        }
 
index c950704..6f304ca 100644 (file)
@@ -14,19 +14,19 @@ DSDisplayDeviceTDMImpl::DSDisplayDeviceTDMImpl()
 
        __tdisplay = tdm_display_init(&terror);
        if (terror != TDM_ERROR_NONE) {
-               DSLOG_ERR("TDM DEVICE", "tdm_display_init fails.\n");
+               DSLOG_ERR("TDM DEVICE", "tdm_display_init fails.");
        }
 
        terror = tdm_display_get_output_count(__tdisplay, &__numOutputs);
        if (__numOutputs <= 0) {
-               DSLOG_ERR("TDM DEVICE", "tdm_display_get_output_count fails.\n");
+               DSLOG_ERR("TDM DEVICE", "tdm_display_get_output_count fails.");
                return;
        }
 
        for (int i = 0; i < __numOutputs; ++i) {
                toutput = tdm_display_get_output(__tdisplay, i, &terror);
                if (terror != TDM_ERROR_NONE) {
-                       DSLOG_ERR("TDM DEVICE", "tdm_display_get_output fails.(output num: %d)\n", i);
+                       DSLOG_ERR("TDM DEVICE", "tdm_display_get_output fails.(output num: %d)", i);
                        return;
                }
 
index dc4551e..82bf8d1 100644 (file)
@@ -12,11 +12,11 @@ DSRenderEngineEcoreEvasImpl::DSRenderEngineEcoreEvasImpl(std::shared_ptr<IDSBuff
          __ee{nullptr}
 {
        if (!evas_init()) {
-               DSLOG_ERR("RENDER_ENGINE_ECORE_EVAS", "evas_init fails\n");
+               DSLOG_ERR("RENDER_ENGINE_ECORE_EVAS", "evas_init fails");
        }
 
        if (!ecore_evas_init()) {
-               DSLOG_ERR("RENDER_ENGINE_ECORE_EVAS", "ecore_evas_init fails\n");
+               DSLOG_ERR("RENDER_ENGINE_ECORE_EVAS", "ecore_evas_init fails");
        }
 
        auto alloc_func = [](void *data, int w, int h) -> void * {
@@ -30,7 +30,7 @@ DSRenderEngineEcoreEvasImpl::DSRenderEngineEcoreEvasImpl(std::shared_ptr<IDSBuff
 
        __ee = ecore_evas_tbm_allocfunc_new("gl_tbm", __bufferWidth, __bufferHeight, alloc_func, free_func, (void *)bufferQueue.get());
        if (!__ee) {
-               DSLOG_ERR("RENDER_ENGINE_ECORE_EVAS", "ecore_evas_tbm_allocfunc_new fails.\n");
+               DSLOG_ERR("RENDER_ENGINE_ECORE_EVAS", "ecore_evas_tbm_allocfunc_new fails.");
        }
 
        ecore_evas_manual_render_set(__ee, 1);
index 794ef07..3546536 100644 (file)
@@ -7,13 +7,9 @@ class DSPropertyTest : public ::testing::Test
 {
 public:
        void SetUp(void) override
-       {
-               std::cout << "setup DSPropertyTest\n";
-       }
+       {}
        void TearDown(void) override
-       {
-               std::cout << "teardown DSPropertyTest\n";
-       }
+       {}
 };
 
 TEST_F(DSPropertyTest, CreateProperty)
index 086ea6b..4fd582c 100644 (file)
@@ -7,13 +7,9 @@ class DSRefBaseTest : public ::testing::Test
 {
 public:
        void SetUp(void) override
-       {
-               std::cout << "setup\n";
-       }
+       {}
        void TearDown(void) override
-       {
-               std::cout << "teardown\n";
-       }
+       {}
 };
 
 TEST_F(DSRefBaseTest, CreateRefBase)
index 7348b3f..8a8935f 100644 (file)
@@ -46,11 +46,11 @@ public:
        ~ReceiverMock() = default;
 
        void slotA(int value) {
-               DSLOG_INF("DSTEST", "%s is invoked! : value, %d\n", __func__, value);
+               DSLOG_INF("DSTEST", "%s is invoked! : value, %d", __func__, value);
        }
 
        void slotB(double value) {
-               DSLOG_INF("DSTEST", "%s is invoked! : value, %lf\n", __func__, value);
+               DSLOG_INF("DSTEST", "%s is invoked! : value, %lf", __func__, value);
        }
 
 private: