diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2015-10-22 13:52:01 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2015-10-22 13:52:20 +0200 |
| commit | d622b72e1d90670e4c8df8a512c8d161e85c2e3f (patch) | |
| tree | 9fee20d173408b8c1dc746cf6e35d7ed6d2c1e76 /qadevOOo | |
| parent | 173fc95b3551c1e69c49626211be8422cb29fb3e (diff) | |
No need to check for non-null here
...as null values are handled just fine by compare()
Change-Id: Ifd17b96187ad3d13be99b107d3c3fa47e51b586e
Diffstat (limited to 'qadevOOo')
| -rw-r--r-- | qadevOOo/runner/lib/MultiPropertyTest.java | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/qadevOOo/runner/lib/MultiPropertyTest.java b/qadevOOo/runner/lib/MultiPropertyTest.java index 38bf2ccfb17d..53cbf0976e6f 100644 --- a/qadevOOo/runner/lib/MultiPropertyTest.java +++ b/qadevOOo/runner/lib/MultiPropertyTest.java @@ -406,17 +406,10 @@ public class MultiPropertyTest extends MultiMethodTest log.println("\t The type seems to be an Any with value of NULL."); log.println("\t Maybe the property should get its own test method."); } - if (resValue != null) + if (!compare(resValue, oldValue)) { - if (!compare(resValue, oldValue)) - { - log.println("But it has changed."); - tRes.tested(propName, true); - } - else - { - tRes.tested(propName, false); - } + log.println("But it has changed."); + tRes.tested(propName, true); } else { |