Filip Roséen authored Mar 02, 2017 and Jean-Baptiste Kempf committed Mar 03, 2017

Hi `vlc-devel`,

On 2017-03-01 09:52, Filip Roséen wrote:

> diff --git a/src/input/control.c b/src/input/control.c
> index f4781eb1a5..ff6df4c2fe 100644
> --- a/src/input/control.c
> +++ b/src/input/control.c
> @@ -399,7 +399,7 @@ int input_vaControl( input_thread_t *p_input, int i_query, va_list args )
>                  return VLC_EGENERIC;
>              }
>
> -            input_title_t *p_title = vlc_input_title_Duplicate( priv->title[*pi_title_to_fetch] );
> +            input_title_t *p_title = priv->title[*pi_title_to_fetch];

I just noticed that this patch results in a warning due to
`input_thread_private_t::title` being *const-qualified*, and `p_title`
lacking such qualifier.

See attached patch for a fixup of [`d1feba6e`][1].

Best Regards,\
Filip

[1]: http://git.videolan.org/?p=vlc.git;a=commit;h=d1feba6e5f90e48e41cc507ab574f3a1091047f5



>From dc5e79e1c83075522e1ef21318ea807a3946d5f9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Filip=20Ros=C3=A9en?= <filip@atch.se>
Date: Thu, 2 Mar 2017 16:39:38 +0100
Subject: [PATCH] input/control: INPUT_GET_SEEKPOINTS: apply const to silence
 warning

priv->title[n] results in a pointer-to-const, which means that the
initialization of p_title discards the const qualifier; this addition
fixes that warning while also making the implementation more correct.

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>