[FFmpeg-devel] [PATCH] configure: add -Wstrict-prototypes when available

Ganesh Ajjanagadde gajjanag at mit.edu
Fri Oct 23 15:12:35 CEST 2015


On Fri, Oct 23, 2015 at 8:49 AM, Ganesh Ajjanagadde <gajjanag at mit.edu> wrote:
> On Thu, Oct 22, 2015 at 2:50 PM, Ganesh Ajjanagadde <gajjanag at mit.edu> wrote:
>> On Thu, Oct 22, 2015 at 2:45 PM, Hendrik Leppkes <h.leppkes at gmail.com> wrote:
>>> On Thu, Oct 22, 2015 at 8:40 PM, Ganesh Ajjanagadde
>>> <gajjanagadde at gmail.com> wrote:
>>>> GCC (and Clang) have this useful warning that is not enabled by -Wall or
>>>> -Wextra. This will ensure that issues like those fixed in
>>>> 4da52e3630343e8d3a79aef2cafcb6bf0b71e8da
>>>> will trigger warnings.
>>>>
>>>
>>> I did get this warning on GCC 5.2, which is how I found it in the
>>> first place. :)
>>> Seems to be enabled by default here at least.
>>
>> Strange, I also run GCC 5.2 and it is not enabled by default, or even
>> with "-Wall -Wextra -pedantic".
>>
>>>
>>> Otherwise probably OK if its not enabled on other compilers that
>>> understand such a flag.
>>
>> Will wait to see if there are issues with some less-used compilers.
>
> In the absence of comments, and low probability of breakage
> (check_cflags enables only when available), pushed.

Missed it, it is already enabled. Reverted.

>
>>
>>>
>>> - Hendrik
>>> _______________________________________________
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel at ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list