[FFmpeg-devel] [PATCH] avformat/mpeg12enc: add support for specifying video_format in the sequence_display_extension

[Michael Niedermayer]

On Sat, Feb 10, 2018 at 07:45:36PM +0100, Marton Balint wrote:
> In a recent commit the default was changed from 0 (component) to 5
> (unspecified), however some standards require using 0. With this option, the
> user will be able to do so.
> 
> Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
>  doc/encoders.texi      |  5 +++++
>  libavcodec/mpeg12enc.c | 12 ++++++++++--
>  libavcodec/mpegvideo.h |  7 +++++++
>  libavcodec/version.h   |  2 +-
>  4 files changed, 23 insertions(+), 3 deletions(-)

it would be better to pass this on from the input but as theres no support
for that, the patch LGTM

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180210/b0342aa2/attachment.sig>