[FFmpeg-devel] [PATCH] demux individual program out of MPEG-TS

Benoit Fouet benoit.fouet
Thu Sep 13 12:15:25 CEST 2007


Nico Sabbi wrote:
> Benoit Fouet wrote:
>   
>> Nico Sabbi wrote:
>>     
>>> Benoit Fouet wrote:
>>>       
>>>>> +    int i, j, k;
>>>>> +    int used = 0, discarded = 0;
>>>>> +    Program_t *p;
>>>>> +    for(i=0; i<ts->nb_prg; i++) {
>>>>> +        p = &ts->prg[i];
>>>>> +        for(j=0; j<p->nb_pids; j++) {
>>>>> +            if(p->pids[j] != pid)
>>>>> +                continue;
>>>>>           
>>>> using if(p->pids[j] == pid) would get rid of the continue
>>>>         
>>> but it would create one more level of bracing and nesting,
>>> making the code less readable
>>>       
>> i just thought this could help the compiler...
>>     
> ? I know gcc is considered stupid, but why should it generate
> faster code with one more nesting level rather than after
> excluding error cases?
>   

because of the "continue", i don't know how it will be compiled
maybe it will generate the same thing as the code i wrote :)

-- 
Ben
Purple Labs S.A.
www.purplelabs.com




More information about the ffmpeg-devel mailing list