On Wed, Dec 8, 2021 at 10:14 AM Lynne <d...@lynne.ee> 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.
>

Care to elaborate on the reasons of this opinion? Using -1 as invalid
and 0...x as valid entries seems quite reasonable to me.

- Hendrik
_______________________________________________
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