[FFmpeg-devel] [PATCH 5/5] libx265: Use proper error code

Michael Niedermayer michaelni at gmx.at
Fri Feb 21 18:55:31 CET 2014


On Fri, Feb 21, 2014 at 05:15:33PM +0000, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> ---
>  libavcodec/libx265.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c
> index 3bdb19a..ea799a7 100644
> --- a/libavcodec/libx265.c
> +++ b/libavcodec/libx265.c
> @@ -224,7 +224,7 @@ static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
>      ret = x265_encoder_encode(ctx->encoder, &nal, &nnal,
>                                pic ? &x265pic : NULL, &x265pic_out);
>      if (ret < 0)
> -        return AVERROR_UNKNOWN;
> +        return AVERROR_EXTERNAL;

does x265 have any specific error codes, if so we could add a function
to map them to ours, but probably thats overkill

patch LGTM


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

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- 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/6ab4d017/attachment.asc>


More information about the ffmpeg-devel mailing list