[FFmpeg-devel] [PATCH 3/5] libx265: Support SAR

[Michael Niedermayer]

On Fri, Feb 21, 2014 at 05:15:31PM +0000, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> ---
>  libavcodec/libx265.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c
> index 9392f1b..b16ee38 100644
> --- a/libavcodec/libx265.c
> +++ b/libavcodec/libx265.c
> @@ -77,6 +77,7 @@ static av_cold int libx265_encode_init(AVCodecContext *avctx)
>      libx265Context *ctx = avctx->priv_data;
>      x265_nal *nal;
>      uint8_t *buf;
> +    int sar_num, sar_den;
>      int nnal;
>      int ret;
>      int i;
> @@ -104,6 +105,14 @@ static av_cold int libx265_encode_init(AVCodecContext *avctx)
>      ctx->params->sourceWidth     = avctx->width;
>      ctx->params->sourceHeight    = avctx->height;
>  
> +    av_reduce(&sar_num, &sar_den,
> +              avctx->sample_aspect_ratio.num,
> +              avctx->sample_aspect_ratio.den, 4096);
> +    ctx->params->bEnableAspectRatioIdc = 1;
> +    ctx->params->aspectRatioIdc        = 255;
> +    ctx->params->sarWidth              = sar_num;
> +    ctx->params->sarHeight             = sar_den;

this probably needs some checks on the apsect ratio being valid
and not 0/1 or 0/0


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140221/3ec2c5e1/attachment.asc>