[FFmpeg-devel] [PATCH] lavfi/buffersrc: set channel layout if it is known.

Michael Niedermayer michaelni at gmx.at
Sun Mar 17 01:38:16 CET 2013


On Sun, Mar 17, 2013 at 01:03:27AM +0100, Nicolas George wrote:
> Le sextidi 26 ventôse, an CCXXI, Michael Niedermayer a écrit :
> > yes until someone removes that to allow midstream configuration
> > changes
> 
> If/when that happens, whoever does it will make sure that c->channels is
> updated as well as c->channel_layout. And if that comes from the fork,
> removing these two lines should be the easiest part of the merge process.
> 
> Would you like it better with a clear comment about the reason for this
> test?
> 

> Also, note that the inconsistent channels/channel_layout you are afraid of
> is caught by an assertion a few cycles later.

if thats the case then i have no objections

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130317/266be54a/attachment.asc>


More information about the ffmpeg-devel mailing list