On Fri, May 17, 2013 at 01:26:58PM +0200, Luca Barbato wrote:
> ---
> libavcodec/jpeglsdec.c | 11 +++++------
> 1 file changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/libavcodec/jpeglsdec.c b/libavcodec/jpeglsdec.c
> index e31329b..8eacf04 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);
> 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);
>
> @@ -346,11 +346,10 @@ int ff_jpegls_decode_picture(MJpegDecodeContext *s, int
> near,
> cur += s->picture_ptr->linesize[0];
> }
> } else if (ilv == 2) { /* sample interleaving */
> - av_log(s->avctx, AV_LOG_ERROR,
> - "Sample interleaved images are not supported.\n");
> + avpriv_report_missing_feature(s->avctx, "Sample interleaved images");
> av_free(state);
> av_free(zero);
> - return -1;
> + return AVERROR_PATCHWELCOME;
> }
>
> 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
