diff options
author | Diego Biurrun <diego@biurrun.de> | 2013-04-03 20:09:58 +0200 |
---|---|---|
committer | Diego Biurrun <diego@biurrun.de> | 2013-04-05 12:24:28 +0200 |
commit | ed16c2dbf47cdd7c48825b4da6e7036698e5dde1 (patch) | |
tree | bb63c74b7e0d32b565877983b7383bde76219793 | |
parent | ae35d91d44c5e676af3d146bf8d05b241c467675 (diff) | |
download | ffmpeg-ed16c2dbf47cdd7c48825b4da6e7036698e5dde1.tar.gz |
h261: Remove H.261 loop filter from dsputil
There is no arch-optimized version of the H.261 loop filter and there
likely will never be, so the dsputil overhead does not give any benefit.
-rw-r--r-- | libavcodec/dsputil.c | 29 | ||||
-rw-r--r-- | libavcodec/dsputil.h | 2 | ||||
-rw-r--r-- | libavcodec/h261.c | 40 |
3 files changed, 34 insertions, 37 deletions
diff --git a/libavcodec/dsputil.c b/libavcodec/dsputil.c index 10d775ca5e..7b7cdc88d4 100644 --- a/libavcodec/dsputil.c +++ b/libavcodec/dsputil.c @@ -1479,33 +1479,6 @@ static void h263_h_loop_filter_c(uint8_t *src, int stride, int qscale){ } } -static void h261_loop_filter_c(uint8_t *src, int stride){ - int x,y,xy,yz; - int temp[64]; - - for(x=0; x<8; x++){ - temp[x ] = 4*src[x ]; - temp[x + 7*8] = 4*src[x + 7*stride]; - } - for(y=1; y<7; y++){ - for(x=0; x<8; x++){ - xy = y * stride + x; - yz = y * 8 + x; - temp[yz] = src[xy - stride] + 2*src[xy] + src[xy + stride]; - } - } - - for(y=0; y<8; y++){ - src[ y*stride] = (temp[ y*8] + 2)>>2; - src[7+y*stride] = (temp[7+y*8] + 2)>>2; - for(x=1; x<7; x++){ - xy = y * stride + x; - yz = y * 8 + x; - src[xy] = (temp[yz-1] + 2*temp[yz] + temp[yz+1] + 8)>>4; - } - } -} - static inline int pix_abs16_c(void *v, uint8_t *pix1, uint8_t *pix2, int line_size, int h) { int s, i; @@ -2735,8 +2708,6 @@ av_cold void ff_dsputil_init(DSPContext* c, AVCodecContext *avctx) c->h263_v_loop_filter= h263_v_loop_filter_c; } - c->h261_loop_filter= h261_loop_filter_c; - c->try_8x8basis= try_8x8basis_c; c->add_8x8basis= add_8x8basis_c; diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h index ce4469dc5c..ca418fe0b0 100644 --- a/libavcodec/dsputil.h +++ b/libavcodec/dsputil.h @@ -277,8 +277,6 @@ typedef struct DSPContext { void (*h263_v_loop_filter)(uint8_t *src, int stride, int qscale); void (*h263_h_loop_filter)(uint8_t *src, int stride, int qscale); - void (*h261_loop_filter)(uint8_t *src, int stride); - /* assume len is a multiple of 8, and arrays are 16-byte aligned */ void (*vector_clipf)(float *dst /* align 16 */, const float *src /* align 16 */, float min, float max, int len /* align 16 */); diff --git a/libavcodec/h261.c b/libavcodec/h261.c index c3abdb23a7..1ffd64a5ec 100644 --- a/libavcodec/h261.c +++ b/libavcodec/h261.c @@ -32,6 +32,34 @@ uint8_t ff_h261_rl_table_store[2][2*MAX_RUN + MAX_LEVEL + 3]; +static void h261_loop_filter(uint8_t *src, int stride) +{ + int x,y,xy,yz; + int temp[64]; + + for(x=0; x<8; x++){ + temp[x ] = 4*src[x ]; + temp[x + 7*8] = 4*src[x + 7*stride]; + } + for(y=1; y<7; y++){ + for(x=0; x<8; x++){ + xy = y * stride + x; + yz = y * 8 + x; + temp[yz] = src[xy - stride] + 2*src[xy] + src[xy + stride]; + } + } + + for(y=0; y<8; y++){ + src[ y*stride] = (temp[ y*8] + 2)>>2; + src[7+y*stride] = (temp[7+y*8] + 2)>>2; + for(x=1; x<7; x++){ + xy = y * stride + x; + yz = y * 8 + x; + src[xy] = (temp[yz-1] + 2*temp[yz] + temp[yz+1] + 8)>>4; + } + } +} + void ff_h261_loop_filter(MpegEncContext *s){ H261Context * h= (H261Context*)s; const int linesize = s->linesize; @@ -43,10 +71,10 @@ void ff_h261_loop_filter(MpegEncContext *s){ if(!(IS_FIL (h->mtype))) return; - s->dsp.h261_loop_filter(dest_y , linesize); - s->dsp.h261_loop_filter(dest_y + 8, linesize); - s->dsp.h261_loop_filter(dest_y + 8 * linesize , linesize); - s->dsp.h261_loop_filter(dest_y + 8 * linesize + 8, linesize); - s->dsp.h261_loop_filter(dest_cb, uvlinesize); - s->dsp.h261_loop_filter(dest_cr, uvlinesize); + h261_loop_filter(dest_y, linesize); + h261_loop_filter(dest_y + 8, linesize); + h261_loop_filter(dest_y + 8 * linesize, linesize); + h261_loop_filter(dest_y + 8 * linesize + 8, linesize); + h261_loop_filter(dest_cb, uvlinesize); + h261_loop_filter(dest_cr, uvlinesize); } |