[FFmpeg-cvslog] lagarith: Add correct line prediction for RGB

Nathan Caldwell git at videolan.org
Thu Nov 17 03:01:32 CET 2011


ffmpeg | branch: master | Nathan Caldwell <saintdev at gmail.com> | Wed Nov 16 00:45:00 2011 -0700| [39616fc30746ca07608f7855e85bd89cb08cfd27] | committer: Martin Storsjö

lagarith: Add correct line prediction for RGB

Signed-off-by: Martin Storsjö <martin at martin.st>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=39616fc30746ca07608f7855e85bd89cb08cfd27
---

 libavcodec/lagarith.c |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/libavcodec/lagarith.c b/libavcodec/lagarith.c
index 10cc71e..9200773 100644
--- a/libavcodec/lagarith.c
+++ b/libavcodec/lagarith.c
@@ -245,21 +245,21 @@ static void lag_pred_line(LagarithContext *l, uint8_t *buf,
 {
     int L, TL;
 
+    /* Left pixel is actually prev_row[width] */
+    L = buf[width - stride - 1];
     if (!line) {
         /* Left prediction only for first line */
         L = l->dsp.add_hfyu_left_prediction(buf + 1, buf + 1,
                                             width - 1, buf[0]);
         return;
     } else if (line == 1) {
-        /* Second line, left predict first pixel, the rest of the line is median predicted */
-        /* FIXME: In the case of RGB this pixel is top predicted */
-        TL = buf[-stride];
+        /* Second line, left predict first pixel, the rest of the line is median predicted
+         * NOTE: In the case of RGB this pixel is top predicted */
+        TL = l->avctx->pix_fmt == PIX_FMT_YUV420P ? buf[-stride] : L;
     } else {
         /* Top left is 2 rows back, last pixel */
         TL = buf[width - (2 * stride) - 1];
     }
-    /* Left pixel is actually prev_row[width] */
-    L = buf[width - stride - 1];
 
     add_lag_median_prediction(buf, buf - stride, buf,
                               width, &L, &TL);



More information about the ffmpeg-cvslog mailing list