[FFmpeg-soc] [soc]: r4387 - als/ffmpeg-patches/hack4als.patch

[Justin Ruggles]

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