summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormikaelpeltier <mikaelpeltier@google.com>2015-06-25 16:02:25 +0200
committerMikael Peltier <mikaelpeltier@google.com>2015-06-25 14:38:42 +0000
commit5b4a0d8e4b7f813d6d6ccc52f6479dd663bf91bc (patch)
tree83099d9e80de2bbc1d64deab696896d40b58a46d
parentc07314ab1aa52499086d12007ef5407991a8fa16 (diff)
downloadtoolchain_jack-5b4a0d8e4b7f813d6d6ccc52f6479dd663bf91bc.tar.gz
toolchain_jack-5b4a0d8e4b7f813d6d6ccc52f6479dd663bf91bc.tar.bz2
toolchain_jack-5b4a0d8e4b7f813d6d6ccc52f6479dd663bf91bc.zip
Update version comparison
- versions with 0 in release and sub-release code are not comparables. - versions with engineering kind are not comparables Change-Id: Id8169577ba2c8e4392f2dcfc8a5fb5157c3758d9 (cherry picked from commit dd5c19eefff3487aac2000e4cfdcf4f02e1fe879)
-rw-r--r--sched/src/com/android/sched/util/Version.java14
1 files changed, 6 insertions, 8 deletions
diff --git a/sched/src/com/android/sched/util/Version.java b/sched/src/com/android/sched/util/Version.java
index c1b7746a..3128edd6 100644
--- a/sched/src/com/android/sched/util/Version.java
+++ b/sched/src/com/android/sched/util/Version.java
@@ -202,14 +202,12 @@ public class Version {
}
int compareTo(@Nonnull Version other) throws UncomparableVersion {
- if ((subReleaseKind == SubReleaseKind.ENGINEERING
- && other.getSubReleaseKind() != SubReleaseKind.ENGINEERING)
- || releaseCode < 0
- || subReleaseCode < 0
- || (subReleaseKind != SubReleaseKind.ENGINEERING
- && other.getSubReleaseKind() == SubReleaseKind.ENGINEERING)
- || other.getReleaseCode() < 0
- || other.getSubReleaseCode() < 0) {
+ if (subReleaseKind == SubReleaseKind.ENGINEERING
+ || releaseCode <= 0
+ || subReleaseCode <= 0
+ || other.getSubReleaseKind() == SubReleaseKind.ENGINEERING
+ || other.getReleaseCode() <= 0
+ || other.getSubReleaseCode() <= 0) {
throw new UncomparableVersion(
getVerboseVersion() + " is not comparable with " + other.getVerboseVersion());
}