RFR: 8325369: @sealedGraph: Bad link to image for tag on nested classes

Jonathan Gibbons jjg at openjdk.org
Wed Jul 10 21:50:56 UTC 2024


On Wed, 10 Jul 2024 18:57:22 GMT, Chen Liang <liach at openjdk.org> wrote:

> Fixes the link to the generated sealed graph svg generated by the sealed graph taglet.
> 
> Affected interfaces includes [`ClassFile.Option`](https://cr.openjdk.org/~nbenalla/GeneratedDocs/8325369-nested-sealed-svg/api/java.base/java/lang/classfile/ClassFile.Option.html) ([before](https://download.java.net/java/early_access/jdk23/docs/api/java.base/java/lang/classfile/ClassFile.Option.html)), [`AnnotationValue.OfConstant`](https://cr.openjdk.org/~nbenalla/GeneratedDocs/8325369-nested-sealed-svg/api/java.base/java/lang/classfile/AnnotationValue.OfConstant.html) ([before](https://download.java.net/java/early_access/jdk23/docs/api/java.base/java/lang/classfile/AnnotationValue.OfConstant.html)), a few [nested interfaces](https://cr.openjdk.org/~nbenalla/GeneratedDocs/8325369-nested-sealed-svg/api/java.base/java/lang/classfile/Signature.html#nested-class-summary) in `Signature` ([original](https://download.java.net/java/early_access/jdk23/docs/api/java.base/java/lang/classfile/Signature.html#nested-class-summary)), and [`TypeAnnotaton.TargetInfo`](https://cr.openjdk.org/~nbena
 lla/GeneratedDocs/8325369-nested-sealed-svg/api/java.base/java/lang/classfile/TypeAnnotation.TargetInfo.html) ([before](https://download.java.net/java/early_access/jdk23/docs/api/java.base/java/lang/classfile/TypeAnnotation.TargetInfo.html)).
> 
> Courtesy to @nizarbenalla for hosting the updated doc rendering.

Approved, with an optional suggestion

make/jdk/src/classes/build/tools/taglet/SealedGraph.java line 324:

> 322:         while (element.getNestingKind() == NestingKind.MEMBER) {
> 323:             element = (TypeElement) element.getEnclosingElement();
> 324:             result = element.getSimpleName().toString() + '.' + result;

This would be slightly simpler as `element.getSimpleName() + "." + result`

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

Marked as reviewed by jjg (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20122#pullrequestreview-2170436370
PR Review Comment: https://git.openjdk.org/jdk/pull/20122#discussion_r1673091121


More information about the build-dev mailing list