--- jsr166/build.xml	2019/10/06 17:51:03	1.305
+++ jsr166/build.xml	2021/03/16 20:38:44	1.323
@@ -23,16 +23,22 @@
   $HOME/jdk/jdk12
   $HOME/jdk/jdk13
   $HOME/jdk/jdk14
+  $HOME/jdk/jdk15
+  $HOME/jdk/jdk16
+  $HOME/jdk/jdk17
   where each of the above is a JDK or a symlink to same, and
   $HOME/jdk/src/jdk11
   $HOME/jdk/src/jdk12
   $HOME/jdk/src/jdk13
   $HOME/jdk/src/jdk14
+  $HOME/jdk/src/jdk15
+  $HOME/jdk/src/jdk16
+  $HOME/jdk/src/jdk17
   where each of the above is a JDK source tree or a symlink to same.
 
   Alternatively, define ant variables thus:
   ant -Djdk$N.home=... -Djdk$N.src.home=...
-  for $N in 11 .. 14 ...
+  for $N in 11 .. 17 ...
 
   As of 2016-03, the sources in src/main are for jdk9+ only.
   As of 2019-08, the sources in src/main are for jdk11+ only.
@@ -96,6 +102,9 @@
   <defjdklocations v="12"/>
   <defjdklocations v="13"/>
   <defjdklocations v="14"/>
+  <defjdklocations v="15"/>
+  <defjdklocations v="16"/>
+  <defjdklocations v="17"/>
 
   <!-- Source locations -->
   <property name="src.dir"              location="${basedir}/src/main"/>
@@ -106,6 +115,9 @@
   <property name="jtreg12.src.dir"      location="${test.src.dir}/jtreg"/>
   <property name="jtreg13.src.dir"      location="${test.src.dir}/jtreg"/>
   <property name="jtreg14.src.dir"      location="${test.src.dir}/jtreg"/>
+  <property name="jtreg15.src.dir"      location="${test.src.dir}/jtreg"/>
+  <property name="jtreg16.src.dir"      location="${test.src.dir}/jtreg"/>
+  <property name="jtreg17.src.dir"      location="${test.src.dir}/jtreg"/>
   <property name="jtreg.src.dir"        location="${jtreg11.src.dir}"/>
   <property name="lib.dir"              location="${basedir}/lib"/>
   <property name="dist.dir"             location="${basedir}/dist"/>
@@ -132,7 +144,10 @@
   <property name="java11.docroot.url" value="https://docs.oracle.com/en/java/javase/11/docs"/>
   <property name="java12.docroot.url" value="https://docs.oracle.com/en/java/javase/12/docs"/>
   <property name="java13.docroot.url" value="https://docs.oracle.com/en/java/javase/13/docs"/>
-  <property name="java14.docroot.url" value="https://download.java.net/java/early_access/jdk14/docs"/>
+  <property name="java14.docroot.url" value="https://docs.oracle.com/en/java/javase/14/docs"/>
+  <property name="java15.docroot.url" value="https://docs.oracle.com/en/java/javase/15/docs"/>
+  <property name="java16.docroot.url" value="https://docs.oracle.com/en/java/javase/16/docs"/>
+  <property name="java17.docroot.url" value="https://download.java.net/java/early_access/jdk17/docs"/>
   <!-- Default jdk doc location (latest stable LTS release seems best) -->
   <property name="java.docroot.url"   value="${java11.docroot.url}"/>
 
@@ -141,6 +156,9 @@
   <property name="java12.api.url"     value="${java12.docroot.url}/api/"/>
   <property name="java13.api.url"     value="${java13.docroot.url}/api/"/>
   <property name="java14.api.url"     value="${java14.docroot.url}/api/"/>
+  <property name="java15.api.url"     value="${java15.docroot.url}/api/"/>
+  <property name="java16.api.url"     value="${java16.docroot.url}/api/"/>
+  <property name="java17.api.url"     value="${java17.docroot.url}/api/"/>
   <property name="java.api.url"       value="${java.docroot.url}/api/"/>
 
   <!-- Define the "jtreg" task -->
