Thilo Borgmann wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > >> To reiterate, you don't _need_ this patch to proceed, the ALS decoder >> should parse the no. of channels from extradata and set channels >> parameter correctly. Yes, it should be done on the container side and >> I'll try and get this in soon. But this is never going to block your >> work in any way. >> > > Without it I can't let ffplay demux the files and pass the frames to the > decoder. It would fail opening the SDL audio device. > > Therefore, it is needed if I do want to write the decoder and test its > features. Isn't it?
Well, not specifically required. I did not look at the patch carefully enough. This is needed for stream copy and would be good to have in the demuxer, but is not required for the decoder. As long as ALS is identified in the demuxer and the proper extradata is set, the init function in the ALS decoder should read from the ALSSpecificConfig in the extradata to set the audio parameters. -Justin _______________________________________________ FFmpeg-soc mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
