Re: [libav-devel] [PATCH] OpenEXR decoder

2014-03-31 Thread Vittorio Giovara
On Wed, Mar 26, 2014 at 4:39 AM, Vittorio Giovara vittorio.giov...@gmail.com wrote: On Sun, Mar 23, 2014 at 4:07 PM, Vittorio Giovara vittorio.giov...@gmail.com wrote: From: Jimmy Christensen ji...@ghost.dk Additional fixes and enhancements by Vittorio Giovara, Gonzalo Garramuno, Nicolas

Re: [libav-devel] [PATCH] OpenEXR decoder

2014-03-25 Thread Vittorio Giovara
On Sun, Mar 23, 2014 at 4:07 PM, Vittorio Giovara vittorio.giov...@gmail.com wrote: From: Jimmy Christensen ji...@ghost.dk Additional fixes and enhancements by Vittorio Giovara, Gonzalo Garramuno, Nicolas George, Paul B Mahol and Michael Niedermayer. Signed-off-by: Vittorio Giovara

Re: [libav-devel] [PATCH] OpenEXR decoder

2014-03-23 Thread Justin Ruggles
On 03/23/2014 11:07 AM, Vittorio Giovara wrote: +// skip any number of 0 until you get to the pixel type +while (bytestream2_get_bytes_left(ch_gb) 0 + bytestream2_get_byte(ch_gb)) +continue; That doesn't do what it

Re: [libav-devel] [PATCH] OpenEXR decoder

2014-03-23 Thread Vittorio Giovara
On Sun, Mar 23, 2014 at 6:55 PM, Justin Ruggles justin.rugg...@gmail.com wrote: On 03/23/2014 11:07 AM, Vittorio Giovara wrote: +// skip any number of 0 until you get to the pixel type +while (bytestream2_get_bytes_left(ch_gb) 0 +

[libav-devel] [PATCH] OpenEXR decoder

2014-03-20 Thread Vittorio Giovara
From: Jimmy Christensen ji...@ghost.dk Additional fixes and enhancements by Vittorio Giovara, Gonzalo Garramuno, Nicolas George, Paul B Mahol and Michael Niedermayer. Signed-off-by: Vittorio Giovara vittorio.giov...@gmail.com --- I ported the header parsing to some sane bytestream2 and moved it

Re: [libav-devel] [PATCH] OpenEXR decoder

2014-03-20 Thread Justin Ruggles
On 03/20/2014 09:01 PM, Vittorio Giovara wrote: +// Process unknown variables +for (i = 0; i 2; i++) // value_name and value_type +while (bytestream2_get_bytes_left(s-gb) 0) +if (bytestream2_get_byte(s-gb) == 0) +break; If

Re: [libav-devel] [PATCH] OpenEXR decoder

2014-03-20 Thread Vittorio Giovara
On Fri, Mar 21, 2014 at 2:59 AM, Justin Ruggles justin.rugg...@gmail.com wrote: On 03/20/2014 09:01 PM, Vittorio Giovara wrote: +// Process unknown variables +for (i = 0; i 2; i++) // value_name and value_type +while (bytestream2_get_bytes_left(s-gb) 0) +

[libav-devel] [PATCH] OpenEXR decoder

2014-03-14 Thread Vittorio Giovara
Additional fixes and enhancements by Gonzalo Garramuno, Nicolas George, Paul B Mahol and Michael Niedermayer. Signed-off-by: Vittorio Giovara vittorio.giov...@gmail.com --- Wrong fixup, sorry, here is the updated version. Vittorio Changelog |1 + configure |1 +

Re: [libav-devel] [PATCH] OpenEXR decoder

2014-03-14 Thread Diego Biurrun
On Fri, Mar 14, 2014 at 04:25:57PM +0100, Vittorio Giovara wrote: Additional fixes and enhancements by Gonzalo Garramuno, Nicolas George, Changelog |1 + configure |1 + doc/general.texi |2 + libavcodec/Makefile|1 +

[libav-devel] [PATCH] OpenEXR decoder

2014-03-14 Thread Vittorio Giovara
Additional fixes and enhancements by Gonzalo Garramuno, Nicolas George, Paul B Mahol and Michael Niedermayer. Signed-off-by: Vittorio Giovara vittorio.giov...@gmail.com --- Added codec_desc. Vittorio Changelog |1 + configure |1 + doc/general.texi|

[libav-devel] [PATCH] OpenEXR decoder

2014-03-13 Thread Vittorio Giovara
Additional fixes and enhancements by Gonzalo Garramuno, Nicolas George, Paul B Mahol and Michael Niedermayer. Signed-off-by: Vittorio Giovara vittorio.giov...@gmail.com --- Here are most of the corrections requested. Memory cleanup on error is moslty performed by the decoder's uninit function.