[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:

> 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

????????

> it's also a ISO standard in ITU,

AVS is not an ISO or ITU standard.

> 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.

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