@@ -242,6 +260,9 @@
     </sequential>
   </macrodef>
 
+  <defjtregtests v="17"/>
+  <defjtregtests v="16"/>
+  <defjtregtests v="15"/>
   <defjtregtests v="14"/>
   <defjtregtests v="13"/>
   <defjtregtests v="12"/>
@@ -264,17 +285,26 @@
     <!-- ant -Djtreg.verbose=time,fail,error jtreg -->
     <property name="jtreg.verbose" value="nopass,fail,error"/>
 
-    <delete dir="@{workdir}/JTwork"   quiet="true"/>
+    <!-- concurrency > 1 mitigates slowness of some jtreg tests -->
+    <!-- BUT concurrency:auto starts "too many" VMs; just 1 free core each -->
+    <!-- In 2020 concurrency:3 seems like a sweet spot -->
+    <!-- Override using:  ant -Djtreg.concurrency="N" jtreg -->
+    <property name="jtreg.concurrency" value="3"/>
+
+    <delete dir="@{workdir}/JTwork" quiet="true"/>
     <jtreg dir="${jtreg@{target}.src.dir}"
            jdk="${jdk@{target}.home}"
            workDir="@{workdir}/JTwork">
       <patternset refid="jdk@{target}.jtreg.tests"/>
-      <arg value="-javacoptions:--patch-module=java.base=@{classes}"/>
-      <arg value="-vmoptions:--patch-module=java.base=@{classes}"/>
+      <arg value="-javacoption:--patch-module=java.base=@{classes}"/>
+      <arg value="-vmoption:--patch-module=java.base=@{classes}"/>
+      <arg value="-vmoption:-Xmx256m"/>
+      <arg value="-concurrency:${jtreg.concurrency}"/>
+      <arg value="-vmoptions:-esa -ea"/>
+      <arg value="-vmoptions:${vmoptions}" if:set="vmoptions"/>
       <arg value="-agentvm"/>
       <arg value="-noreport"/>
       <arg value="-verbose:@{verbose}"/>
-      <arg value="-vmoptions:-esa -ea"/>
       <arg value="-automatic"/>
       <arg value="-k:!ignore"/>
       <arg line="@{jtregflags}"/>
@@ -344,7 +374,11 @@
 <!--   Xlint:-removal for jdk12 Unsafe Object -> Reference renaming -->
       <compilerarg value="-Xlint:all,-removal"/>
       <compilerarg line="--doclint-format html5"/>
-      <compilerarg value="-Xdoclint:all/protected,reference/private"/>
+<!--  Temporarily disable doclint for private elements, due to -->
+<!--  https://bugs.openjdk.java.net/browse/JDK-8214571 -->
+<!--  which is expected to be fixed in openjdk 11.0.8 -->
+<!--  <compilerarg value="-Xdoclint:all/protected,reference/private"/> -->
+      <compilerarg value="-Xdoclint:all/protected"/>
       <compilerarg value="-Xdoclint/package:java.util.*"/>
       <compilerarg value="-Werror"/>
       <compilerarg line="-Xmaxerrs 1000 -Xmaxwarns 1000"/>
@@ -372,7 +406,7 @@
   <target name="docs"
           description="Builds javadocs for src/main to dist dir">
 
-    <delete dir="${docs.dir}"/>
+    <delete dir="${docs.dir}" quiet="true"/>
     <mkdir dir="${docs.dir}"/>
 
     <!-- old school frames via docs/index.html?overview-summary.html -->
@@ -401,12 +435,14 @@
              executable="${build.main.javadoc}">
 <!-- TODO: JDK-8214571 failonerror = "true" -->
       <fileset dir="${src.dir}" defaultexcludes="yes">
