Re: [FFmpeg-devel] [PATCH]Fix changed channel layout log info

2015-03-16 Thread Michael Niedermayer
On Mon, Mar 16, 2015 at 05:34:59PM +, Carl Eugen Hoyos wrote: > Nicolas George nsup.org> writes: > > > The line you change seems completely wrong by itself. > > IMHO, it should read something like > > "av_assert0(decoded_frame->channel_layout == > > avctx->channel_layout);", to allow findin

Re: [FFmpeg-devel] [PATCH]Fix changed channel layout log info

2015-03-16 Thread Carl Eugen Hoyos
Nicolas George nsup.org> writes: > The line you change seems completely wrong by itself. > IMHO, it should read something like > "av_assert0(decoded_frame->channel_layout == > avctx->channel_layout);", to allow finding the places > where lavc returns inconsistent layouts. That happens for dca

Re: [FFmpeg-devel] [PATCH]Fix changed channel layout log info

2015-03-16 Thread Nicolas George
Le sextidi 26 ventôse, an CCXXIII, Carl Eugen Hoyos a écrit : > It seems to me that the line that is shown when the channel > layout changes uses an incorrect new layout. > The current output can be (line breaks for readability): > > Input stream #0:1 frame changed from > rate:48000 fmt:fltp ch: