[FFmpeg-devel] [patch] Fix a warning in fft_mmx.asm on mac

[Nico Weber]

On Tue, Apr 17, 2012 at 12:39 PM, Nico Weber <thakis at chromium.org> wrote:
>> > > Also if you are concerned about warnings, there are many that can be
>> > > removed with less risk and troubble.
>> >
>> > This is the last warning in all of chromium, including all third-party
>>>  code. (Admittedly, we disabled several harmless warnings that ffmpeg
>> > doesn't seem to care about in the C parts. This yasm warning can't be
>> > disabled though.)
>>
>> maybe instead of a yes or no to the removial of this code, it could
>> just be put under a __YASM_VERSION_ID__ based check ?
>
> That's a good idea. Attached. I ran `make fate` with this change and
> verified that all tests still pass (on mac, with yasm 1.2).

Ping :-)