[GitHub] commons-compress pull request #59: 10% speed improvements and decreased memo...

[dweiss <gi...@git.apache.org>]

GitHub user dweiss opened a pull request:

    https://github.com/apache/commons-compress/pull/59

    10% speed improvements and decreased memory allocation rate in deflate64

    Just something to consider: don't reallocate the run buffer over and over (this is a very hot loop). I piggybacked removal of long-to-byte masking {{0xff}} since the cast is sufficient on its own (truncates to byte value anyway).

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dweiss/commons-compress master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/commons-compress/pull/59.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #59
    
----
commit a40231b2bf2d1b3d73726ec3dae8fdc14fa672ad
Author: Dawid Weiss <da...@...>
Date:   2018-01-11T17:11:11Z

    10% speed improvements and decreased memory allocation rate in deflate64

----


---

[GitHub] commons-compress pull request #59: 10% speed improvements and decreased memo...

[asfgit <gi...@git.apache.org>]

Github user asfgit closed the pull request at:

    https://github.com/apache/commons-compress/pull/59


---

[GitHub] commons-compress issue #59: 10% speed improvements and decreased memory allo...

[coveralls <gi...@git.apache.org>]

Github user coveralls commented on the issue:

    https://github.com/apache/commons-compress/pull/59
  
    
    [![Coverage Status](https://coveralls.io/builds/15007315/badge)](https://coveralls.io/builds/15007315)
    
    Coverage decreased (-0.02%) to 86.045% when pulling **a40231b2bf2d1b3d73726ec3dae8fdc14fa672ad on dweiss:master** into **49f6547038ca5e0c9aed0db061dff518cfaf240e on apache:master**.



---