[FFmpeg-devel] [PATCH]Do not redefine av_restrict

Ronald S. Bultje rsbultje at gmail.com
Wed Sep 19 20:33:25 CEST 2012


Hi,

On Wed, Sep 19, 2012 at 11:00 AM, Hendrik Leppkes <h.leppkes at gmail.com> wrote:
> On Wed, Sep 19, 2012 at 7:55 PM, Ronald S. Bultje <rsbultje at gmail.com>wrote:
>
>> Hi,
>>
>> On Wed, Sep 19, 2012 at 9:34 AM, Carl Eugen Hoyos <cehoyos at ag.or.at>
>> wrote:
>> > Hi!
>> >
>> > Attached patch should silence another msvc-warning, to make sure no real
>> > warnings are hidden in the noise.
>> > (Untested.)
>> >
>> > Please comment, Carl Eugen
>>
>> If there's two occurrences of av_restrict, shouldn't we fix it by
>> removing the one that shouldn't be there?
>>
>> Where does the second one live?
>>
>
> There is one in avutil attributes.h (in a installed header), and apparently
> mplayer started using it right away, and thus people are wary about
> removing it.

Fuck mplayer, fix the bug.

Ronald


More information about the ffmpeg-devel mailing list