[FFmpeg-devel] [PATCH] QT RLE header parser refactoring

Mike Melanson mike
Tue Mar 11 17:44:36 CET 2008


Michael Niedermayer wrote:
> On Mon, Mar 10, 2008 at 09:01:32PM -0700, Mike Melanson wrote:
>> Mike Melanson wrote:
>>> Yeah. I didn't like the idea either, to be honest. :) I have better
>>> ideas for refactoring I'll try next.
>> Here's my next idea. This is not a final draft; it's just a prototype to 
>> see if I'm going in the right direction. For the 8-bit decoding case, I 
>> moved the header parsing code into the main decode function and passed 3 
>> extra parameters into the 8-bit decoding function. There's a lot of cruft 
>> in this one but that would go away in the final draft.
> 
> Yes, the header parsing should be factored as well. So yes this is the right
> direction.
> 
> And after that the bpp sould be passed into the function instead of having
> a dozen functions.
> 
> And then the AVPalette stuff should be cleaned up
> Then the CHECK_STREAM_PTR / CHECK_PIXEL_PTR should be removed and
> a minimum of checks be implemented cleanly.
> 
> :)

Great. Thanks for the roadmap. I'm not sure about the second step--
passing the bpp into the function. This implies making 1 unified RLE
decoder function rather than 7. But each RLE unpacker is slightly
different for each. Should there be 7 different functions, or 1 function
that has 7 case statements?

	Thanks...
-- 
	-Mike Melanson




More information about the ffmpeg-devel mailing list