From: Boram Park <boram1288.park@samsung.com>
Date: Mon, 9 Apr 2018 11:43:57 +0000 (+0900)
Subject: common: remove unuseful variable
X-Git-Tag: accepted/tizen/unified/20180412.073819~5
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Flibtdm.git;a=commitdiff_plain;h=3ad3e82dc0bc043697fc72d34c5cf7d051b69af4

common: remove unuseful variable

Change-Id: I1f47f58fb65b5b18f08e64b790e11c4c195e1c32
---

diff --git a/src/tdm.c b/src/tdm.c
index 15840f4..1a84fc0 100644
--- a/src/tdm.c
+++ b/src/tdm.c
@@ -477,19 +477,17 @@ tdm_display_update_output(tdm_private_module *private_module,
 	return TDM_ERROR_NONE;
 }
 
-static tdm_output **
+static void
 _tdm_display_set_main_first(tdm_output **outputs, int index)
 {
 	tdm_output *output_tmp = NULL;
 
 	if (index == 0)
-		return outputs;
+		return;
 
 	output_tmp = outputs[0];
 	outputs[0] = outputs[index];
 	outputs[index] = output_tmp;
-
-	return outputs;
 }
 
 static tdm_output **
@@ -497,7 +495,6 @@ _tdm_display_get_ordered_outputs(tdm_private_module *private_module, int *count)
 {
 	tdm_func_display *func_display = &private_module->func_display;
 	tdm_output **outputs = NULL;
-	tdm_output **new_outputs = NULL;
 	tdm_output *output_dsi = NULL;
 	tdm_output *output_lvds = NULL;
 	tdm_output *output_hdmia = NULL;
@@ -580,32 +577,28 @@ _tdm_display_get_ordered_outputs(tdm_private_module *private_module, int *count)
 	if (output_connected_count == 0) {
 		/* hdmi > dsi > lvds > else */
 		if (output_hdmia != NULL)
-			new_outputs = _tdm_display_set_main_first(outputs, index_hdmia);
+			_tdm_display_set_main_first(outputs, index_hdmia);
 		else if (output_hdmib != NULL)
-			new_outputs = _tdm_display_set_main_first(outputs, index_hdmib);
+			_tdm_display_set_main_first(outputs, index_hdmib);
 		else if (output_dsi != NULL)
-			new_outputs = _tdm_display_set_main_first(outputs, index_dsi);
+			_tdm_display_set_main_first(outputs, index_dsi);
 		else if (output_lvds != NULL)
-			new_outputs = _tdm_display_set_main_first(outputs, index_lvds);
-		else
-			new_outputs = outputs;
+			_tdm_display_set_main_first(outputs, index_lvds);
 	} else { /* (output_connected_count > 1) */
 		/* dsi > lvds > hdmi > else */
 		if (output_dsi != NULL)
-			new_outputs = _tdm_display_set_main_first(outputs, index_dsi);
+			_tdm_display_set_main_first(outputs, index_dsi);
 		else if (output_lvds != NULL)
-			new_outputs = _tdm_display_set_main_first(outputs, index_lvds);
+			_tdm_display_set_main_first(outputs, index_lvds);
 		else if (output_hdmia != NULL)
-			new_outputs = _tdm_display_set_main_first(outputs, index_hdmia);
+			_tdm_display_set_main_first(outputs, index_hdmia);
 		else if (output_hdmib != NULL)
-			new_outputs = _tdm_display_set_main_first(outputs, index_hdmib);
-		else
-			new_outputs = outputs;
+			_tdm_display_set_main_first(outputs, index_hdmib);
 	}
 
-	private_module->outputs = new_outputs;
+	private_module->outputs = outputs;
 
-	return new_outputs;
+	return outputs;
 
 failed_get_outputs:
 	free(outputs);