Merge branch 'release/1.3.5'

Author: G00fY2

README.md

@@ -12,15 +12,15 @@ Pure Java (java.util), no dependencies, very small method count.
 **Gradle:**
 ```
 dependencies {
-  implementation 'com.g00fy2:versioncompare:1.3.4'
+  implementation 'com.g00fy2:versioncompare:1.3.5'
 }
 ```
 **Maven:**
 ```
 <dependency>
   <groupId>com.g00fy2</groupId>
   <artifactId>versioncompare</artifactId>
-  <version>1.3.4</version>
+  <version>1.3.5</version>
 </dependency>
 ```
 
@@ -89,7 +89,7 @@ suffix compare logic                          ||
 ## Sample App
 ![Image](https://raw.githubusercontent.com/G00fY2/version-compare/gh-pages/images/version_compare_sampleapp_framed.png)
 
-**Try out the sample app to compare your version inputs: [Download APK](https://github.com/G00fY2/version-compare/releases/download/1.3.4/version-compare-1.3.4-sample.apk)**
+**Try out the sample app to compare your version inputs: [Download APK](https://github.com/G00fY2/version-compare/releases/download/1.3.5/version-compare-1.3.5-sample.apk)**
 
 ## License
 	Copyright (C) 2018 Thomas Wirth

build.gradle

@@ -4,14 +4,15 @@ buildscript {
         jcenter()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:3.5.3'
-        classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.8.4'
+        classpath 'com.android.tools.build:gradle:4.0.2'
+        classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.8.5'
     }
 }
 
 allprojects {
     repositories {
         google()
+        mavenCentral()
         jcenter()
     }
 }

gradle/wrapper/gradle-wrapper.jar

@@ -1,5 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-5.6.4-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-6.6.1-all.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

gradle/wrapper/gradle-wrapper.properties

@@ -3,11 +3,11 @@ apply plugin: 'kotlin-android'
 apply plugin: 'kotlin-android-extensions'
 
 android {
-    compileSdkVersion 29
+    compileSdkVersion 30
     defaultConfig {
         applicationId 'com.g00fy2.versioncomparesample'
         minSdkVersion 14
-        targetSdkVersion 29
+        targetSdkVersion 30
         versionCode 1
         versionName '1.0'
     }
@@ -30,7 +30,7 @@ dependencies {
 }
 
 buildscript {
-    ext.kotlin_version = '1.3.61'
+    ext.kotlin_version = '1.4.10'
     repositories {
         mavenCentral()
     }

gradlew

@@ -4,7 +4,8 @@ apply plugin: 'jacoco'
 dependencies {
     compileOnly 'com.google.code.findbugs:jsr305:3.0.2' // only required at compile time
 
-    testImplementation 'junit:junit:4.12'
+    testImplementation 'junit:junit:4.13'
+    testImplementation 'nl.jqno.equalsverifier:equalsverifier:3.4.3'
 }
 
 jacoco {

gradlew.bat

@@ -11,7 +11,7 @@ ext {
     issueUrl = 'https://github.com/G00fY2/version-compare/issues'
     gitUrl = 'https://github.com/G00fY2/version-compare.git'
 
-    libraryVersion = '1.3.4'
+    libraryVersion = '1.3.5'
 
     developerId = 'g00fy2'
     developerName = 'Thomas Wirth'

sample/build.gradle

@@ -10,6 +10,7 @@ public class Version implements Comparable<Version> {
 
   @Nullable private final String originalString;
   @Nonnull private final List<Integer> subversionNumbers = new ArrayList<>();
+  @Nonnull private final List<Integer> subversionNumbersWithoutTrailingZeros = new ArrayList<>();
   @Nonnull private String suffix = "";
 
   /**
@@ -122,8 +123,9 @@ public boolean isHigherThan(String otherVersion) {
    * @see #isHigherThan(String otherVersion)
    */
   public boolean isHigherThan(Version otherVersion) {
-    int subversionsResult =
-        VersionComparator.compareSubversionNumbers(subversionNumbers, otherVersion.subversionNumbers);
+    int subversionsResult = VersionComparator.compareSubversionNumbers(
+        subversionNumbersWithoutTrailingZeros,
+        otherVersion.subversionNumbersWithoutTrailingZeros);
     if (subversionsResult != 0) {
       return subversionsResult > 0;
     }
@@ -151,8 +153,10 @@ public boolean isLowerThan(String otherVersion) {
    * @see #isLowerThan(String otherVersion)
    */
   public boolean isLowerThan(Version otherVersion) {
-    int subversionsResult =
-        VersionComparator.compareSubversionNumbers(subversionNumbers, otherVersion.subversionNumbers);
+    int subversionsResult = VersionComparator.compareSubversionNumbers(
+        subversionNumbersWithoutTrailingZeros,
+        otherVersion.subversionNumbersWithoutTrailingZeros
+    );
     if (subversionsResult != 0) {
       return subversionsResult < 0;
     }
@@ -180,8 +184,10 @@ public boolean isEqual(String otherVersion) {
    * @see #isEqual(String otherVersion)
    */
   public boolean isEqual(Version otherVersion) {
-    return VersionComparator.compareSubversionNumbers(subversionNumbers, otherVersion.subversionNumbers) == 0
-        && VersionComparator.compareSuffix(suffix, otherVersion.suffix) == 0;
+    return VersionComparator.compareSubversionNumbers(
+        subversionNumbersWithoutTrailingZeros,
+        otherVersion.subversionNumbersWithoutTrailingZeros
+    ) == 0 && VersionComparator.compareSuffix(suffix, otherVersion.suffix) == 0;
   }
 
   /**
@@ -231,8 +237,10 @@ public boolean isAtLeast(String otherVersion, boolean ignoreSuffix) {
    * @see #isAtLeast(String otherVersion, boolean ignoreSuffix)
    */
   public boolean isAtLeast(Version otherVersion, boolean ignoreSuffix) {
-    int subversionsResult =
-        VersionComparator.compareSubversionNumbers(subversionNumbers, otherVersion.subversionNumbers);
+    int subversionsResult = VersionComparator.compareSubversionNumbers(
+        subversionNumbersWithoutTrailingZeros,
+        otherVersion.subversionNumbersWithoutTrailingZeros
+    );
     if (subversionsResult == 0 && !ignoreSuffix) {
       return VersionComparator.compareSuffix(suffix, otherVersion.suffix) >= 0;
     }
@@ -267,20 +275,38 @@ private void initVersion() {
           }
         }
       }
+      subversionNumbersWithoutTrailingZeros.addAll(subversionNumbers);
+      while (subversionNumbersWithoutTrailingZeros.lastIndexOf(0) >= 0) {
+        subversionNumbersWithoutTrailingZeros.remove(subversionNumbersWithoutTrailingZeros.lastIndexOf(0));
+      }
       if (suffixSb != null) suffix = suffixSb.toString();
     }
   }
 
   @Override
-  public int compareTo(@Nonnull Version version) {
-      if (this.isEqual(version)) return 0;
-      else if (this.isLowerThan(version)) return -1;
-      else return 1;
+  final public int compareTo(@Nonnull Version version) {
+    if (this.isEqual(version)) return 0;
+    else if (this.isLowerThan(version)) return -1;
+    else return 1;
   }
 
   @Override
-  public boolean equals(Object o) {
-      if (o instanceof Version && this.isEqual((Version)o)) return true;
-      else return super.equals(o);
+  final public boolean equals(Object o) {
+    if (o instanceof Version && this.isEqual((Version) o)) return true;
+    else return super.equals(o);
+  }
+
+  @Override
+  final public int hashCode() {
+    final int prime = 31;
+    int hash = 1;
+    hash = prime * hash + subversionNumbersWithoutTrailingZeros.hashCode();
+    if (suffix.isEmpty()) return hash;
+
+    int releaseQualifier = VersionComparator.qualifierToNumber(suffix);
+    int releaseQualifierVersion = VersionComparator.preReleaseVersion(suffix, releaseQualifier);
+    hash = prime * hash + releaseQualifier;
+    hash = prime * hash + releaseQualifierVersion;
+    return hash;
   }
 }