8 Dec 2021, 11:13 by c...@passwd.hu:

>
>
> On Wed, 8 Dec 2021, Lynne wrote:
>
>> 8 Dec 2021, 02:06 by jamr...@gmail.com:
>>
>>>
>>> +enum AVChannel {
>>> +    ///< Invalid channel index
>>> +    AV_CHAN_NONE = -1,
>>> +    AV_CHAN_FRONT_LEFT,
>>>
>>
>> No, not the pixfmt mistake again. Set AV_CHAN_NONE to 0,
>> the rest can follow. Or keep AV_CHAN_NONE to -1
>> and add a new AV_CHAN_UNSPECIFIED as 0.
>>
>>
>>> +    AV_CHAN_FRONT_RIGHT,
>>> +    AV_CHAN_FRONT_CENTER,
>>> +    AV_CHAN_LOW_FREQUENCY,
>>> +    AV_CHAN_BACK_LEFT,
>>> +    AV_CHAN_BACK_RIGHT,
>>> +    AV_CHAN_FRONT_LEFT_OF_CENTER,
>>> +    AV_CHAN_FRONT_RIGHT_OF_CENTER,
>>> +    AV_CHAN_BACK_CENTER,
>>> +    AV_CHAN_SIDE_LEFT,
>>> +    AV_CHAN_SIDE_RIGHT,
>>> +    AV_CHAN_TOP_CENTER,
>>> +    AV_CHAN_TOP_FRONT_LEFT,
>>> +    AV_CHAN_TOP_FRONT_CENTER,
>>> +    AV_CHAN_TOP_FRONT_RIGHT,
>>> +    AV_CHAN_TOP_BACK_LEFT,
>>> +    AV_CHAN_TOP_BACK_CENTER,
>>> +    AV_CHAN_TOP_BACK_RIGHT,
>>> +    /** Stereo downmix. */
>>> +    AV_CHAN_STEREO_LEFT = 29,
>>> +    /** See above. */
>>> +    AV_CHAN_STEREO_RIGHT,
>>> +    AV_CHAN_WIDE_LEFT,
>>> +    AV_CHAN_WIDE_RIGHT,
>>> +    AV_CHAN_SURROUND_DIRECT_LEFT,
>>> +    AV_CHAN_SURROUND_DIRECT_RIGHT,
>>> +    AV_CHAN_LOW_FREQUENCY_2,
>>> +    AV_CHAN_TOP_SIDE_LEFT,
>>> +    AV_CHAN_TOP_SIDE_RIGHT,
>>> +    AV_CHAN_BOTTOM_FRONT_CENTER,
>>> +    AV_CHAN_BOTTOM_FRONT_LEFT,
>>> +    AV_CHAN_BOTTOM_FRONT_RIGHT,
>>> +
>>> +    /** Channel is empty can be safely skipped. */
>>> +    AV_CHAN_SILENCE = 64,
>>> +};
>>>
>>
>> Why is AV_CHAN_SILENCE set to 64? If it's special,
>> set it to follow just after AV_CHAN_NONE or
>> AV_CHAN_UNSPECIFIED.
>>
>
> Because of the channel layout bitmask representation and its relation to the 
> channel ID. We might want to add real channel designations which can be 
> represented using the bitmask layout, and using SILENCE for that does not 
> make a lot of sense. E.g. there can be more than one silent channel. And if 
> somebody wants sparse channel layout then making the extended representation 
> a hard requirement seems also a good idea.
>
> Regards,
> Marton
>
If it's a flag, then I'd like for it to be specified with a hint,
either AV_CHAN_FLAG_SILENCE, or a comment, or
using AV_CHAN_SILENCE = 1 << 16.
_______________________________________________
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".

Reply via email to