[FFmpeg-devel] [RFC] remove lrintf fallback implementation
Måns Rullgård
mans
Tue Oct 30 21:43:29 CET 2007
Diego Biurrun <diego at biurrun.de> writes:
> On Mon, Oct 29, 2007 at 08:10:08PM +0100, V?ctor Paesa wrote:
>> V?ctor Paesa said:
>>
>> > The route I'm pondering by now is to create my own (minimalistic)
>> > libm99: if gcc delivers a C99 function missing in newlib I can do the
>> > same too, and it be would just a matter of using --extra-libs instead of
>> > patching ffmpeg.
>>
>> It ended up that I do not need to create a separate library, as Cygwin
>> and MinGW use the same COFF format: I can leverage its llrint()
>> implementation.
>>
>> [...]
>>
>> Hence, instead of patching FFmpeg, my attached patch proposal modifies
>> the documentation for compilation under Cygwin.
>
> Well, patching newlib would be better, but this is OK as well ..
>
>> --- doc/general.texi (revision 10872)
>> +++ doc/general.texi (working copy)
>> @@ -530,27 +530,41 @@
>>
>> @subsection Compilation under Cygwin
>>
>> -Cygwin works very much like Unix.
>> + at url{http://www.cygwin.com/, Cygwin} works very much like Unix,
>> +the main issue being that its C library is
>> + at url{http://sources.redhat.com/newlib/, newlib},
>> +instead of the more usual
>> + at url{http://www.gnu.org/software/libc/, GNU C library}.
>> +
>> +Newlib lacks llrint(), but we can leverage the llrint() implementation
>> +in @url{http://www.mingw.org/, MinGW}.
>
> There are systems other than Linux and they don't all run glibc. Also,
> we don't need to explain what Cygwin is. I propose:
>
> The main issue with Cygwin is that newlib, its C library, does not
you forgot that comma ----^
> contain llrint(). However, it is possible to leverage the
> implementation in MinGW.
Diego, I can't believe how sloppy you are.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list