Skip to content

Commit 2ec3c51

Browse files
committed
Findbugs fixes
1 parent be2134f commit 2ec3c51

File tree

4 files changed

+12
-6
lines changed

4 files changed

+12
-6
lines changed

build.gradle

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,6 @@ jar {
5656

5757
test {
5858
maxHeapSize = "1g"
59-
// forkEvery = 50
60-
// maxParallelForks = Runtime.runtime.availableProcessors()
6159
}
6260

6361
checkstyle {
@@ -67,7 +65,7 @@ checkstyle {
6765

6866
findbugs {
6967
ignoreFailures = true
70-
effort = 'max'
68+
//reportLevel = 'low'
7169
excludeFilter = file('config/findbugs/findbugs-exclude.xml')
7270
}
7371

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<FindBugsFilter>
22
<Match>
3-
<Bug category="MALICIOUS_CODE" />
3+
<Bug category="MALICIOUS_CODE"/>
44
</Match>
55
<Match>
66
<Bug pattern="OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE"/>
@@ -9,4 +9,11 @@
99
<Class name="com.imsweb.staging.IntegrationUtils"/>
1010
<Bug pattern="RV_RETURN_VALUE_IGNORED_BAD_PRACTICE"/>
1111
</Match>
12+
<Match>
13+
<Bug pattern="EI_EXPOSE_REP"/>
14+
</Match>
15+
<Match>
16+
<Bug pattern="EI_EXPOSE_REP2"/>
17+
</Match>
18+
1219
</FindBugsFilter>

src/main/java/com/imsweb/decisionengine/DecisionEngine.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public class DecisionEngine {
3232
private static Pattern _TEMPLATE_REFERENCE = Pattern.compile("\\{\\{(.*?)\\}\\}");
3333

3434
// string to use for blank or null in error strings
35-
public static String _BLANK_OUTPUT = "<blank>";
35+
public static final String _BLANK_OUTPUT = "<blank>";
3636

3737
private DataProvider _provider;
3838

src/main/java/com/imsweb/staging/Staging.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import java.util.ArrayList;
77
import java.util.Arrays;
88
import java.util.Calendar;
9+
import java.util.Collections;
910
import java.util.HashMap;
1011
import java.util.HashSet;
1112
import java.util.Iterator;
@@ -36,7 +37,7 @@ public final class Staging {
3637
public static final String CTX_YEAR_CURRENT = "ctx_year_current";
3738

3839
// list of all context keys
39-
public static final List<String> CONTEXT_KEYS = Arrays.asList(CTX_ALGORITHM_VERSION, CTX_YEAR_CURRENT);
40+
public static final List<String> CONTEXT_KEYS = Collections.unmodifiableList(Arrays.asList(CTX_ALGORITHM_VERSION, CTX_YEAR_CURRENT));
4041

4142
private DecisionEngine _engine = null;
4243
private StagingDataProvider _provider = null;

0 commit comments

Comments
 (0)