[FFmpeg-devel] [PATCH] better reporting of programs
Fri Oct 19 03:18:37 CEST 2007
On Wed, Oct 17, 2007 at 11:52:54PM +0200, Nico Sabbi wrote:
> Il Tuesday 16 October 2007 23:20:55 Michael Niedermayer ha scritto:
> > wouldnt it be better to use the index (like in
> > AVFormatContext.streams[index]) or even a AVStream pointer
> > instead of AVStream.id ? it would be easier to find the AVStream
> > with the index/pointer, also finding the id from the index is a
> > mere .streams[index].id the other way around is slightly harder and
> > less efficient
> implemented with an index
> > by how much would the complexity increas if this where (re)alloc()
> > instead of fixed 64 ?
> > also the minor version should be bumped
> both done
> > could you split the cosmetic code moving from the functional
> > changes?
> > [...]
> Patches are in this order:
> f1: cosmetical; moved the core of the stream description to a separate
> f2: API extension
> f3: actual use of the API in mpegts.c
iam not mpegts.c maintainer ...
> f4: changes to ffmpeg.c to support the programid switch
> (currently 1 program alone until Reimar implements the
> code to assign arrays)
will review tomorrow
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The misfortune of the wise is better than the prosperity of the fool.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel