[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).

[bodewig]

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 <xenoam...@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;
                 }
             }
         }