Should there be a patch 5 which makes vl_compositor_init_state return void?
On Thu, Dec 15, 2016 at 8:34 AM, Nayan Deshmukh
<nayan26deshm...@gmail.com> wrote:
> vl_compositor_init_state always return true so we don't
> need to handle false case
>
> Signed-off-by: Nayan Deshmukh <nayan26deshm...@gmail.com>
> ---
> src/gallium/state_trackers/xvmc/context.c | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/src/gallium/state_trackers/xvmc/context.c
> b/src/gallium/state_trackers/xvmc/context.c
> index 469e1c1..4ae7d7b 100644
> --- a/src/gallium/state_trackers/xvmc/context.c
> +++ b/src/gallium/state_trackers/xvmc/context.c
> @@ -262,11 +262,7 @@ Status XvMCCreateContext(Display *dpy, XvPortID port,
> int surface_type_id,
> goto no_compositor;
> }
>
> - if (!vl_compositor_init_state(&context_priv->cstate, pipe)) {
> - XVMC_MSG(XVMC_ERR, "[XvMC] Could not create VL compositor state.\n");
> - goto no_compositor_state;
> - }
> -
> + vl_compositor_init_state(&context_priv->cstate, pipe)
>
> context_priv->color_standard =
> debug_get_bool_option("G3DVL_NO_CSC", FALSE) ?
> @@ -299,8 +295,6 @@ Status XvMCCreateContext(Display *dpy, XvPortID port, int
> surface_type_id,
>
> return Success;
>
> -no_compositor_state:
> - vl_compositor_cleanup(&context_priv->compositor);
> no_compositor:
> context_priv->decoder->destroy(context_priv->decoder);
> no_decoder:
> --
> 2.9.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
