[FFmpeg-devel] [PATCH 1/2] configure: support --disable-sdl

Reimar Döffinger Reimar.Doeffinger at gmx.de
Tue Sep 23 23:32:08 CEST 2014

On 23.09.2014, at 02:27, Michael Niedermayer <michaelni at gmx.at> wrote:

> On Tue, Sep 23, 2014 at 01:01:26AM +0200, Reimar Döffinger wrote:
>> On Tue, Sep 23, 2014 at 12:42:53AM +0200, Michael Niedermayer wrote:
>>> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
>> Maybe more readable if you do reindentation separately?
> yes, will do
>> Because I was going to suggest to use elif until I saw that that part
>> is not actually used.
>> Also doesn't this patch also add --enable-sdl? Seems it will be ignored,
>> shouldn't we rather fail if --enable-sdl was given but we do not find
>> it? I believe that is how FFmpeg configure normally behaves.
> posted a seperate patch for that

I _think_ they are all ok, but I did only a quick review and no testing.

More information about the ffmpeg-devel mailing list