8134365: Test test/sun/misc/Version/Version.java should follow Verona rules for trailing zeros
authoramurillo
Wed, 16 Sep 2015 10:52:10 -0700
changeset 34002 5b7b63906ad4
parent 34001 40bb086ea060
child 34003 3c4ba277fdb6
8134365: Test test/sun/misc/Version/Version.java should follow Verona rules for trailing zeros Reviewed-by: mchung, iris
jdk/test/sun/misc/Version/Version.java
--- a/jdk/test/sun/misc/Version/Version.java	Mon Sep 14 10:00:25 2015 -0700
+++ b/jdk/test/sun/misc/Version/Version.java	Wed Sep 16 10:52:10 2015 -0700
@@ -22,9 +22,10 @@
  */
 
 /* @test
- * @bug 6994413
+ * @bug 6994413, 8134365
  * @summary Check the JDK and JVM version returned by sun.misc.Version
- *          matches the versions defined in the system properties
+ *          matches the versions defined in the system properties.
+ *          Should use the API described in JDK-8136651 when available
  * @modules java.base/sun.misc
  * @compile -XDignore.symbol.file Version.java
  * @run main Version
@@ -90,12 +91,21 @@
 
         public String toString() {
             StringBuilder sb = new StringBuilder();
-            sb.append(major + "." + minor + "." + security);
+            // Do not include trailing zeros
             if (patch > 0) {
-                sb.append("." + patch);
+                sb.insert(0, "." + patch);
+            }
+            if (security > 0 || sb.length() > 0) {
+                sb.insert(0, "." + security);
             }
+            if (minor > 0 || sb.length() > 0) {
+                sb.insert(0, "." + minor);
+            }
+            sb.insert(0, major);
 
-            sb.append("+" + build);
+            if (build >= 0)
+                sb.append("+" + build);
+
             return sb.toString();
         }
     }