Merge changes I366435e2,I64577f8f
* changes: [findBugs] Silence returning null for StringUtils.toBooleanOrNull() [findBugs] Prefer short-cut logic as it's more performant
This commit is contained in:
commit
ce02c82b82
|
@ -45,4 +45,11 @@
|
||||||
<Class name="org.eclipse.jgit.util.TemporaryBuffer$BlockInputStream" />
|
<Class name="org.eclipse.jgit.util.TemporaryBuffer$BlockInputStream" />
|
||||||
<Bug pattern="UR_UNINIT_READ" />
|
<Bug pattern="UR_UNINIT_READ" />
|
||||||
</Match>
|
</Match>
|
||||||
|
|
||||||
|
<!-- Silence returning null for Boolean return type -->
|
||||||
|
<Match>
|
||||||
|
<Class name="org.eclipse.jgit.util.StringUtils" />
|
||||||
|
<Method name="toBooleanOrNull" />
|
||||||
|
<Bug pattern="NP_BOOLEAN_RETURN_NULL" />
|
||||||
|
</Match>
|
||||||
</FindBugsFilter>
|
</FindBugsFilter>
|
||||||
|
|
|
@ -688,7 +688,7 @@ int inflate(DfsPackFile pack, long position, byte[] dstbuf,
|
||||||
for (;;) {
|
for (;;) {
|
||||||
dstoff = block.inflate(inf, position, dstbuf, dstoff);
|
dstoff = block.inflate(inf, position, dstbuf, dstoff);
|
||||||
|
|
||||||
if (headerOnly & dstoff == dstbuf.length)
|
if (headerOnly && dstoff == dstbuf.length)
|
||||||
return dstoff;
|
return dstoff;
|
||||||
if (inf.needsInput()) {
|
if (inf.needsInput()) {
|
||||||
position += block.remaining(position);
|
position += block.remaining(position);
|
||||||
|
|
Loading…
Reference in New Issue