Small refactor to improve all test failures. Probably defining the default
severity level in the SeverityLevel class is not the best thing to do - but I could not think of an other obvious place.
This commit is contained in:
parent
26c4b0998f
commit
d4cee63283
|
|
@ -48,7 +48,7 @@ public abstract class Check extends AutomaticBean
|
|||
private int mTabWidth = 8; // meaningful default
|
||||
|
||||
/** the severity level of any violations found */
|
||||
private SeverityLevel mSeverityLevel = SeverityLevel.ERROR;
|
||||
private SeverityLevel mSeverityLevel = SeverityLevel.DEFAULT_LEVEL;
|
||||
|
||||
/** current class loader */
|
||||
private ClassLoader mLoader =
|
||||
|
|
|
|||
|
|
@ -55,8 +55,6 @@ public final class LocalizedMessage
|
|||
|
||||
/** the severity level **/
|
||||
private final SeverityLevel mSeverityLevel;
|
||||
/** the default severity level if one is not specified */
|
||||
private static final SeverityLevel DEFAULT_SEVERITY = SeverityLevel.WARNING;
|
||||
|
||||
/** key for the message format **/
|
||||
private final String mKey;
|
||||
|
|
@ -129,7 +127,7 @@ public final class LocalizedMessage
|
|||
mKey = aKey;
|
||||
mArgs = aArgs;
|
||||
mBundle = aBundle;
|
||||
mSeverityLevel = DEFAULT_SEVERITY;
|
||||
mSeverityLevel = SeverityLevel.DEFAULT_LEVEL;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
|
|
@ -67,6 +67,9 @@ public final class SeverityLevel implements Comparable, Serializable
|
|||
public static final SeverityLevel ERROR =
|
||||
new SeverityLevel(SEVERITYCODE_ERROR, SEVERITYNAME_ERROR);
|
||||
|
||||
/** the default severity */
|
||||
public static final SeverityLevel DEFAULT_LEVEL = ERROR;
|
||||
|
||||
/** map from level names to the respective level */
|
||||
private static final Map NAME_TO_LEVEL = new HashMap();
|
||||
static {
|
||||
|
|
|
|||
Loading…
Reference in New Issue