JDK 10 RFR of JDK-8193191: Update JavacTestingAbstractProcessor for JDK 10
Jonathan Gibbons
jonathan.gibbons at oracle.com
Thu Dec 7 17:29:14 UTC 2017
+1
-- Jon
On 12/7/17 9:12 AM, joe darcy wrote:
> Hello,
>
> Please review the straightforward patch below to update the
> JavacTestingAbstractProcessor utility superclass to JDK 10.
>
> JDK-8193191: Update JavacTestingAbstractProcessor for JDK 10
>
> Besides the use of Set.of, most of the updates are just bumping
> @SupportedSourceVersion annotation from 9 to 10.
>
> Thanks,
>
> -Joe
>
> diff -r 56d3576aadd5
> test/langtools/tools/javac/lib/JavacTestingAbstractProcessor.java
> ---
> a/test/langtools/tools/javac/lib/JavacTestingAbstractProcessor.java
> Mon Dec 04 19:12:57 2017 +0100
> +++
> b/test/langtools/tools/javac/lib/JavacTestingAbstractProcessor.java
> Thu Dec 07 09:04:27 2017 -0800
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2010, 2015, Oracle and/or its affiliates. All rights
> reserved.
> + * Copyright (c) 2010, 2017, Oracle and/or its affiliates. All rights
> reserved.
> * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
> *
> * This code is free software; you can redistribute it and/or modify it
> @@ -31,13 +31,7 @@
> * An abstract annotation processor tailored to {@code javac}
> regression testing.
> */
> public abstract class JavacTestingAbstractProcessor extends
> AbstractProcessor {
> - private static final Set<String> allAnnotations;
> -
> - static {
> - Set<String> tmp = new HashSet<>();
> - tmp.add("*");
> - allAnnotations = Collections.unmodifiableSet(tmp);
> - }
> + private static final Set<String> allAnnotations = Set.of("*");
>
> protected Elements eltUtils;
> protected Elements elements;
> @@ -116,7 +110,7 @@
> * corresponding platform visitor type.
> */
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static abstract class AbstractAnnotationValueVisitor<R, P>
> extends AbstractAnnotationValueVisitor9<R, P> {
>
> /**
> @@ -127,7 +121,7 @@
> }
> }
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static abstract class AbstractElementVisitor<R, P> extends
> AbstractElementVisitor9<R, P> {
> /**
> * Constructor for concrete subclasses to call.
> @@ -137,7 +131,7 @@
> }
> }
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static abstract class AbstractTypeVisitor<R, P> extends
> AbstractTypeVisitor9<R, P> {
> /**
> * Constructor for concrete subclasses to call.
> @@ -147,7 +141,7 @@
> }
> }
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static class ElementKindVisitor<R, P> extends
> ElementKindVisitor9<R, P> {
> /**
> * Constructor for concrete subclasses; uses {@code null} for
> the
> @@ -168,7 +162,7 @@
> }
> }
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static class ElementScanner<R, P> extends
> ElementScanner9<R, P> {
> /**
> * Constructor for concrete subclasses; uses {@code null} for
> the
> @@ -187,7 +181,7 @@
> }
> }
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static class SimpleAnnotationValueVisitor<R, P> extends
> SimpleAnnotationValueVisitor9<R, P> {
> /**
> * Constructor for concrete subclasses; uses {@code null} for
> the
> @@ -208,7 +202,7 @@
> }
> }
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static class SimpleElementVisitor<R, P> extends
> SimpleElementVisitor9<R, P> {
> /**
> * Constructor for concrete subclasses; uses {@code null} for
> the
> @@ -229,7 +223,7 @@
> }
> }
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static class SimpleTypeVisitor<R, P> extends
> SimpleTypeVisitor9<R, P> {
> /**
> * Constructor for concrete subclasses; uses {@code null} for
> the
> @@ -250,7 +244,7 @@
> }
> }
>
> - @SupportedSourceVersion(RELEASE_9)
> + @SupportedSourceVersion(RELEASE_10)
> public static class TypeKindVisitor<R, P> extends
> TypeKindVisitor9<R, P> {
> /**
> * Constructor for concrete subclasses to call; uses {@code
> null}
>
More information about the compiler-dev
mailing list