[FFmpeg-devel] [PATCH] ffplay: properly detect all window size changes

[Marton Balint]

On Wed, 14 Aug 2019, Reino Wijnsma wrote:

> On 2019-08-13T23:20:08+0200, Marton Balint <cus at passwd.hu> wrote:
>> SDL_WINDOWEVENT_SIZE_CHANGED should be used instead of SDL_WINDOWEVENT_RESIZED
>> because SDL_WINDOWEVENT_RESIZED is only emitted if the resize happened due to
>> an external event.
>>
>> Fixes ticket #8072.
>>
>> Additional references:
>> https://bugzilla.libsdl.org/show_bug.cgi?id=4760
>> https://wiki.libsdl.org/SDL_WindowEventID
>>
>> Signed-off-by: Marton Balint <cus at passwd.hu>
>> ---
>>  fftools/ffplay.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/fftools/ffplay.c b/fftools/ffplay.c
>> index 8fb8faeb06..fee0619f7c 100644
>> --- a/fftools/ffplay.c
>> +++ b/fftools/ffplay.c
>> @@ -3436,7 +3436,7 @@ static void event_loop(VideoState *cur_stream)
>>              break;
>>          case SDL_WINDOWEVENT:
>>              switch (event.window.event) {
>> -                case SDL_WINDOWEVENT_RESIZED:
>> +                case SDL_WINDOWEVENT_SIZE_CHANGED:
>>                      screen_width  = cur_stream->width  = event.window.data1;
>>                      screen_height = cur_stream->height = event.window.data2;
>>                      if (cur_stream->vis_texture) {
>
> Applied patch and compiled ffplay locally. I can confirm this patch fixes ticket #8072.

Thanks for testing, pushed to git master and 4.2 release branch.

Regards,
Marton