JDK 15 RFR of 8237450: JDK13 annotation processors not run when a supported annotation type specifies a module

Jonathan Gibbons jonathan.gibbons at oracle.com
Wed Feb 19 00:36:58 UTC 2020


Looks good to me.

-- Jon


On 02/15/2020 01:42 PM, Joe Darcy wrote:
> Hello,
>
> Please review the small fix for
>
>     8237450: JDK13 annotation processors not run when a supported 
> annotation type specifies a module
>     http://cr.openjdk.java.net/~darcy/8237450.0/
>
> Patch below. The modified test fails without the fix and passes with it.
>
> Thanks,
>
> -Joe
>
> --- 
> old/src/jdk.compiler/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java 
> 2020-02-15 13:09:24.235878000 -0800
> +++ 
> new/src/jdk.compiler/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java 
> 2020-02-15 13:09:23.999995999 -0800
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright (c) 2005, 2019, Oracle and/or its affiliates. All rights 
> reserved.
> + * Copyright (c) 2005, 2020, 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
> @@ -1758,7 +1758,7 @@
>              pkg = s;
>          } else {
>              String moduleName = s.substring(0, slash);
> -            if (!SourceVersion.isIdentifier(moduleName)) {
> +            if (!SourceVersion.isName(moduleName)) {
>                  return warnAndNoMatches(s, p, log, lint);
>              }
>              module = Pattern.quote(moduleName + "/");
> --- 
> old/test/langtools/tools/javac/diags/examples/RedundantTypesWithWildcardProc/processors/AnnoProc.java 
> 2020-02-15 13:09:24.715637999 -0800
> +++ 
> new/test/langtools/tools/javac/diags/examples/RedundantTypesWithWildcardProc/processors/AnnoProc.java 
> 2020-02-15 13:09:24.475757999 -0800
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright (c) 2010, 2019, Oracle and/or its affiliates. All rights 
> reserved.
> + * Copyright (c) 2010, 2020, 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
> @@ -26,7 +26,7 @@
>  import javax.lang.model.*;
>  import javax.lang.model.element.*;
>
> - at SupportedAnnotationTypes({"java.lang.SuppressWarnings", "*"})
> + at SupportedAnnotationTypes({"java.base/java.lang.SuppressWarnings", "*"})
>  public class AnnoProc extends AbstractProcessor {
>      public boolean process(Set<? extends TypeElement> elems, 
> RoundEnvironment renv) {
>          return true;
>



More information about the compiler-dev mailing list