You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by be...@apache.org on 2008/08/03 13:49:39 UTC

svn commit: r682130 - /maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/versions/DefaultVersionInfo.java

Author: bentmann
Date: Sun Aug  3 04:49:39 2008
New Revision: 682130

URL: http://svn.apache.org/viewvc?rev=682130&view=rev
Log:
o Added explanatory comment for string handling

Modified:
    maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/versions/DefaultVersionInfo.java

Modified: maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/versions/DefaultVersionInfo.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/versions/DefaultVersionInfo.java?rev=682130&r1=682129&r2=682130&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/versions/DefaultVersionInfo.java (original)
+++ maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/versions/DefaultVersionInfo.java Sun Aug  3 04:49:39 2008
@@ -234,6 +234,7 @@
         else
         {
             // TODO: this is a workaround for a bug in DefaultArtifactVersion - fix there - it should not consider case in comparing the qualifier
+            // NOTE: The combination of upper-casing and lower-casing is an approximation of String.equalsIgnoreCase()
             String thisVersion = strVersion.toUpperCase( Locale.ENGLISH ).toLowerCase( Locale.ENGLISH );
             String thatVersion = that.strVersion.toUpperCase( Locale.ENGLISH ).toLowerCase( Locale.ENGLISH );