summaryrefslogtreecommitdiff
path: root/qadevOOo
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2015-10-22 13:52:01 +0200
committerStephan Bergmann <sbergman@redhat.com>2015-10-22 13:52:20 +0200
commitd622b72e1d90670e4c8df8a512c8d161e85c2e3f (patch)
tree9fee20d173408b8c1dc746cf6e35d7ed6d2c1e76 /qadevOOo
parent173fc95b3551c1e69c49626211be8422cb29fb3e (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.java13
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
{