diff options
author | Marc R. Hoffmann <hoffmann@mountainminds.com> | 2015-01-07 21:46:13 +0100 |
---|---|---|
committer | Marc R. Hoffmann <hoffmann@mountainminds.com> | 2015-01-07 21:46:13 +0100 |
commit | 6e5a8197072aad904473a1850cb06ce9e7006cd8 (patch) | |
tree | 482be676f6e55263829b940a88a23492a535b145 /org.jacoco.core | |
parent | 6ab169bf46e98d5d6bce822a56f2c588696fbd7a (diff) | |
download | jacoco-6e5a8197072aad904473a1850cb06ce9e7006cd8.tar.gz |
Fix some Sonar warnings.
Diffstat (limited to 'org.jacoco.core')
-rw-r--r-- | org.jacoco.core/src/org/jacoco/core/internal/flow/Instruction.java | 3 | ||||
-rw-r--r-- | org.jacoco.core/src/org/jacoco/core/internal/instr/FieldProbeArrayStrategy.java | 4 |
2 files changed, 4 insertions, 3 deletions
diff --git a/org.jacoco.core/src/org/jacoco/core/internal/flow/Instruction.java b/org.jacoco.core/src/org/jacoco/core/internal/flow/Instruction.java index 99c242b0..1b082334 100644 --- a/org.jacoco.core/src/org/jacoco/core/internal/flow/Instruction.java +++ b/org.jacoco.core/src/org/jacoco/core/internal/flow/Instruction.java @@ -63,7 +63,8 @@ public class Instruction { * branch. */ public void setCovered() { - for (Instruction i = this; i != null && i.coveredBranches++ == 0;) { + Instruction i = this; + while (i != null && i.coveredBranches++ == 0) { i = i.predecessor; } } diff --git a/org.jacoco.core/src/org/jacoco/core/internal/instr/FieldProbeArrayStrategy.java b/org.jacoco.core/src/org/jacoco/core/internal/instr/FieldProbeArrayStrategy.java index 98284321..6259aa38 100644 --- a/org.jacoco.core/src/org/jacoco/core/internal/instr/FieldProbeArrayStrategy.java +++ b/org.jacoco.core/src/org/jacoco/core/internal/instr/FieldProbeArrayStrategy.java @@ -27,12 +27,12 @@ class FieldProbeArrayStrategy implements IProbeArrayStrategy { /** * Frame stack with a single boolean array. */ - public static final Object[] FRAME_STACK_ARRZ = new Object[] { InstrSupport.DATAFIELD_DESC }; + private static final Object[] FRAME_STACK_ARRZ = new Object[] { InstrSupport.DATAFIELD_DESC }; /** * Empty frame locals. */ - public static final Object[] FRAME_LOCALS_EMPTY = new Object[0]; + private static final Object[] FRAME_LOCALS_EMPTY = new Object[0]; private final String className; private final long classId; |