Re: [FFmpeg-devel] [PATCH 12/13] lavc/videotoolboxenc: Support pixel aspect ratio

2016-04-22 Thread Richard Kern
> On Apr 10, 2016, at 12:24 AM, Carl Eugen Hoyos wrote: > > Rick Kern gmail.com> writes: > >> +if (status) { >> +av_log( >> + avctx, >> + AV_LOG_ERROR, >> + "Error setting pixel aspect ratio to %d:%d: %d.\n", > > Why wo

Re: [FFmpeg-devel] [PATCH 12/13] lavc/videotoolboxenc: Support pixel aspect ratio

2016-04-09 Thread Carl Eugen Hoyos
Rick Kern gmail.com> writes: > +if (status) { > +av_log( > + avctx, > + AV_LOG_ERROR, > + "Error setting pixel aspect ratio to %d:%d: %d.\n", Why would the function call fail? I would expect that the pixel aspect ratio ha

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

2016-04-09 Thread Rick Kern
Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 54 1 file changed, 54 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 0a79ee0..72743b5 100644 --- a/libavcodec/videotoolboxenc.c +++ b/libavcodec