[MPlayer-dev-eng] [Patch] h.264 sps scaling list parsing fix

Marco Munderloh munderl at tnt.uni-hannover.de
Thu Jun 25 10:55:01 CEST 2009


Carl Eugen Hoyos wrote:
> Hi!
> 
> Marco Munderloh <munderl <at> tnt.uni-hannover.de> writes:
> 
>> As viewing the patch, I recognized that a function made for returning an
>> singed value but specified as "static unsigned int" is totally stupid...
>>
>> Find attached a corrected version.
> 
> I cannot comment on functionality (could you point to a test-case?)

I got three files from Loren and have several from my own Canon HF10.
All play fine.

> but I do have a comment:
> 
>> -    if(getbits(buf, n++, 1)){
> 
> [...]
> 
>> +    if(getbits(buf, n++, 1))
>> +    {

Sure. Find it attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mpeg_hdr.diff
Type: text/x-patch
Size: 1146 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20090625/baf1d282/attachment.bin>


More information about the MPlayer-dev-eng mailing list