[vlc-devel] [PATCH] TS demux: handle es stream type 0x42, which is CAVS

[Måns Rullgård]

Can Wu <wu.canus at gmail.com> writes:

> 2011/11/17 Måns Rullgård <mans at mansr.com>
>
>> Can Wu <wu.canus at gmail.com> writes:
>>
>> > On Thu, Nov 17, 2011 at 9:25 AM, Laurent Aimar <fenrir at elivagar.org>
>> wrote:
>> >
>> >> On Wed, Nov 16, 2011 at 11:13:03PM +0800, Can Wu wrote:
>> >> > ---
>> >> >  modules/demux/ts.c |    3 +++
>> >> >  1 files changed, 3 insertions(+), 0 deletions(-)
>> >> >
>> >> > diff --git a/modules/demux/ts.c b/modules/demux/ts.c
>> >> > index ff3ba66..352c8a3 100644
>> >> > --- a/modules/demux/ts.c
>> >> > +++ b/modules/demux/ts.c
>> >> > @@ -2405,6 +2405,9 @@ static int PIDFillFormat( ts_pid_t *pid, int i_stream_type )
>> >> >      case 0x1B:  /* H264 <- check transport syntax/needed descriptor */
>> >> >          es_format_Init( fmt, VIDEO_ES, VLC_CODEC_H264 );
>> >> >          break;
>> >> > +    case 0x42:  /* CAVS (Chinese AVS) */
>> >> > +        es_format_Init( fmt, VIDEO_ES, VLC_CODEC_CAVS );
>> >> > +        break;
>> >>  Is there a registration descriptor or something equivalent to
>> >> detect CAVS streams? if not, do you know if this stream type has
>> >> been properly assigned by ISO?
>> >>
>> > Yeah, I couldn't find the define from a iso spec, I'm just got it
>> > from the sample ts file, and some sample tools like
>> > http://code.google.com/p/tstools, use this value.
>> >
>> > AVS will be the standardard video coding standard, maybe the only
>> > one, and
>>
>> ????????
>>
> Oh, I mean in china

The ISO and ITU websites are not, to my knowledge, blocked by the Great
Firewall of China.

>> > it's also a ISO standard in ITU,
>>
>> AVS is not an ISO or ITU standard.
>>
> it's ITU standard, not?

No.

>> > I think this stream_type value should not be random used.
>>
>> The latest edition of the MPEG-TS spec (2006-05 + amendments) lists 0x42
>> as "reserved".  There is (predictably) no mention of AVS or any other
>> non-ISO standard.
>>
> need more source to check...

I checked the spec before writing this.

-- 
Måns Rullgård
mans at mansr.com