[FFmpeg-devel] [PATCH]Move MLP channel layout setting from parser to decoder

[John Stebbins]

On 04/12/2011 02:38 PM, Carl Eugen Hoyos wrote:
> Hi!
>
> This allows the mlp channel re-order patch to also work with other
> applications than ffmpeg/ffplay.
>
> Please comment, Carl Eugen
Is there an order of operations problem here?  channel_layout gets set before channels in read_major_sync.  If channels 
hasn't been set yet, doesn't the test for av_get_channel_layout_nb_channels(channel_layout) != channels fail the first 
time through?