From 6b2f66469168724273593d2c23eb78089afc0e46 Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Tue, 27 Jul 2004 01:30:44 +0000 Subject: [PATCH] better context model (0.2-1% lower bitrate) a tiny bit cleanup Originally committed as revision 3348 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavcodec/snow.c | 53 +++++++++++++++++++++++++++-------------------- 1 file changed, 30 insertions(+), 23 deletions(-) diff --git a/libavcodec/snow.c b/libavcodec/snow.c index a377ccc92e..0f1ebb056e 100644 --- a/libavcodec/snow.c +++ b/libavcodec/snow.c @@ -1420,7 +1420,7 @@ static void encode_subband(SnowContext *s, SubBand *b, DWTELEM *src, DWTELEM *pa for(y=0; y 1){ - if(orientation==1) pri2= src[y + (x-2)*stride]; - else pri2= src[x - 2 + y*stride]; + /*if(x > 1){ + if(orientation==1) ll= src[y + (x-2)*stride]; + else ll= src[x - 2 + y*stride]; }*/ } - if(!(lt|rt|t|l)){ + if(!(/*ll|*/l|lt|t|rt)){ if(v){ runs[run_index++]= run; run=0; @@ -1458,12 +1458,12 @@ static void encode_subband(SnowContext *s, SubBand *b, DWTELEM *src, DWTELEM *pa run_index=0; run= runs[run_index++]; - put_symbol(&s->c, b->state[1+7], run, 0); + put_symbol(&s->c, b->state[1], run, 0); for(y=0; y 1){ - if(orientation==1) pri2= src[y + (x-2)*stride]; - else pri2= src[x - 2 + y*stride]; + /*if(x > 1){ + if(orientation==1) ll= src[y + (x-2)*stride]; + else ll= src[x - 2 + y*stride]; }*/ } - if(lt|rt|t|l){ - put_cabac(&s->c, &b->state[0][(!l) + 2*(!t) + 4*(!lt) + 8*(!rt)], !v); + if(/*ll|*/l|lt|t|rt){ + int context= av_log2(/*ABS(ll) + */3*ABS(l) + ABS(lt) + 2*ABS(t) + ABS(rt)); + + put_cabac(&s->c, &b->state[0][context], !!v); }else{ if(!run){ run= runs[run_index++]; - put_symbol(&s->c, b->state[1+7], run, 0); + put_symbol(&s->c, b->state[1], run, 0); assert(v); }else{ run--; @@ -1500,7 +1502,9 @@ static void encode_subband(SnowContext *s, SubBand *b, DWTELEM *src, DWTELEM *pa } } if(v){ - put_symbol(&s->c, b->state[quant13[(ABS(l) + ABS(t))&0x7F] + 1], ABS(v)-1, 0); + int context= av_log2(/*ABS(ll) + */3*ABS(l) + ABS(lt) + 2*ABS(t) + ABS(rt)); + + put_symbol(&s->c, b->state[context + 2], ABS(v)-1, 0); put_cabac(&s->c, &b->state[0][16 + 1 + 3 + quant3b[l&0xFF] + 3*quant3b[t&0xFF]], v<0); } } @@ -1635,11 +1639,11 @@ static inline void decode_subbandX(SnowContext *s, SubBand *b, DWTELEM *src, DWT h= b->height; } x=y=0; - run= get_symbol(&s->c, b->state[1+7], 0); + run= get_symbol(&s->c, b->state[1], 0); for(y=0; y 1){ - if(orientation==1) pri2= src[y + (x-2)*stride]; - else pri2= src[x - 2 + y*stride]; + /*if(x > 1){ + if(orientation==1) ll= src[y + (x-2)*stride]; + else ll= src[x - 2 + y*stride]; }*/ } - if(lt|rt|t|l){ - v=!get_cabac(&s->c, &b->state[0][(!l) + 2*(!t) + 4*(!lt) + 8*(!rt)]); + if(/*ll|*/l|lt|t|rt){ + int context= av_log2(/*ABS(ll) + */3*ABS(l) + ABS(lt) + 2*ABS(t) + ABS(rt)); + + v=get_cabac(&s->c, &b->state[0][context]); }else{ if(!run){ - run= get_symbol(&s->c, b->state[1+7], 0); + run= get_symbol(&s->c, b->state[1], 0); //FIXME optimize this here //FIXME try to store a more naive run v=1; @@ -1675,7 +1681,8 @@ static inline void decode_subbandX(SnowContext *s, SubBand *b, DWTELEM *src, DWT } } if(v){ - v= get_symbol(&s->c, b->state[quant13[(ABS(l) + ABS(t))&0x7F] + 1], 0) + 1; + int context= av_log2(/*ABS(ll) + */3*ABS(l) + ABS(lt) + 2*ABS(t) + ABS(rt)); + v= get_symbol(&s->c, b->state[context + 2], 0) + 1; if(get_cabac(&s->c, &b->state[0][16 + 1 + 3 + quant3b[l&0xFF] + 3*quant3b[t&0xFF]])) v= -v; if(orientation==1) src[y + x*stride]= v;