[vlc-devel] [PATCH] input: fix the rate sign check
Thomas Guillem
thomas at gllm.fr
Wed Jun 5 13:59:10 CEST 2019
LGTM.
On Wed, Jun 5, 2019, at 13:55, Steve Lhomme wrote:
> fabsf( param.val.f_float ) cannot be negative.
>
> Compilers should warn about this.
> ---
> src/input/input.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/input/input.c b/src/input/input.c
> index a6116ab8e7..97ee9a6bb6 100644
> --- a/src/input/input.c
> +++ b/src/input/input.c
> @@ -1920,7 +1920,7 @@ static bool Control( input_thread_t *p_input,
> {
> /* Get rate and direction */
> float rate = fabsf( param.val.f_float );
> - int i_rate_sign = rate < 0 ? -1 : 1;
> + int i_rate_sign = param.val.f_float < 0 ? -1 : 1;
>
> /* Check rate bound */
> if( rate > INPUT_RATE_MAX )
> --
> 2.17.1
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
More information about the vlc-devel
mailing list