diff options
author | Noel Grandin <noel@peralex.com> | 2014-10-15 14:43:35 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2014-10-16 08:15:48 +0200 |
commit | 973eb2f6db60c0939299a968a3121e3310e6d1f5 (patch) | |
tree | 9eece355c20bc4d930e7e58943fc2d33bedfcfd0 /xmerge | |
parent | fa652cdd2314f485359119a8ff081a7afd1c01b0 (diff) |
java: reduce the depth of some deeply nested if blocks
Change-Id: I3c0c7f08d4d8ea594e72fc0d9b93d085d4ab4bf5
Diffstat (limited to 'xmerge')
-rw-r--r-- | xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java | 32 |
1 files changed, 12 insertions, 20 deletions
diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java index ac7ba521884b..cf9fad8c15ad 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java @@ -151,28 +151,20 @@ public final class Record { */ @Override public boolean equals(Object obj) { - - boolean bool = false; - - if (obj instanceof Record) { - - Record rec = (Record) obj; - - checkLabel: { - - if (rec.getAttributes() != attributes) { - break checkLabel; - } - if (rec.getSize() == data.length) { - for (int i = 0; i < data.length; i++) { - if (data[i] != rec.data[i]) { - break checkLabel; - } - } - bool = true; + if (!(obj instanceof Record)) { + return false; + } + Record rec = (Record) obj; + if (rec.getAttributes() != attributes) { + return false; + } + if (rec.getSize() == data.length) { + for (int i = 0; i < data.length; i++) { + if (data[i] != rec.data[i]) { + return false; } } } - return bool; + return false; } }
\ No newline at end of file |