[FFmpeg-devel] [PATCH] Make usage of local st instead of s->streams[0]

Sebastian Vater cdgs.basty
Sun May 9 16:25:29 CEST 2010


Jai Menon a ?crit :
> On Sun, May 9, 2010 at 7:28 PM, Sebastian Vater
> <cdgs.basty at googlemail.com> wrote:
>   
>> Hello guys!
>>
>> In libavformat/iff.c, iff_read_packet declares:
>>    AVStream *st = s->streams[0];
>>
>> But the code afterwards still uses s->streams[0] instead of st.
>>
>> Therefore, I created a small patch which accesses the local st instead
>> of always doing s->streams[0].
>>     
>
> why not just remove the unused variable?
>   

Would be possible, too. ;-)

Although the file size of source code would grow then, so I decided to
handle it the way I did in the patch, which makes the total number of
bytes for libavformat/iff.c a little bit smaller ;)

-- 

Best regards,
                   :-) Basty/CDGS (-:




More information about the ffmpeg-devel mailing list