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:

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

2015-03-15 Thread Carl Eugen Hoyos
Hi! 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:8 chl:7.1 to rate:48000 fmt:fltp ch:8 chl:7.1 if the channel layout