Re: RFR: 8244146: javac changes for JEP 306

2021-05-05 Thread Srikanth Adayapalam
On Sat, 1 May 2021 23:00:05 GMT, Joe Darcy wrote: > 8244146: javac changes for JEP 306 Overall, looks good other than the various minor issues called out. I wonder if the tests would have turned out to be a good bit simpler if we simply checked diagnostics against a golden file and skipped the

Re: RFR: 8244146: javac changes for JEP 306

2021-05-04 Thread Srikanth Adayapalam
On Sat, 1 May 2021 23:00:05 GMT, Joe Darcy wrote: > 8244146: javac changes for JEP 306 src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Check.java line 1326: > 1324: private void warnOnExplicitStrictfp(DiagnosticPosition pos, JCTree > tree) { > 1325: DiagnosticPosition prevL

Re: RFR: 8244146: javac changes for JEP 306

2021-05-04 Thread Srikanth Adayapalam
On Sat, 1 May 2021 23:00:05 GMT, Joe Darcy wrote: > 8244146: javac changes for JEP 306 src/jdk.compiler/share/classes/com/sun/tools/javac/code/Source.java line 228: > 226: SEALED_CLASSES(JDK17, Fragments.FeatureSealedClasses, > DiagKind.PLURAL), > 227: // todo: will need to sup

Re: RFR: 8244146: javac changes for JEP 306

2021-05-04 Thread Srikanth Adayapalam
On Sat, 1 May 2021 23:00:05 GMT, Joe Darcy wrote: > 8244146: javac changes for JEP 306 src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties line 1769: > 1767: compiler.warn.strictfp=\ > 1768: as of release 17, all floating-point expressions are evaluated > stri

Re: RFR: 8244146: javac changes for JEP 306

2021-05-04 Thread Srikanth Adayapalam
On Sat, 1 May 2021 23:00:05 GMT, Joe Darcy wrote: > 8244146: javac changes for JEP 306 src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassWriter.java line 1704: > 1702: if (Feature.REDUNDANT_STRICTFP.allowedInSource(source)) > 1703: result = result & ~STRICTFP; > 1