[FFmpeg-devel] [PATCH] mov.c: factorize ff_mov_read_stsd_entries()

[Baptiste Coudurier]

Hi Ronald,

On 3/9/2009 1:13 PM, Ronald S. Bultje wrote:
> Hi,
> 
> as per $subj, this patch factorizes reading of a single STSD atom into
> its own function. This sole atom is the stream descriptor in QT
> streams sent over RTSP, and so this function alone is sufficient to
> read a complete RTSP header and prepare for stream playback (which
> I'll do in yet another patch).
> 

Well I would prefer if the whole function would get changed to parse
these structures and populate a MOVStsd struct. You could then use these
structs. Besides I believe it would be easier to then create separate
tracks and handle multiple data reference.

What do you think ?

[...]

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org