On Fri, May 17, 2013 at 12:36:30PM +0200, Luca Barbato wrote:
> ---
>  libavcodec/jpeglsdec.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/libavcodec/jpeglsdec.c b/libavcodec/jpeglsdec.c
> index d4ecf3d..ca85def 100644
> --- a/libavcodec/jpeglsdec.c
> +++ b/libavcodec/jpeglsdec.c
> @@ -68,13 +68,13 @@ int ff_jpegls_decode_lse(MJpegDecodeContext *s)
>      case 2:
>      case 3:
>          av_log(s->avctx, AV_LOG_ERROR, "palette not supported\n");
> -        return -1;
> +        return AVERROR(ENOSYS);

patchwelcome for this particular case?

>      case 4:
>          av_log(s->avctx, AV_LOG_ERROR, "oversize image not supported\n");
> -        return -1;
> +        return AVERROR(ENOSYS);
>      default:
>          av_log(s->avctx, AV_LOG_ERROR, "invalid id %d\n", id);
> -        return -1;
> +        return AVERROR_INVALIDDATA;
>      }
>      av_dlog(s->avctx, "ID=%i, T=%i,%i,%i\n", id, s->t1, s->t2, s->t3);
>  
> @@ -350,7 +350,7 @@ int ff_jpegls_decode_picture(MJpegDecodeContext *s, int 
> near,
>                 "Sample interleaved images are not supported.\n");
>          av_free(state);
>          av_free(zero);
> -        return -1;
> +        return AVERROR(ENOSYS);
>      }
>  
>      if (shift) { /* we need to do point transform or normalize samples */
> -- 

LGTM
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to