Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 08:08:10AM +0200, Reimar Döffinger wrote: > On 19 September 2014 07:34:11 CEST, Pascal Massimino > wrote: > >Hi Reimar, > >while i don't necessarily disagree with the above generally speaking > >[*], > >let me repeat that > >this is a pragmatic choice. I'm not going to giv

Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-19 Thread Michael Niedermayer
On Thu, Sep 18, 2014 at 10:34:11PM -0700, Pascal Massimino wrote: > Hi Reimar, > > On Thu, Sep 18, 2014 at 12:28 PM, Reimar Döffinger > wrote: > > > On 18 September 2014 10:55:00 CEST, Pascal Massimino < > > pascal.massim...@gmail.com> wrote: > > >Hi Reimar, > > > > > >On Thu, Sep 18, 2014 at 9:

Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-18 Thread Reimar Döffinger
On 19 September 2014 07:34:11 CEST, Pascal Massimino wrote: >Hi Reimar, >while i don't necessarily disagree with the above generally speaking >[*], >let me repeat that >this is a pragmatic choice. I'm not going to give up on the speed-up i >have >right now for >some hypothetical decoder later. I

Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-18 Thread Pascal Massimino
Hi Reimar, On Thu, Sep 18, 2014 at 12:28 PM, Reimar Döffinger wrote: > On 18 September 2014 10:55:00 CEST, Pascal Massimino < > pascal.massim...@gmail.com> wrote: > >Hi Reimar, > > > >On Thu, Sep 18, 2014 at 9:33 AM, Reimar Döffinger > > > >wrote: > > > >> On 18.09.2014, at 08:45, Pascal Massimi

Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-18 Thread Reimar Döffinger
On 18 September 2014 10:55:00 CEST, Pascal Massimino wrote: >Hi Reimar, > >On Thu, Sep 18, 2014 at 9:33 AM, Reimar Döffinger > >wrote: > >> On 18.09.2014, at 08:45, Pascal Massimino > >> wrote: >> > Hi, >> > >> > the webp lossless doc was unclear regarding palette index falling >out of >> > range

Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-18 Thread Pascal Massimino
Hi Reimar, On Thu, Sep 18, 2014 at 9:33 AM, Reimar Döffinger wrote: > On 18.09.2014, at 08:45, Pascal Massimino > wrote: > > Hi, > > > > the webp lossless doc was unclear regarding palette index falling out of > > range. > > See issue #206 [1] for the bug report. > > The solution retained was t

Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-18 Thread Reimar Döffinger
On 18.09.2014, at 08:45, Pascal Massimino wrote: > Hi, > > the webp lossless doc was unclear regarding palette index falling out of > range. > See issue #206 [1] for the bug report. > The solution retained was to treat out-of-range index as color 0x, > so we could keep the speed-up in lib

[FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-17 Thread Pascal Massimino
Hi, the webp lossless doc was unclear regarding palette index falling out of range. See issue #206 [1] for the bug report. The solution retained was to treat out-of-range index as color 0x, so we could keep the speed-up in libwebp (we use fake extra entries in the cmap to handle out-of-bou