Re: [FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-08-08 Thread Carl Eugen Hoyos
2016-08-08 20:58 GMT+02:00 Andrey Utkin : > On Thu, Jul 14, 2016 at 05:27:08PM +, Carl Eugen Hoyos wrote: >> Andrey Utkin fastmail.com> writes: >> >> > It is impossible to pass "aspect" parameter to encoder from ffmpeg >> > CLI because option from lavc/options_table.h is eclipsed by option >>

Re: [FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-08-08 Thread Andrey Utkin
On Thu, Jul 14, 2016 at 05:27:08PM +, Carl Eugen Hoyos wrote: > Andrey Utkin fastmail.com> writes: > > > It is impossible to pass "aspect" parameter to encoder from ffmpeg > > CLI because option from lavc/options_table.h is eclipsed by option > > with same name in ffmpeg_opt.c, which has di

Re: [FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-07-14 Thread Carl Eugen Hoyos
Andrey Utkin fastmail.com> writes: > It is impossible to pass "aspect" parameter to encoder from ffmpeg > CLI because option from lavc/options_table.h is eclipsed by option > with same name in ffmpeg_opt.c, which has different meaning (DAR, > not SAR). Wasn't the old behaviour intended becaus

Re: [FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-05-04 Thread Michael Niedermayer
On Wed, May 04, 2016 at 08:40:04AM +0300, Andrey Utkin wrote: > On Wed, May 04, 2016 at 05:07:10AM +0200, Michael Niedermayer wrote: > > doesnt this require an update to teh docs ? > > Indeed, thanks for commenting. > > But I wonder how should I format reference to "aspect" correctly and pretty.

Re: [FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-05-03 Thread Andrey Utkin
On Wed, May 04, 2016 at 05:07:10AM +0200, Michael Niedermayer wrote: > doesnt this require an update to teh docs ? Indeed, thanks for commenting. But I wonder how should I format reference to "aspect" correctly and pretty. Now I have come up with this, which I think is not perfect. Any help is v

Re: [FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-05-03 Thread Michael Niedermayer
On Wed, May 04, 2016 at 03:06:31AM +0300, Andrey Utkin wrote: > It is impossible to pass "aspect" parameter to encoder from ffmpeg CLI > because option from lavc/options_table.h is eclipsed by option with same > name in ffmpeg_opt.c, which has different meaning (DAR, not SAR). > --- > > v2: Update

[FFmpeg-devel] [PATCH v2] Add "sar" alias to "aspect" option of video encoders

2016-05-03 Thread Andrey Utkin
It is impossible to pass "aspect" parameter to encoder from ffmpeg CLI because option from lavc/options_table.h is eclipsed by option with same name in ffmpeg_opt.c, which has different meaning (DAR, not SAR). --- v2: Updated tests reference results as suggested by Michael Niedermayer. libavcode