[FFmpeg-devel] [PATCH v2 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

[Vittorio Gambaletta (VittGam)]

Hi,

It seems to me that whitespace was right on my end in the other
patches...

This time I've literally piped the mails to my mailserver using
netcat, like I always do with OpenWrt patches anyway, so please
check if I'm still getting something wrong with whitespace now.

Thank you,
Vittorio

On 16/01/2016 22:41:30 CET, Vittorio Gambaletta (VittGam) wrote:
> Seeking by clicking on the video window can be annoying, because
> the user might click on it accidentally while eg. trying to get
> focus on it, and ffplay seeks instead.
>
> This commit changes that behaviour to seek only when the right
> mouse button is used to click and drag on the window.
>
> Signed-off-by: Vittorio Gambaletta <ffmpeg-dev at vittgam.net>
>
> ---
>  ffplay.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/ffplay.c b/ffplay.c
> index d2e3dc6..2fa7165 100644
> --- a/ffplay.c
> +++ b/ffplay.c
> @@ -3480,9 +3480,11 @@ static void event_loop(VideoState *cur_stream)
>              }
>              cursor_last_shown = av_gettime_relative();
>              if (event.type == SDL_MOUSEBUTTONDOWN) {
> +                if (event.button.button != SDL_BUTTON_RIGHT)
> +                    break;
>                  x = event.button.x;
>              } else {
> -                if (event.motion.state != SDL_PRESSED)
> +                if (!(event.motion.state & SDL_BUTTON_RMASK))
>                      break;
>                  x = event.motion.x;
>              }
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel