[FFmpeg-devel] [patch] Fix a warning in fft_mmx.asm on mac
michaelni at gmx.at
Thu Apr 19 17:17:52 CEST 2012
On Thu, Apr 19, 2012 at 07:59:44AM -0700, Nico Weber wrote:
> 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 :-)
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel