[PATCH weston 1/7] compositor-drm: remove superfluos get_disable_state call

Emre Ucan eucan at de.adit-jv.com
Tue Mar 20 14:28:21 UTC 2018


drm_output_get_disable_state function returns
a duplicated output_state object.

Here we are creating the object, but we are
never using it. Therefore, it is safe to remove.

(Found by clang source code analyzer)

Signed-off-by: Emre Ucan <eucan at de.adit-jv.com>
---
 libweston/compositor-drm.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c
index 81ca67d..f8c13ee 100644
--- a/libweston/compositor-drm.c
+++ b/libweston/compositor-drm.c
@@ -3860,7 +3860,6 @@ drm_set_dpms(struct weston_output *output_base, enum dpms_enum level)
 		state = drm_pending_state_get_output(pending_state, output);
 		if (state)
 			drm_output_state_free(state);
-		state = drm_output_get_disable_state(pending_state, output);
 		return;
 	}
 
-- 
2.7.4



More information about the wayland-devel mailing list