[FFmpeg-devel] [PATCH] exr: piz decompression
Paul B Mahol
onemda at gmail.com
Wed Mar 6 13:54:13 CET 2013
On 3/5/13, Reimar Doeffinger <Reimar.Doeffinger at gmx.de> wrote:
> On Tue, Mar 05, 2013 at 08:06:37PM +0000, Paul B Mahol wrote:
>> On 3/5/13, Reimar Doeffinger <Reimar.Doeffinger at gmx.de> wrote:
>> > On Tue, Mar 05, 2013 at 01:47:43PM +0000, Paul B Mahol wrote:
>> >> On 2/27/13, Paul B Mahol <onemda at gmail.com> wrote:
>> >> > Signed-off-by: Paul B Mahol <onemda at gmail.com>
>> >> > ---
>> >> > libavcodec/exr.c | 477
>> >> > +++++++++++++++++++++++++++++++++++++++++++++++++++++++
>> >> > 1 file changed, 477 insertions(+)
>> >> >
>> >> ping
>> > Well, it's a good bit of code so I don't have time to properly
>> > review, but maybe you explain why it's not possible to reuse
>> > the existing huffman and/or VLC code?
>> This one is 16 bit, and other huffman code looks different.
> Well, it looks that this one specifies the coding by the lengths,
> I can't find a table for that.
Table for what?
> But the original symbols are 16 bit for the existing huffman code,
> so I don't know what you mean by that?
> Also I was also wondering about why it isn't reusing the get_vlc code,
> because that one looks flexible enough to me to reuse it.
I can't see how get_vlc is gonna be useful.
> Though I admit I don't know how code it would save in the end.
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
More information about the ffmpeg-devel