[FFmpeg-devel] [PATCH 2/2] avdevice/sdl2 : add option to set window position

[Gyan]

On Tue, Oct 2, 2018 at 2:47 AM Dave Rice <dave at dericed.com> wrote:

> Allows arrangement of multiple windows such as:
> ffmpeg -re -f lavfi -i mandelbrot -f sdl -window_x 1 -window_y 1
> mandelbrot -vf waveform,format=yuv420p -f sdl -window_x 641 -window_y 1
> waveform -vf vectorscope,format=yuv420p -f sdl -window_x 1 -window_y 481
> vectorscop
>
> From 00438983c96b5db227b9975a2c160fc6aac5219d Mon Sep 17 00:00:00 2001
> From: Dave Rice <dave at dericed.com>
> Date: Mon, 1 Oct 2018 17:08:35 -0400
> Subject: [PATCH 2/2] avdevice/sdl2 : add option to set window position
>
> +    if (!sdl->window_x)
> +        sdl->window_x = SDL_WINDOWPOS_CENTERED;
> +    if (!sdl->window_y)
> +        sdl->window_y = SDL_WINDOWPOS_CENTERED;
> +    SDL_SetWindowPosition(sdl->window, sdl->window_x, sdl->window_y);
>

What happens if the user value implies fully or partially out-of-canvas
rendering?

For the former case, we may want to print a warning and reposition the
window.
If a partial window is drawable, then negative values can be valid and the
lower range bound should be INT_MIN
Also, the user can't position a window at top-left (0,0), so the default
should probably be INT_MAX.

Gyan