> +int ff_mov_get_channel_config_from_layout(const AVChannelLayout > *layout, int *config) > +{ > + // Set default value which means any setup in 23001-8 > + *config = 0; > + for (int i = 0; i < FF_ARRAY_ELEMS(iso_channel_configuration); > i++) { > + if (!av_channel_layout_compare(layout, > iso_channel_configuration + i)) { > + *config = i; > + break; > + } > + } > + > + return 0;
Is always returning 0 intended? If so then what is the point? ff_mov_get_channel_positions_from_layout() returns non-zero on error.. /Tomas _______________________________________________ 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".