[MPlayer-dev-eng] demux acc problem[patch]

[Nico Sabbi]

qwen dux wrote:

>the last cvs version of mplayer demuxed some aac audio files with
>undetermined sequence,my aac file was demuxed with mpeg audio,here is info:
>Playing /home/qwdu/media/music/aac/100percent.aac.
>get_path('sub/') -> '/home/qwdu/.mplayer/sub/'
>[file] File size is 691925 bytes
>STREAM: [file] /home/qwdu/media/music/aac/100percent.aac
>STREAM: Description: File
>STREAM: Author: Albeu
>STREAM: Comment: based on the code from ??? (probably Arpi)
>Checking for YUV4MPEG2
>ASF_check: not ASF guid!
>Checking for NuppelVideo
>Checking for REAL
>Checking for SMJPEG
>Searching demuxer type for filename
>/home/qwdu/media/music/aac/100percent.aac ext: .aac
>Checking for Nullsoft Streaming Video
>Checking for MOV
>Checking for VIVO
>AVS: avs_check_file - attempting to open file
>/home/qwdu/media/music/aac/100percent.aac
>AVS: File is too big, aborting...
>Checking for PVA
>Checking for MPEG-TS...
>TRIED UP TO POSITION 69681, FOUND 47, packet_size= 0, SEEMS A TS? 0
>Checking for LMLM4 Stream Format
>Invalid packet in LMLM4 stream: ch=65521 size=618659030
>LMLM4 Stream Format not found
>sync_mpeg_ps: seems to be MP3 stream...
>MPEG Stream reached EOF
>ds_fill_buffer: EOF reached (stream: video)
>MPEG packet stats: p100: 2  p101: 0 p1B6: 0 p12x: 2 sli: 1 a: 1 b: 0 c: 0
>idr: 0 sps: 0 pps: 0 PES: 16  MP3: 104, synced: 0
>Not MPEG System Stream format... (maybe Transport Stream?)
>Encrypted VOB file! Read DOCS/HTML/en/dvd.html.
>  
>
[snip]

>here is the modified version of demuxer.c,and the aac audio can be detected
>and played normal.
>Index: demuxer.c
>===================================================================
>RCS file: /cvsroot/mplayer/main/libmpdemux/demuxer.c,v
>retrieving revision 1.211
>diff -u -r1.211 demuxer.c
>--- demuxer.c   17 Dec 2005 10:46:48 -0000      1.211
>+++ demuxer.c   29 Dec 2005 03:13:54 -0000
>@@ -101,6 +101,7 @@
>   &demuxer_desc_pva,
>   &demuxer_desc_mpeg_ts,
>   &demuxer_desc_lmlm4,
>+  &demuxer_desc_aac,
>   &demuxer_desc_mpeg_ps,
>   &demuxer_desc_mpeg_pes,
>   &demuxer_desc_mpeg_es,
>@@ -122,7 +123,6 @@
> #ifdef USE_LIBAVFORMAT
>   &demuxer_desc_lavf,
> #endif
>-  &demuxer_desc_aac,
>   NULL
> };
>
>  
>

for the time being this patch is rejected because it can't be safely 
applied without possibly breaking
demux_mpg, (because certain synccode sequences in one of each may look 
like content of the other one).
Eventually  demux_mpg should be fixed so as not to try to make too many 
things at once.

You can safely rely on the filename extension (or content-type if that 
audio content is streamed from a network protocol)
to auto-select the appropriare demuxer.

    Nico