Hi,

On Tue, Apr 26, 2011 at 2:07 PM, Diego Biurrun <[email protected]> wrote:
>  libavcodec/alpha/dsputil_alpha.c        |   72 ---------
>  libavcodec/alpha/motion_est_alpha.c     |   43 -----

These have comments that explain why they are there. The code isn't
equal to the generic C code, so please keep them.

>  libavcodec/cabac.h                      |   83 ----------

Here, the purpose is to show why our implementation was chosen. Please keep.

>  libavcodec/celp_filters.c               |    9 -
>  libavcodec/dwt.c                        |    8 -

These document code, I even asked for the celp_filters.c one.

>  libavcodec/sh4/idct_sh4.c               |  151 ------------------

OK.

>  libavcodec/x86/idct_mmx.c               |  112 --------------

Please keep.

>  libavcodec/x86/mpegvideo_mmx_template.c |   11 --

OK.

>  libavcodec/x86/simple_idct_mmx.c        |  256 
> -------------------------------

Please keep.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to