[FFmpeg-devel] [Ffmpeg-devel] [PATCH] ported SGI decoder to the new API

[Baptiste Coudurier]

Diego Biurrun wrote:
> On Sat, Apr 07, 2007 at 06:49:30PM +0200, Michael Niedermayer wrote:
>> On Sun, Apr 08, 2007 at 12:19:38AM +0800, Xiaohui Sun wrote:
>>> Michael Niedermayer wrote:
>>>> On Sat, Apr 07, 2007 at 08:39:06PM +0800, Xiaohui Sun wrote:
>>>>  
>>>>>>> I think the problem lies in the RLE encoding, which should count up to 
>>>>>>> 127 consecutive pixels and I have sent the path before.
>>>>>>> Please use the attached path and have a test, thanks.
>>>>>>>        
>>>>>> this change needs testing to see what effects it has on compression rate 
>>>>>> with a few of the stanard test images like lena, ...
>>>  
>>>  I have made a test for tga and tiff format using some standard 
>>> pictures from _http://links.uwaterloo.ca/bragzone.base.html_
>>> The second column is with the consecutive pixels set to 127, and the 
>>> third column is the consecutive pixels set to 128, and a compression 
>>> rate lost
>>> is calculated as " (127-RLE size - 128-RLE size) / 128-RLE size"
>>> Hope this will be helpful.
>> yes, iam ok with the patch
> 
> Please send in a version that updates the regression test checksums.
> 

Applied with updated regression tests.


-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312