From: Ivan Kalvachev <ikalvachev@gmail.com>
Date: Sat, 14 Feb 2009 19:09:33 +0000 (+0000)
Subject: Clarify TODO comment
X-Git-Tag: v0.5~488
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=788aa94a9dc5fd290e1454dc76081641a2fddd6b;p=platform%2Fupstream%2Flibav.git

Clarify TODO comment

Originally committed as revision 17264 to svn://svn.ffmpeg.org/ffmpeg/trunk
---

diff --git a/libavcodec/mpegvideo_xvmc.c b/libavcodec/mpegvideo_xvmc.c
index 2cc6dd8..c8a055f 100644
--- a/libavcodec/mpegvideo_xvmc.c
+++ b/libavcodec/mpegvideo_xvmc.c
@@ -271,7 +271,9 @@ void ff_xvmc_decode_mb(MpegEncContext *s)
                 s->pblocks[i][0] -= 1 << 10;
             if (!render->idct) {
                 s->dsp.idct(s->pblocks[i]);
-                //!!TODO!clip!!!
+                /*It is unclear if MC hardware requires pixel diff values to be in
+                range [-255;255]. TODO cliping if such hardware is ever found.
+                As of now it would only be unnecessery slowdown. */
             }
 //copy blocks only if the codec doesn't support pblocks reordering
             if (s->avctx->xvmc_acceleration == 1) {