diff options
| author | Noel Grandin <noel@peralex.com> | 2014-08-13 10:19:51 +0200 |
|---|---|---|
| committer | Noel Grandin <noel@peralex.com> | 2014-08-20 10:35:53 +0200 |
| commit | 60f152caeee38579433a31dd3a98e91dc3d23ff6 (patch) | |
| tree | 15ba69f938e09bb173ca7de2e1b21b62e740ea40 /forms | |
| parent | 2922a967a1da5f9c0a07b5390906307d0ae6fe48 (diff) | |
java: avoid unnecessary comparisons in boolean expressions
i.e. stuff like "x == true"
Change-Id: Ib82a4a30e736df392405332fa197b588482cffcf
Diffstat (limited to 'forms')
| -rw-r--r-- | forms/qa/integration/forms/BooleanValidator.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/forms/qa/integration/forms/BooleanValidator.java b/forms/qa/integration/forms/BooleanValidator.java index dfd3fcaa5eb8..730c45f65eb9 100644 --- a/forms/qa/integration/forms/BooleanValidator.java +++ b/forms/qa/integration/forms/BooleanValidator.java @@ -43,7 +43,7 @@ public class BooleanValidator extends integration.forms.ControlValidator if ( AnyConverter.isVoid( Value ) ) return "'indetermined' is not an allowed state"; boolean value = ((Boolean)Value).booleanValue(); - if ( m_preventChecked && ( value == true ) ) + if ( m_preventChecked && ( value ) ) return "no no no. Don't check it."; } catch( java.lang.Exception e ) @@ -61,7 +61,7 @@ public class BooleanValidator extends integration.forms.ControlValidator return false; boolean value = ((Boolean)Value).booleanValue(); - if ( m_preventChecked && ( value == true ) ) + if ( m_preventChecked && ( value ) ) return false; return true; } |