diff options
author | Caolán McNamara <caolanm@redhat.com> | 2015-12-21 09:07:05 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2015-12-21 10:46:57 +0000 |
commit | 91fde32383c94e8840e4e8fab8f47be83d23d20c (patch) | |
tree | f972d3af7371445f4a9482bcef4328c503a0da22 /qadevOOo/tests/java/ifc/chart | |
parent | 72f9ee58e88fbd94252faa3e1e6a4f7fd7517f49 (diff) |
coverity#1326924 NS: Suspicious use of non-short-circuit boolean operator
Change-Id: I2450cdd6e74e1ebab13c61f3a88668df412f6a61
Diffstat (limited to 'qadevOOo/tests/java/ifc/chart')
-rw-r--r-- | qadevOOo/tests/java/ifc/chart/_XChartDataArray.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/qadevOOo/tests/java/ifc/chart/_XChartDataArray.java b/qadevOOo/tests/java/ifc/chart/_XChartDataArray.java index 06296ef1a364..5021657ea99e 100644 --- a/qadevOOo/tests/java/ifc/chart/_XChartDataArray.java +++ b/qadevOOo/tests/java/ifc/chart/_XChartDataArray.java @@ -166,7 +166,9 @@ public class _XChartDataArray extends MultiMethodTest { if (bResult) { for (int i = 0; i < dscs.length; i++) { log.println("Col " + i + ": got " + dscs[i] + " expected: " + colDscs[i]); - bResult &= dscs[i].equals(colDscs[i]); + if (!dscs[i].equals(colDscs[i])) { + bResult = false; + } } } |