[FFmpeg-cvslog] r23823 - trunk/configure

[Måns Rullgård]

Ramiro Polla <ramiro.polla at gmail.com> writes:

> On Sun, Jun 27, 2010 at 11:16 AM, mru <subversion at mplayerhq.hu> wrote:
>> Author: mru
>> Date: Sun Jun 27 16:16:54 2010
>> New Revision: 23823
>>
>> Log:
>> Make implicit function declarations an error
>>
>> Modified:
>> ? trunk/configure
>>
>> Modified: trunk/configure
>> ==============================================================================
>> --- trunk/configure ? ? Sun Jun 27 16:16:46 2010 ? ? ? ?(r23822)
>> +++ trunk/configure ? ? Sun Jun 27 16:16:54 2010 ? ? ? ?(r23823)
>> @@ -2697,7 +2697,7 @@ elif enabled ccc; then
>> ? ? add_cflags -msg_disable unreachcode
>> ?elif enabled gcc; then
>> ? ? check_cflags -fno-tree-vectorize
>> - ? ?check_cflags -Werror=implicit
>> + ? ?check_cflags -Werror=implicit-function-declaration
>> ? ? check_cflags -Werror=missing-prototypes
>> ?elif enabled clang; then
>> ? ? check_cflags -Qunused-arguments
>
> This uncovered more bugs in MinGW:
> https://sourceforge.net/tracker/?func=detail&aid=3022047&group_id=2435&atid=102435
> https://sourceforge.net/tracker/?func=detail&aid=3022054&group_id=2435&atid=102435
>
> The second one is related to tempnam(), which is only used by
> av_tempfile(), which is only used by the xivd wrapper. av_tempfile()
> is not even exported in headers, shouldn't it be moved to an
> xvid-specific file and the av_ prefix dropped? (or dropped altogether
> along with the xvid wrapper?)

I've been advocating the removal of the xvid wrapper for a long time.
If that is not desired, the av_tempfile() function should at least be
compiled conditionally.  It is causing me problems elsewhere too.

-- 
M?ns Rullg?rd
mans at mansr.com