-        <include name="**/*.java"/>
+        <include name="java/**/*.java"/>
       </fileset>
       <arg line="-Xdocrootparent ${java.docroot.url}"/>
       <arg line="-Xmaxerrs 1000 -Xmaxwarns 1000"/>
+      <arg value="-quiet"/>
       <arg value="-XDignore.symbol.file=true"/>
       <arg value="-html5"/>
+      <arg value="-Xdoclint:all,-missing"/>
       <arg value="--patch-module=java.base=${src.dir}"/>
       <arg value="--frames" if:set="use-frames"/>
       <arg value="--override-methods=summary"/>
@@ -498,93 +534,154 @@
           description="Runs tck and jtreg tests for src/main"/>
 
   <target name="test-version-permutations"
-          depends="test11, test12, test13, test14, test11-12, test11-13, test11-14, clean, test, docs">
-  </target>
+          depends="test11, test12, test13, test14, test15, test16,
+          test11-12, test11-13, test11-14, test11-15, test11-16,
+          clean, test, docs"
+          description="Runs tck and jtreg tests for various build-time and runtime java versions"/>
 
   <target name="test11">
-    <antcall target="clean"/>
     <antcall>
       <param name="java.runtime.target" value="11"/>
       <param name="build.main.javac" value="${javac11}"/>
       <param name="build.main.javadoc" value="${javadoc12}"/>
       <param name="build.main.javadoc.source" value="11"/>
+      <target name="clean"/>
       <target name="test"/>
       <target name="docs"/>
     </antcall>
   </target>
 
   <target name="test12">
-    <antcall target="clean"/>
     <antcall>
       <param name="java.runtime.target" value="12"/>
       <param name="build.main.javac" value="${javac12}"/>
       <param name="build.main.javadoc" value="${javadoc12}"/>
       <param name="build.main.javadoc.source" value="12"/>
+      <target name="clean"/>
       <target name="test"/>
       <target name="docs"/>
     </antcall>
   </target>
 
   <target name="test13">
-    <antcall target="clean"/>
     <antcall>
       <param name="java.runtime.target" value="13"/>
       <param name="build.main.javac" value="${javac13}"/>
       <param name="build.main.javadoc" value="${javadoc13}"/>
       <param name="build.main.javadoc.source" value="13"/>
+      <target name="clean"/>
       <target name="test"/>
       <target name="docs"/>
     </antcall>
   </target>
 
   <target name="test14">
-    <antcall target="clean"/>
     <antcall>
       <param name="java.runtime.target" value="14"/>
       <param name="build.main.javac" value="${javac14}"/>
       <param name="build.main.javadoc" value="${javadoc14}"/>
       <param name="build.main.javadoc.source" value="14"/>
+      <target name="clean"/>
+      <target name="test"/>
+      <target name="docs"/>
+    </antcall>
+  </target>
+
+  <target name="test15">
+    <antcall>
+      <param name="java.runtime.target" value="15"/>
+      <param name="build.main.javac" value="${javac15}"/>
+      <param name="build.main.javadoc" value="${javadoc15}"/>
+      <param name="build.main.javadoc.source" value="15"/>
+      <target name="clean"/>
+      <target name="test"/>
+      <target name="docs"/>
+    </antcall>
+  </target>
+
+  <target name="test16">
+    <antcall>
+      <param name="java.runtime.target" value="16"/>
+      <param name="build.main.javac" value="${javac16}"/>
+      <param name="build.main.javadoc" value="${javadoc16}"/>
+      <param name="build.main.javadoc.source" value="16"/>
+      <target name="clean"/>
       <target name="test"/>
       <target name="docs"/>
     </antcall>
   </target>
 
   <target name="test11-12">
-    <antcall target="clean"/>
     <antcall>
       <param name="java.runtime.target" value="12"/>
       <param name="build.main.javac" value="${javac11}"/>
       <param name="build.main.javadoc" value="${javadoc12}"/>
       <param name="build.main.javadoc.source" value="11"/>
