On Tue, Aug 09, 2011 at 11:26:42AM +0100, Måns Rullgård wrote:
> Kostya Shishkov <[email protected]> writes:
>
> > diff --git a/libavcodec/dsputil.c b/libavcodec/dsputil.c
> > index d318601..039cf0b 100644
> > --- a/libavcodec/dsputil.c
> > +++ b/libavcodec/dsputil.c
> > @@ -1280,16 +1280,16 @@ static void wmv2_mspel8_h_lowpass(uint8_t *dst,
> > uint8_t *src, int dstStride, int
> > }
> >
> > #if CONFIG_RV40_DECODER
> > -static void put_rv40_qpel16_mc33_c(uint8_t *dst, uint8_t *src, int stride){
> > +void ff_put_rv40_qpel16_mc33_c(uint8_t *dst, uint8_t *src, int stride){
> > put_pixels16_xy2_8_c(dst, src, stride, 16);
> > }
> > -static void avg_rv40_qpel16_mc33_c(uint8_t *dst, uint8_t *src, int stride){
> > +void ff_avg_rv40_qpel16_mc33_c(uint8_t *dst, uint8_t *src, int stride){
> > avg_pixels16_xy2_8_c(dst, src, stride, 16);
> > }
> > -static void put_rv40_qpel8_mc33_c(uint8_t *dst, uint8_t *src, int stride){
> > +void ff_put_rv40_qpel8_mc33_c(uint8_t *dst, uint8_t *src, int stride){
> > put_pixels8_xy2_8_c(dst, src, stride, 8);
> > }
> > -static void avg_rv40_qpel8_mc33_c(uint8_t *dst, uint8_t *src, int stride){
> > +void ff_avg_rv40_qpel8_mc33_c(uint8_t *dst, uint8_t *src, int stride){
> > avg_pixels8_xy2_8_c(dst, src, stride, 8);
> > }
> > #endif /* CONFIG_RV40_DECODER */
>
> Is there no way to remove these from dsputil.c entirely?
There is a way - by duplicating code for those {put,avg}_pixels{8,16}_xy2_8_c
I also don't like having those functions left in general dsputil.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
