[FFmpeg-devel] [PATCH] remove disabled code part 3

Michael Niedermayer michaelni
Tue Aug 18 22:53:39 CEST 2009


On Sat, Aug 08, 2009 at 05:16:32PM +0200, Diego Biurrun wrote:
> On Fri, Aug 07, 2009 at 11:15:19AM +0200, Diego Biurrun wrote:
> > I started looking into disabled code, there is lots of it and probably
> > mostly cruft.  Here is a patch that removes it, starting at the top
> > level and the tests subdirectory.  More patches shall follow later.
> 
> next up: libpostproc
> 
> Diego

>  postprocess_template.c |  212 -------------------------------------------------
>  1 file changed, 212 deletions(-)
> 4a686d91ad7536f1bf2230984290dedace918a77  if_zero_part3_libpostproc.diff
> Index: libpostproc/postprocess_template.c
> ===================================================================
> --- libpostproc/postprocess_template.c	(revision 19607)
> +++ libpostproc/postprocess_template.c	(working copy)
> @@ -349,111 +349,7 @@
>  }
>  #endif //HAVE_ALTIVEC
>  
> -#if 0
>  /**
> - * Experimental implementation of the filter (Algorithm 1) described in a paper from Ramkishor & Karandikar
> - * values are correctly clipped (MMX2)
> - * values are wraparound (C)
> - * Conclusion: It is fast, but introduces ugly horizontal patterns
> - * if there is a continuous gradient.
> -        0 8 16 24
> -        x = 8
> -        x/2 = 4
> -        x/8 = 1
> -        1 12 12 23
> - */
> -static inline void RENAME(vertRK1Filter)(uint8_t *src, int stride, int QP)

ok


[...]
> @@ -605,115 +501,7 @@
>  */
>      src+= stride*4;
>      __asm__ volatile(
> -
> -#if 0 //slightly more accurate and slightly slower
> -        "pxor %%mm7, %%mm7                      \n\t" // 0
>          "lea (%0, %1), %%"REG_a"                \n\t"

id like to keep this

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090818/fbd9a20e/attachment.pgp>



More information about the ffmpeg-devel mailing list