[FFmpeg-cvslog] r21735 - trunk/libavutil/internal.h

[Måns Rullgård]

Michael Kostylev <michael.kostylev at gmail.com> writes:

> On Fri Feb 19 16:55:44 2010
> M?ns Rullg?rd wrote:
>
>>> On Tue Feb  9 23:55:16 2010
>>> mru wrote:
>>>
>>>> Define missing llrint() as macro instead of inline function
>>>> 
>>>> This fixes building on some broken systems.
>>>> 
>>>> Modified:
>>>>    trunk/libavutil/internal.h
>>>> 
>>>> Modified: trunk/libavutil/internal.h
>>>> ==============================================================================
>>>> --- trunk/libavutil/internal.h	Tue Feb  9 23:55:13 2010	(r21734)
>>>> +++ trunk/libavutil/internal.h	Tue Feb  9 23:55:16 2010	(r21735)
>>>> @@ -225,10 +225,8 @@ static inline av_const unsigned int ff_s
>>>>  #endif /* HAVE_EXP2F */
>>>>  
>>>>  #if !HAVE_LLRINT
>>>> -static av_always_inline av_const long long llrint(double x)
>>>> -{
>>>> -    return rint(x);
>>>> -}
>>>> +#undef llrint
>>>> +#define llrint(x) rint(x)
>>>
>>> The dos machine is unhappy with this macro.
>> 
>> Please elaborate.
>
> A bunch of tests from the regression suite and the idroq-video-encode
> test no longer pass since r21735 (reverting it or applying a quick
> hack attached in the previous message make them pass again).

Weird.  Which call to llrint() is causing the trouble?

-- 
M?ns Rullg?rd
mans at mansr.com