On Sun, Sep 23, 2012 at 07:00:17PM +0200, Jordi Ortiz wrote:
>
> --- a/libavcodec/dsputil.c
> +++ b/libavcodec/dsputil.c
> @@ -2998,3 +2998,88 @@ av_cold void ff_dsputil_init(DSPContext* c,
> AVCodecContext *avctx)
> +
> +#if CONFIG_DIRAC_DECODER
> +void ff_put_pixels8_8_c(uint8_t *dst, const uint8_t *src, int stride, int h)
> +{
> + put_pixels8_8_c(dst, src, stride, h);
> +}
What's the difference between put_pixels8_8_c() and ff_put_pixels8_8_c()?
If - as I suspect - it's just the ff_ prefix, why don't you directly
export the functions instead of adding wrappers and exporting those?
> --- a/libavcodec/dsputil.h
> +++ b/libavcodec/dsputil.h
> @@ -531,6 +531,42 @@ typedef struct DSPContext {
>
> +#if CONFIG_DIRAC_DECODER
> +void ff_put_pixels8_8_c(uint8_t *dst, const uint8_t *src, int stride, int h);
There is no need to #ifdef extern/function declarations.
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