+      <target name="clean"/>
       <target name="test"/>
       <target name="docs"/>
     </antcall>
   </target>
 
   <target name="test11-13">
-    <antcall target="clean"/>
     <antcall>
       <param name="java.runtime.target" value="13"/>
       <param name="build.main.javac" value="${javac11}"/>
       <param name="build.main.javadoc" value="${javadoc13}"/>
       <param name="build.main.javadoc.source" value="11"/>
+      <target name="clean"/>
       <target name="test"/>
       <target name="docs"/>
     </antcall>
   </target>
 
   <target name="test11-14">
-    <antcall target="clean"/>
     <antcall>
       <param name="java.runtime.target" value="14"/>
       <param name="build.main.javac" value="${javac11}"/>
       <param name="build.main.javadoc" value="${javadoc14}"/>
       <param name="build.main.javadoc.source" value="11"/>
+      <target name="clean"/>
       <target name="test"/>
       <target name="docs"/>
     </antcall>
   </target>
 
+  <target name="test11-15">
+    <antcall>
+      <param name="java.runtime.target" value="15"/>
+      <param name="build.main.javac" value="${javac11}"/>
+      <param name="build.main.javadoc" value="${javadoc15}"/>
+      <param name="build.main.javadoc.source" value="11"/>
+      <target name="clean"/>
+      <target name="test"/>
+      <target name="docs"/>
+    </antcall>
+  </target>
+
+  <target name="test11-16">
+    <antcall>
+      <param name="java.runtime.target" value="16"/>
+      <param name="build.main.javac" value="${javac11}"/>
+      <param name="build.main.javadoc" value="${javadoc16}"/>
+      <param name="build.main.javadoc.source" value="11"/>
+      <target name="clean"/>
+      <target name="test"/>
+      <target name="docs"/>
+    </antcall>
+  </target>
+
+  <target name="docs16">
+    <antcall>
+      <param name="java.runtime.target" value="16"/>
+      <param name="build.main.javac" value="${javac16}"/>
+      <param name="build.main.javadoc" value="${javadoc16}"/>
+      <param name="build.main.javadoc.source" value="16"/>
+      <target name="clean"/>
+      <target name="docs"/>
+    </antcall>
+  </target>
+
 
   <target name="configure-compiler">
     <fail message="ant version too old">
@@ -662,15 +759,17 @@
   https://github.com/google/error-prone/issues/1143
 =================================================================== -->
   <path id="errorprone.processorpath.path">
-    <pathelement location="${lib.dir}/error_prone_core-2.3.2-with-dependencies.jar"/>
+    <pathelement location="${lib.dir}/error_prone_core-2.3.4-with-dependencies.jar"/>
     <pathelement location="${lib.dir}/dataflow-2.5.7.jar"/>
     <pathelement location="${lib.dir}/javacutil-2.5.7.jar"/>
     <pathelement location="${lib.dir}/jFormatString-3.0.0.jar"/>
+    <pathelement location="${lib.dir}/caffeine-2.8.0.jar"/>
   </path>
   <property name="errorprone.processorpath" refid="errorprone.processorpath.path" />
   <property name="errorprone.jsr166.user.flags" value=""/>
   <property name="errorprone.jsr166.flags"
             value="-Xep:HashtableContains:OFF
+                   -Xep:PreferJavaTimeOverload:OFF
                    -Xep:JdkObsolete:OFF
                    -Xep:MissingOverride:OFF
                    -Xep:MissingFail:OFF
@@ -690,6 +789,7 @@
   <property name="errorprone.jsr166.test.flags"
             value="-Xep:StringSplitter:OFF
                    -Xep:BoxedPrimitiveConstructor:OFF
+                   -Xep:UnnecessaryParentheses:OFF
                    -Xep:ModifyingCollectionWithItself:OFF"/>
   <!-- ant -emacs errorprone |& grep -EA1 '(warning|error):|warnings' -->
   <target name="errorprone"