RFR: 8284220: TypeMirror#toString omits enclosing class names after JDK-8281238 [v5]

Liam Miller-Cushon cushon at openjdk.java.net
Tue Apr 26 17:11:42 UTC 2022


> Please consider this fix for https://bugs.openjdk.java.net/browse/JDK-8284220, which updates `ClassType#toString` to include enclosing class names when type annotations are present.
> 
> For example, `java.util. at A Map.Entry<?,?>` is now printed as `java.util. at A Map.Entry<?,?>`, instead of `java.util. at A Entry<?,?>` (note the missing `Map`).

Liam Miller-Cushon has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:

 - Merge branch 'master' into JDK-8284220
 - Add more documentation to test
 - Merge branch 'openjdk:master' into JDK-8284220
 - Refactor test based on review feedback
 - 8284220: TypeMirror#toString omits enclosing class names after JDK-8281238

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8084/files
  - new: https://git.openjdk.java.net/jdk/pull/8084/files/cc0531fb..67103c77

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8084&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8084&range=03-04

  Stats: 19355 lines in 1068 files changed: 12067 ins; 2824 del; 4464 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8084.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8084/head:pull/8084

PR: https://git.openjdk.java.net/jdk/pull/8084


More information about the compiler-dev mailing list