Zing version check for compiler hints compatibility is incorrect (for releases > 5)

Nitsan Wakart nitsanw at yahoo.com
Wed Jul 22 14:57:58 UTC 2015


My bad.
Please correct with following change:
--- a/jmh-core/src/main/java/org/openjdk/jmh/runner/CompilerHints.javaTue Jul 14 00:28:56 2015 +0300
+++ b/jmh-core/src/main/java/org/openjdk/jmh/runner/CompilerHints.javaWed Jul 22 16:54:29 2015 +0200
@@ -107,7 +107,10 @@
             try {
                 // get the version digits
                 String[] versionDigits = version.substring(version.indexOf('_') + 1).split("\\.");
-                if (Integer.valueOf(versionDigits[0]) >= 5 && Integer.valueOf(versionDigits[1]) >= 10) {
+                if (Integer.valueOf(versionDigits[0]) > 5) {
+                    return true;
+                }
+                else if (Integer.valueOf(versionDigits[0]) == 5 && Integer.valueOf(versionDigits[1]) >= 10) {
                     return true;
                 }
             } catch (NumberFormatException e) {


More information about the jmh-dev mailing list