Re: [FFmpeg-devel] [PATCH]lavc/flicvideo: Implement padding in COPY chunks.

2016-03-26 Thread Carl Eugen Hoyos
Reimar Döffinger gmx.de> writes: > > +if (s->avctx->width & 3) > > +bytestream2_skip(&g2, 4 - (s->avctx->width & 3)); > > Maybe it's more reliable/clear to keep like that Yes, it definitely is imo;-) > but I think this could also be written as > bytestream2_skip(&g2, -s-≥avctx->wid

Re: [FFmpeg-devel] [PATCH]lavc/flicvideo: Implement padding in COPY chunks.

2016-03-25 Thread Reimar Döffinger
On Thu, Mar 24, 2016 at 01:40:07AM +0100, Carl Eugen Hoyos wrote: > @@ -432,6 +432,8 @@ static int flic_decode_frame_8BPP(AVCodecContext *avctx, > y_ptr += s->frame->linesize[0]) { > bytestream2_get_buffer(&g2, &pixels[y_ptr], >

Re: [FFmpeg-devel] [PATCH]lavc/flicvideo: Implement padding in COPY chunks.

2016-03-25 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > Attached patch fixes ticket #5364 for me. Fixes several different samples, I'll push if nobody objects. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-d

[FFmpeg-devel] [PATCH]lavc/flicvideo: Implement padding in COPY chunks.

2016-03-23 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #5364 for me. Please review, Carl Eugen diff --git a/libavcodec/flicvideo.c b/libavcodec/flicvideo.c index 3e0573a..7535a40 100644 --- a/libavcodec/flicvideo.c +++ b/libavcodec/flicvideo.c @@ -423,7 +423,7 @@ static int flic_decode_frame_8BPP(AVCodecContext *avctx,