[Mesa-dev] [PATCH 2/4] st/vdpau: remove dead code

Nayan Deshmukh nayan26deshmukh at gmail.com
Thu Dec 15 13:34:12 UTC 2016


vl_compositor_init_state always return true so we don't
need to handle false case

Signed-off-by: Nayan Deshmukh <nayan26deshmukh at gmail.com>
---
We may as well make vl_compositor_init_state a void function
but I don't know the reason as to why it is there in first 
place so haven't touched it yet
---
 src/gallium/state_trackers/vdpau/presentation.c | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/src/gallium/state_trackers/vdpau/presentation.c b/src/gallium/state_trackers/vdpau/presentation.c
index f35d73a..980d5cb 100644
--- a/src/gallium/state_trackers/vdpau/presentation.c
+++ b/src/gallium/state_trackers/vdpau/presentation.c
@@ -66,11 +66,7 @@ vlVdpPresentationQueueCreate(VdpDevice device,
    pq->drawable = pqt->drawable;
 
    pipe_mutex_lock(dev->mutex);
-   if (!vl_compositor_init_state(&pq->cstate, dev->context)) {
-      pipe_mutex_unlock(dev->mutex);
-      ret = VDP_STATUS_ERROR;
-      goto no_compositor;
-   }
+   vl_compositor_init_state(&pq->cstate, dev->context)
    pipe_mutex_unlock(dev->mutex);
 
    *presentation_queue = vlAddDataHTAB(pq);
@@ -82,7 +78,6 @@ vlVdpPresentationQueueCreate(VdpDevice device,
    return VDP_STATUS_OK;
 
 no_handle:
-no_compositor:
    DeviceReference(&pq->device, NULL);
    FREE(pq);
    return ret;
-- 
2.9.3



More information about the mesa-dev mailing list