[FFmpeg-devel] [patch] Fix a warning in fft_mmx.asm on mac
thakis at chromium.org
Thu Apr 19 16:59:44 CEST 2012
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).
More information about the ffmpeg-devel