From 299253ae1b010eeee81d5a2f291a490627fa482d Mon Sep 17 00:00:00 2001 From: James Almer Date: Sun, 25 Sep 2022 20:00:46 -0300 Subject: [PATCH] avutil/channel_layout: move and improve the comment about unknown orders Don't place it as doxy specific for the order field, and generalize it both to also cover already defined orders and to not make it seem like the user is required to handle a layout they don't fully support or understand. Signed-off-by: James Almer --- libavutil/channel_layout.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavutil/channel_layout.h b/libavutil/channel_layout.h index 9e685fab72..ff2b57bbe3 100644 --- a/libavutil/channel_layout.h +++ b/libavutil/channel_layout.h @@ -266,6 +266,9 @@ typedef struct AVChannelCustom { * A channel layout here is defined as a set of channels ordered in a specific * way (unless the channel order is AV_CHANNEL_ORDER_UNSPEC, in which case an * AVChannelLayout carries only the channel count). + * All orders may be treated as if they were AV_CHANNEL_ORDER_UNSPEC by + * ignoring everything but the channel count, as long as av_channel_layout_check() + * considers they are valid. * * Unlike most structures in Libav, sizeof(AVChannelLayout) is a part of the * public ABI and may be used by the caller. E.g. it may be allocated on stack @@ -290,9 +293,6 @@ typedef struct AVChannelCustom { typedef struct AVChannelLayout { /** * Channel order used in this layout. - * Any value not defined in the AVChannelOrder enum in a layout that - * av_channel_layout_check() doesn't reject must be treated as if it was - * AV_CHANNEL_ORDER_UNSPEC. * This is a mandatory field. */ enum AVChannelOrder order;