Marton Balint (12020-08-20): > Why it is better if we don't set it? 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. 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".
