[FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: fix aspect ratio when writing 16:9 DV frames

[Michael Niedermayer]

On Thu, Sep 14, 2017 at 03:44:41PM +0200, Tobias Rapp wrote:
> Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
> ---
>  libavformat/mxfenc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> index 7289e0b..da4d7b4 100644
> --- a/libavformat/mxfenc.c
> +++ b/libavformat/mxfenc.c
> @@ -1810,7 +1810,7 @@ static int mxf_parse_dv_frame(AVFormatContext *s, AVStream *st, AVPacket *pkt)
>      stype    = vs_pack[3] & 0x1f;
>      pal      = (vs_pack[3] >> 5) & 0x1;
>  
> -    if ((vs_pack[2] & 0x07) == 0x02)
> +    if ((vsc_pack[2] & 0x07) == 0x02)
>          sc->aspect_ratio = (AVRational){ 16, 9 };
>      else
>          sc->aspect_ratio = (AVRational){ 4, 3 };

LGTM

thanks

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

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170915/a98a310c/attachment.sig>