[FFmpeg-devel] [PATCH] lavu/rational: add syntactic sugar.

Michael Niedermayer michaelni at gmx.at
Wed Jan 1 14:57:01 CET 2014


On Tue, Dec 31, 2013 at 05:32:41PM +0100, Nicolas George wrote:
> And a few patches to make use of it. I only changed the places that are
> different from the fork, so as not to make merging harder.
> 
> On one example, gcc and clang did produce exactly the same code, but on the
> whole project there were subtle changes.
> 
> FATE passes, but the following files were not built on my setup, I will try
> to give them a try later:
> 

> libavcodec/crystalhd.c
> libavcodec/libvorbisdec.c
> libavdevice/dshow.c
> libavfilter/vf_frei0r.c

above 4 build fine

about the patch, i think the function is primarely usefull for
things in public headers which need to work with C++ / pre c99
compilers
for files i maintain, either style is fine with me, feel free to
replace the AVRational stuff in them by the function if you like

Thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- 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/20140101/60b7720c/attachment.asc>


More information about the ffmpeg-devel mailing list