Marton Balint (12020-08-20): > > It is how the API was supposed to be used from the start: these values > > set the formats list for buffersink. If you said you accept anything > > with two channels, then you do not need to say you accept stereo, it is > > already said. It was already documented in formats.h: > > > > * - The list must not contain a layout with a known disposition and a > > * channel count with unknown disposition with the same number of channels > > * (e.g. AV_CH_LAYOUT_STEREO and FF_COUNT2LAYOUT(2). > > > > Of course, a private header is not the proper place to document a public > > API, which is what I correct in patch #2. > > > > In practice, applications know if they want a number of channels or a > > specific channel layout, and should only set one. In this case I am not > > familiar enough with SDL's API to be sure, so I made the minimal change. > > LGTM, thanks.
Thanks, pushed. Regards, -- Nicolas George
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
