[FFmpeg-devel] [PATCH] RealAudio SIPR @16k decoder (4/4) demuxer

Benjamin Larsson banan
Thu Oct 16 18:54:51 CEST 2008


Michael Niedermayer wrote:
> On Thu, Oct 16, 2008 at 01:38:23PM +0200, Benjamin Larsson wrote:
>   
>> Michael Niedermayer wrote:
>>     
>>> On Wed, Sep 24, 2008 at 10:33:53AM +0200, Benjamin Larsson wrote:
>>>   
>>>       
>>>> Michael Niedermayer wrote:
>>>>     
>>>>         
>>>>> On Sat, Jul 12, 2008 at 02:17:03PM +0700, Vladimir Voroshilov wrote:
>>>>>       
>>>>>           
>>> [...]
>>>   
>>>       
>>>>>       
>>>>>           
>>>>>> +                          // Perform reordering
>>>>>> +                          for(n=0; n < 38; n++) {
>>>>>> +                              int j;
>>>>>> +                              int i = bs * sipr_swaps[n][0];
>>>>>> +                              int o = bs * sipr_swaps[n][1];
>>>>>> +                                // swap nibbles of block 'i' with 'o'
>>>>>> +                                for(j = 0;j < bs; j++) {
>>>>>>         
>>>>>>             
>>>>> Fix the indention, and check if the code cannot write out of the
>>>>> array. Ill checkit as well but the rm demuxer is messy its better if
>>>>> its checked by more than 1 person.
>>>>>       
>>>>>           
>>>> Indentation fixed. To me by inspection the code looks ok, and this code
>>>> has been running in mplayer for as long as it had binary rm codec
>>>> support. And I ran it through valgrind with no complaints on a test file.
>>>>     
>>>>         
>>> Could you try the code with some damaged files? (created by tools/trasher or
>>> some other tool)
>>>
>>> ill review the new patch soon.
>>>
>>>   
>>>       
>> Ping ?
>>     
>
> ive looked through the thread and honestly iam not sure which patch ive
> not reviewed.
> So a hint is welcome ...
>
>   

Hmm, I looked through it now there is nothing more to review. I'll ask 
roberto about the packet size issue.

MvH
Benjamin Larsson






More information about the ffmpeg-devel mailing list