[FFmpeg-devel] [PATCH v2 11/12] lavc/videotoolboxenc: Support pixel aspect ratio

[Carl Eugen Hoyos]

Rick Kern <kernrj <at> gmail.com> writes:

> +        if (status) {
> +            av_log(avctx,
> +                   AV_LOG_ERROR,
> +                   "Error setting pixel aspect ratio to %d:%d: %d.\n",
> +                   avctx->sample_aspect_ratio.num,
> +                   avctx->sample_aspect_ratio.den,
> +                   status);
> +
> +            return AVERROR_EXTERNAL;

Why is it necessary to quit here?
Especially since we don't know what aspects osx accepts.

Carl Eugen