flac dev - Aug 2012 - Git branch with compiling fixes for win32

On Fri, May 04, 2012 at 05:53:23PM +0200, Miroslav Lichvar
wrote:
> The most interesting part of the patch is the rewrite of the
> FLAC__bitreader_read_rice_signed_block function, which in the git repo
> seems to have only couple lines changed since 1.2.1.
Here is that part of the patch rebased against current git. In a quick
test it gives a 10% speedup in decoding.

-- 
Miroslav Lichvar
-------------- next part --------------

El 04/05/12 12:09, Miroslav Lichvar escribi?:
> On Fri, May 04, 2012 at 05:53:23PM +0200, Miroslav Lichvar wrote:
>> The most interesting part of the patch is the rewrite of the
>> FLAC__bitreader_read_rice_signed_block function, which in the git repo
>> seems to have only couple lines changed since 1.2.1.
>
> Here is that part of the patch rebased against current git. In a quick
> test it gives a 10% speedup in decoding.
>
Looking at the openSUSE trees, we carry this patch since Aug 2011 and 
have recieved no reports on it being broken, also it works fine in my 
machine ;)

>From: Miroslav Lichvar <mlichvar at redhat.com>
>Sent: Friday, May 4, 2012 9:09 AM
> 
>On Fri, May 04, 2012 at 05:53:23PM +0200, Miroslav Lichvar wrote:
>> The most interesting part of the patch is the rewrite of the
>> FLAC__bitreader_read_rice_signed_block function, which in the git repo
>> seems to have only couple lines changed since 1.2.1.
>
>Here is that part of the patch rebased against current git. In a quick
>test it gives a 10% speedup in decoding.
Excellent!

On Fri, May 04, 2012 at 06:09:06PM +0200, Miroslav Lichvar
wrote:
> On Fri, May 04, 2012 at 05:53:23PM +0200, Miroslav Lichvar wrote:
> > The most interesting part of the patch is the rewrite of the
> > FLAC__bitreader_read_rice_signed_block function, which in the git repo
> > seems to have only couple lines changed since 1.2.1.
> 
> Here is that part of the patch rebased against current git. In a quick
> test it gives a 10% speedup in decoding.
Will be this patch included in the upcoming release? Or does it need
more work before it can be accepted?

-- 
Miroslav Lichvar

Miroslav Lichvar wrote:
> Will be this patch included in the upcoming release? Or does it need
> more work before it can be accepted?
The patch does not currently apply against git HEAD.

If you or someone ports it to apply cleanly to HEAD I don't see
any reason why it won't be applied.

Cheers,
Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/