diff options
author | Mike Melanson <mike@multimedia.cx> | 2005-05-21 19:31:16 +0000 |
---|---|---|
committer | Mike Melanson <mike@multimedia.cx> | 2005-05-21 19:31:16 +0000 |
commit | 2935001c86fe647fde79395f15a4cf679cc1c3b2 (patch) | |
tree | 0376b751c470149c819fbf96576bf629146fc645 /libavcodec/vp3.c | |
parent | e2b9cf4e2e575335c68be1e7e8636b639a442736 (diff) | |
download | ffmpeg-streaming-2935001c86fe647fde79395f15a4cf679cc1c3b2.zip ffmpeg-streaming-2935001c86fe647fde79395f15a4cf679cc1c3b2.tar.gz |
fix bugs in new loop filter code; also, refrain from filtering against
data that has yet to be rendered; still #if'd out, will revisit when
proper algorithm can be validated
Originally committed as revision 4297 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'libavcodec/vp3.c')
-rw-r--r-- | libavcodec/vp3.c | 56 |
1 files changed, 25 insertions, 31 deletions
diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c index 00361ce..3dba5f4 100644 --- a/libavcodec/vp3.c +++ b/libavcodec/vp3.c @@ -2247,7 +2247,7 @@ static void render_slice(Vp3DecodeContext *s, int slice) slice_height = y + FRAGMENT_PIXELS; i = s->macroblock_fragments[current_macroblock_entry + 5]; } - fragment_width = plane_width / 2; + fragment_width = plane_width / FRAGMENT_PIXELS; if(ABS(stride) > 2048) return; //various tables are fixed size @@ -2396,42 +2396,36 @@ static void render_slice(Vp3DecodeContext *s, int slice) } #if 0 - /* do not perform left edge filter for left columns frags */ + /* perform the left edge filter if: + * - the fragment is not on the left column + * - the fragment is coded in this frame + * - the fragment is not coded in this frame but the left + * fragment is coded in this frame (this is done instead + * of a right edge filter when rendering the left fragment + * since this fragment is not available yet) */ if ((x > 0) && - (s->all_fragments[i].coding_method != MODE_COPY)) { + ((s->all_fragments[i].coding_method != MODE_COPY) || + ((s->all_fragments[i].coding_method == MODE_COPY) && + (s->all_fragments[i - 1].coding_method != MODE_COPY)) )) { horizontal_filter( - output_plane + s->all_fragments[i].first_pixel - 7*stride, - stride, bounding_values); + output_plane + s->all_fragments[i].first_pixel + 7*stride, + -stride, bounding_values); } - /* do not perform top edge filter for top row fragments */ + /* perform the top edge filter if: + * - the fragment is not on the top row + * - the fragment is coded in this frame + * - the fragment is not coded in this frame but the above + * fragment is coded in this frame (this is done instead + * of a bottom edge filter when rendering the above + * fragment since this fragment is not available yet) */ if ((y > 0) && - (s->all_fragments[i].coding_method != MODE_COPY)) { + ((s->all_fragments[i].coding_method != MODE_COPY) || + ((s->all_fragments[i].coding_method == MODE_COPY) && + (s->all_fragments[i - fragment_width].coding_method != MODE_COPY)) )) { vertical_filter( - output_plane + s->all_fragments[i].first_pixel + stride, - stride, bounding_values); - } - - /* do not perform right edge filter for right column - * fragments or if right fragment neighbor is also coded - * in this frame (it will be filtered for next fragment) */ - if ((x < plane_width - 1) && - (s->all_fragments[i].coding_method != MODE_COPY) && - (s->all_fragments[i + 1].coding_method == MODE_COPY)) { - horizontal_filter( - output_plane + s->all_fragments[i + 1].first_pixel - 7*stride, - stride, bounding_values); - } - - /* do not perform bottom edge filter for bottom row - * fragments or if bottom fragment neighbor is also coded - * in this frame (it will be filtered in the next row) */ - if ((y < plane_height - 1) && - (s->all_fragments[i].coding_method != MODE_COPY) && - (s->all_fragments[i + fragment_width].coding_method == MODE_COPY)) { - vertical_filter( - output_plane + s->all_fragments[i + fragment_width].first_pixel + stride, - stride, bounding_values); + output_plane + s->all_fragments[i].first_pixel - stride, + -stride, bounding_values); } #endif } |