[FFmpeg-devel] [PATCH] vp3.c: Don't compile a debug loop
Michael Niedermayer
michaelni
Wed Oct 1 12:07:57 CEST 2008
On Tue, Sep 30, 2008 at 03:32:11PM -0400, David Conrad wrote:
> On Sep 30, 2008, at 12:57 PM, Michael Niedermayer wrote:
>
>> On Mon, Sep 29, 2008 at 09:51:35PM -0700, Mike Melanson wrote:
>>> David Conrad wrote:
>>>> Patch attached (with small cosmetic patchs), as well as another to
>>>> remove unused code that's been moved elsewhere.
>>>
>>> Wow, I wrote a lot of debug infrastructure. Patch #1 looks fine.
>>> Deferring to Michael on 2 and 3 to make sure they meet his standards.
>>
>> 2 looked a little like it combined unrelated changes
>> 3, if its just reindent should be ok
>
> Here's 2 split up even further.
hmm this is ok but the reason why i asked for it to be split was:
@@ -1260,8 +1260,6 @@ static void reverse_dc_prediction(Vp3DecodeContext *s,
else if (FFABS(predicted_dc - vul) > 128)
predicted_dc = vul;
}
-
- DC_COEFF(i));
}
/* at long last, apply the predictor */
@@ -1298,7 +1296,6 @@ static void vertical_filter(unsigned char *first_pixel, int stride,
static void render_slice(Vp3DecodeContext *s, int slice)
{
int x;
- int m, n;
int16_t *dequantizer;
DECLARE_ALIGNED_16(DCTELEM, block[64]);
int motion_x = 0xdeadbeef, motion_y = 0xdeadbeef;
-----------
which was in patch #2 ...
and that surely does not belong in the keyframe patch ...
i didnt mean so much that the part you splited has to be split ...
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081001/de06f0ed/attachment.pgp>
More information about the ffmpeg-devel
mailing list