[FFmpeg-devel] [PATCH 0/4] ffplay and lavd SDL2 set

James Almer jamrial at gmail.com
Thu Sep 15 01:56:40 EEST 2016


On 9/14/2016 7:44 PM, Thomas Volkert wrote:
> On 15.09.2016 00:27, Josh de Kock wrote:
>> Hi,
>>
>> Resending this set with ffplay now having two versions, a SDL2 and a
>> SDL1 version. I've integrated all comments up until now (hopefully).
>>
>> Josh
>>
>> Josh de Kock (3):
>>    lavd: Add SDL2 output device
>>    ffplay: make copy for SDL1
>>    MAINTAINERS: update my entries
>>
>> Marton Balint (1):
>>    ffplay: add SDL2 support
>>
>>   MAINTAINERS               |   4 +-
>>   Makefile                  |   1 +
>>   configure                 |  35 ++-
>>   ffplay.c                  | 599 +++++++++++++++++++---------------------------
>>   ffplay.c => ffplay_sdl1.c |   1 -
>>
> 
> Maintaining two versions of ffplay sounds as too much overhead to me. Is it not possible to add some more code abstraction or even some simple #ifdef constructs to support SDL1 as well as SDL2 in only one ffplay version in parallel?
> So, the actually used SDL version could be selected during the configure step. After some months, the (hopefully) deprecated SDL1 code could be dropped...
> 
> Best regards,
> Thomas.

This is not two separate ffplay programs. It's exactly what you said
except using two separate source files instead of #ifdeffery in a
single source file.

Look at patches 2 and 3 in the set.


More information about the ffmpeg-devel mailing list