Re: [FFmpeg-devel] [PATCH]Correctly read RLE encoding from dpx files

2015-02-07 Thread Carl Eugen Hoyos
Christophe Gisquet gmail.com> writes: > 2015-02-07 16:47 GMT+01:00 Carl Eugen Hoyos: > > Attached patch intends to fix reading the > > RLE-attribute from dpx files. > > I don't think this is valid. You are skipping > a byte, as if there was an additional element > to skip. But there is none.

Re: [FFmpeg-devel] [PATCH]Correctly read RLE encoding from dpx files

2015-02-07 Thread Christophe Gisquet
Hi, 2015-02-07 16:47 GMT+01:00 Carl Eugen Hoyos : > Attached patch intends to fix reading the RLE-attribute from dpx files. I don't think this is valid. You are skipping a byte, as if there was an additional element to skip. But there is none. The bitstream is really: bits per component [1 byte]

[FFmpeg-devel] [PATCH]Correctly read RLE encoding from dpx files

2015-02-07 Thread Carl Eugen Hoyos
Hi! Attached patch intends to fix reading the RLE-attribute from dpx files. Please review, Carl Eugen diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index 3b78486..8584a03 100644 --- a/libavcodec/dpx.c +++ b/libavcodec/dpx.c @@ -136,6 +136,7 @@ static int decode_frame(AVCodecContext *avctx,