[Ffmpeg-devel] THP decoder

[Michael Niedermayer]

Hi

On Sat, Apr 07, 2007 at 07:26:54PM +0400, Kislyakov Maxim wrote:
[...]
> > [...]
> >> +    if (thpDemux->componentTypes[0] == 0) {
> >> +        st->codec->width = get_be32(pb);
> >> +        st->codec->height = get_be32(pb);
> >> +        if (version == 0x110)
> >> +            get_be32(pb);
> >> +    }
> > 
> > what if video is not the first component?
> 
> Video is the first component always.

how do you know that?


[...]
> > [...]
> >> +        pkt->stream_index = 1;
> > 
> > this is wrong if there is no video (i dont know if that can occure but your
> > demuxer is checking for the case so i assume it can)
> 
> Video is always there because it's video format, audio may be
> optional (it's written in specification)

i cant find that part in the spec ...

[....]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070407/accf4b7e/attachment.pgp>