[MPlayer-dev-eng] [PATCH] improved AIFF PCM Audio support - take 4
Jake Luck
mplay at 10k.org
Fri Dec 10 19:11:20 CET 2004
> > @@ -167,6 +169,10 @@ int demux_audio_open(demuxer_t* demuxer)
> > frmt = fLaC;
> > stream_skip(s,-4);
> > break;
> > + } else if( hdr[0] == 'F' && hdr[1] == 'O' && hdr[2] == 'R' && hdr[3] == 'M' ) {
> > + frmt = AIFF;
> > + stream_skip(s,-4);
> > + break;
>
> I didn't read the patch closely, but is it possible to avoid that
> backwards-seek? If yes, please do so, because then e.g. playing from
> stdin will work (even without -cache).
>
> Greetings,
> Reimar Döffinger
Reimar, thank you for reviewing the patch.
I used stream_skip(s, -4) to follow the same mental model suggested by the
existing processing code in demux_audio.c. Currently, WAVE, FLAC and MP3
all perform backseek to reset the stream and then carry out a full header
parse. One advantage from this approach is that one no longer needs to
maintain a separate partial header parser.
I feel that if we do choose to eliminate backward-seek we should consider
doing it globally across all codecs to ensure consistency in both code
organization and user behavior.
What do you think?
jake
More information about the MPlayer-dev-eng
mailing list