From 2c1a0a98fccce08d68f9328fd954f2ac9437600c Mon Sep 17 00:00:00 2001
From: SooChan Lim <sc1.lim@samsung.com>
Date: Fri, 10 Jul 2020 13:00:20 +0900
Subject: [PATCH] DSLOG: remove new lines

Change-Id: I027f45c33f5e56f7ee120079c5d72fc0fcd0155c
---
 src/DSBuffer/DSBufferQueueTBMImpl.cpp                |  4 ++--
 src/DSBuffer/DSBufferTBMImpl.cpp                     |  6 +++---
 src/DSDisplayDevice/DSDisplayDeviceHWCTDMImpl.cpp    | 14 +++++++-------
 src/DSDisplayDevice/DSDisplayDeviceOutputTDMImpl.cpp | 10 +++++-----
 src/DSDisplayDevice/DSDisplayDeviceTDMImpl.cpp       |  6 +++---
 src/DSRender/DSRenderEngineEcoreEvasImpl.cpp         |  6 +++---
 tests/DSProperty-test.cpp                            |  8 ++------
 tests/DSRefBase-test.cpp                             |  8 ++------
 tests/DSSignal-test.cpp                              |  4 ++--
 9 files changed, 29 insertions(+), 37 deletions(-)

diff --git a/src/DSBuffer/DSBufferQueueTBMImpl.cpp b/src/DSBuffer/DSBufferQueueTBMImpl.cpp
index d4e35d8..08eb77e 100644
--- a/src/DSBuffer/DSBufferQueueTBMImpl.cpp
+++ b/src/DSBuffer/DSBufferQueueTBMImpl.cpp
@@ -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;
 	}
 
diff --git a/src/DSBuffer/DSBufferTBMImpl.cpp b/src/DSBuffer/DSBufferTBMImpl.cpp
index 9a3dc9c..bf17ae6 100644
--- a/src/DSBuffer/DSBufferTBMImpl.cpp
+++ b/src/DSBuffer/DSBufferTBMImpl.cpp
@@ -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;
diff --git a/src/DSDisplayDevice/DSDisplayDeviceHWCTDMImpl.cpp b/src/DSDisplayDevice/DSDisplayDeviceHWCTDMImpl.cpp
index f7b1cce..fea17fd 100644
--- a/src/DSDisplayDevice/DSDisplayDeviceHWCTDMImpl.cpp
+++ b/src/DSDisplayDevice/DSDisplayDeviceHWCTDMImpl.cpp
@@ -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;
 	}
 
diff --git a/src/DSDisplayDevice/DSDisplayDeviceOutputTDMImpl.cpp b/src/DSDisplayDevice/DSDisplayDeviceOutputTDMImpl.cpp
index 21d1e3e..4ef1821 100644
--- a/src/DSDisplayDevice/DSDisplayDeviceOutputTDMImpl.cpp
+++ b/src/DSDisplayDevice/DSDisplayDeviceOutputTDMImpl.cpp
@@ -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;
 	}
 
diff --git a/src/DSDisplayDevice/DSDisplayDeviceTDMImpl.cpp b/src/DSDisplayDevice/DSDisplayDeviceTDMImpl.cpp
index c950704..6f304ca 100644
--- a/src/DSDisplayDevice/DSDisplayDeviceTDMImpl.cpp
+++ b/src/DSDisplayDevice/DSDisplayDeviceTDMImpl.cpp
@@ -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;
 		}
 
diff --git a/src/DSRender/DSRenderEngineEcoreEvasImpl.cpp b/src/DSRender/DSRenderEngineEcoreEvasImpl.cpp
index dc4551e..82bf8d1 100644
--- a/src/DSRender/DSRenderEngineEcoreEvasImpl.cpp
+++ b/src/DSRender/DSRenderEngineEcoreEvasImpl.cpp
@@ -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);
diff --git a/tests/DSProperty-test.cpp b/tests/DSProperty-test.cpp
index 794ef07..3546536 100644
--- a/tests/DSProperty-test.cpp
+++ b/tests/DSProperty-test.cpp
@@ -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)
diff --git a/tests/DSRefBase-test.cpp b/tests/DSRefBase-test.cpp
index 086ea6b..4fd582c 100644
--- a/tests/DSRefBase-test.cpp
+++ b/tests/DSRefBase-test.cpp
@@ -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)
diff --git a/tests/DSSignal-test.cpp b/tests/DSSignal-test.cpp
index 7348b3f..8a8935f 100644
--- a/tests/DSSignal-test.cpp
+++ b/tests/DSSignal-test.cpp
@@ -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:
-- 
2.7.4