[FFmpeg-cvslog] r25881 - trunk/libavcodec/x86/dsputil_mmx.c

[Baptiste Coudurier]

On 12/5/10 11:03 AM, Stefano Sabatini wrote:
> On date Sunday 2010-12-05 16:43:42 -0200, Ramiro Polla wrote:
>> On Sat, Dec 4, 2010 at 11:31 PM, Baptiste Coudurier
>> <baptiste.coudurier at gmail.com> wrote:
>>> On 12/4/10 6:26 AM, Aurelien Jacobs wrote:
>>>> On Sat, Dec 04, 2010 at 02:06:06PM +0100, bcoudurier wrote:
>>>>> Author: bcoudurier
>>>>> Date: Sat Dec  4 14:06:06 2010
>>>>> New Revision: 25881
>>>>>
>>>>> Log:
>>>>> 10l, add ff_pw_1 to dsputil_mmx for yadif sse2
>>>>>
>>>>> Modified:
>>>>>    trunk/libavcodec/x86/dsputil_mmx.c
>>>>
>>>> I thought that we were trying to cut all the dependencies between
>>>> libavfilter and libavcodec (and that we actually managed to drop them
>>>> all at some point). Did I missed something ?
>>>
>>> Oh, yes why not. I can change the code if that's wanted.
>>> What do others think ?
>>
>> Please do.
> 
> Yes lavfi shouldn't depend on libavcodec. If you want you can add a
> specific dependency of yadif on libavcodec, but this should be avoided
> if possible. (And my long term plan is to move DSP libavcore, where it
> may be used by filters and external apps without to depend on
> libavcodec.)
> 
>>
>> BTW linking is still broken for shared libraries that don't allow
>> undefined references.

All right, I changed it.

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org