[FFmpeg-devel] [PATCH] Move ff_inverse to lavu

[Måns Rullgård]

Ramiro Polla <ramiro.polla at gmail.com> writes:

> 2010/7/18 M?ns Rullg?rd <mans at mansr.com>:
>> Ramiro Polla <ramiro.polla at gmail.com> writes:
>>> ff_inverse currently resides in lavc, but it's being used by FASTDIV,
>>> which is defined in lavu.
>>
>> This has come up before [1], and it's not as simple as it seems.
>> Although the macro is defined in lavu, it is only used from lavc.
>> Moving the table to lavu would make lookups more expensive with shared
>> libs.
>>
>> [1] http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/75376
>
> Thanks for the pointer. My issue was with suncc just like Diego's.
>
> There was no outcome on the discussion. Should it just be left as is?
> It seems weird for FASTDIV to be defined in lavu since it's only used
> in lavc.

I agree it's weird.  I wouldn't mind moving it to lavc.

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