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

[Michael Niedermayer]

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 :-)

applied

thanks

[...]
-- 
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
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120419/f28b1e2f/attachment.asc>