[commons-compress] branch master updated: maybe we can add a break here Hi. Can we add a break here? Also, should we try implement a data structure or something for doing this? a loop like this sounds slow. If you don't mind I will give it a try several hours later (if I still have some time).

[bo...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

bodewig pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-compress.git


The following commit(s) were added to refs/heads/master by this push:
     new 204faf5  maybe we can add a break here Hi. Can we add a break here? Also, should we try implement a data structure or something for doing this? a loop like this sounds slow. If you don't mind I will give it a try several hours later (if I still have some time).
     new 497cb94  Merge pull request #106 from XenoAmess/should_we_add_break_here
204faf5 is described below

commit 204faf58f69386d1b6f8d0410dcf2667d6d08051
Author: XenoAmess <xe...@gmail.com>
AuthorDate: Wed Jun 3 19:24:51 2020 +0800

    maybe we can add a break here
    Hi.
    Can we add a break here?
    Also, should we try implement a data structure or something for doing this? a loop like this sounds slow.
    If you don't mind I will give it a try several hours later (if I still have some time).
---
 .../commons/compress/compressors/bzip2/BZip2CompressorOutputStream.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/main/java/org/apache/commons/compress/compressors/bzip2/BZip2CompressorOutputStream.java b/src/main/java/org/apache/commons/compress/compressors/bzip2/BZip2CompressorOutputStream.java
index a6e811b..f92bb5f 100644
--- a/src/main/java/org/apache/commons/compress/compressors/bzip2/BZip2CompressorOutputStream.java
+++ b/src/main/java/org/apache/commons/compress/compressors/bzip2/BZip2CompressorOutputStream.java
@@ -975,6 +975,7 @@ public class BZip2CompressorOutputStream extends CompressorOutputStream
             for (int j = 16; --j >= 0;) {
                 if (inUse[i16 + j]) {
                     inUse16[i] = true;
+                    break;
                 }
             }
         }