Added a getAbstractOption() method. I am going to change the design of this
class to improve the following: - Not have protected member variables that are accessed directly. Checkstyle reports this error. - By default the mOption member is not initialised to a default value. This is causing strange behaviour in the EmptyBlockCheck when no option is specified, as it does nothing. It should default to statement.
This commit is contained in:
parent
c9bcfe0be0
commit
903f780d58
|
|
@ -44,4 +44,14 @@ public abstract class AbstractOptionCheck
|
|||
throw new ConversionException("unable to parse " + aOption);
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* @return the <code>AbstractOption</code> set
|
||||
*/
|
||||
public AbstractOption getAbstractOption()
|
||||
{
|
||||
// WARNING!! Do not rename this method to getOption(). It breaks
|
||||
// BeanUtils, which will silently not call setOption. Very annoying!
|
||||
return mOption;
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue