[jdk11u-dev] Withdrawn: 8222669: Create and use new html.Entity class
Amos Shi
ashi at openjdk.org
Mon Apr 29 21:36:11 UTC 2024
On Thu, 25 Apr 2024 21:46:44 GMT, Amos Shi <ashi at openjdk.org> wrote:
> Backport of [JDK-8222669](https://bugs.openjdk.org/browse/JDK-8222669). The reason is `8222669` is a dependency to [JDK-8253700](https://bugs.openjdk.org/browse/JDK-8253700).
> - This PR contains two commits
> - Commit 1: is the `git apply` change
> - Commit 2: is the manual merge of the `.rej` files. All changes has two types
> - Add missing `import`, or re-organize the `import` section based on the original commit
> - Change `Contents.SPACE` to `Entity.NO_BREAK_SPACE` based on the original commit
>
> Sample manual merge file
> - `NestedClassWriterImpl.java.rej`
>
>
> @@ -32,6 +32,7 @@
> import javax.lang.model.element.TypeElement;
>
> import jdk.javadoc.internal.doclets.formats.html.markup.ContentBuilder;
> +import jdk.javadoc.internal.doclets.formats.html.markup.Entity;
> import jdk.javadoc.internal.doclets.formats.html.markup.HtmlStyle;
> import jdk.javadoc.internal.doclets.formats.html.markup.HtmlTree;
> import jdk.javadoc.internal.doclets.formats.html.markup.StringContent;
> @@ -154,7 +155,7 @@
> : resources.getText("doclet.Nested_Classes_Interfaces_Inherited_From_Class"));
> }
> Content labelHeading = HtmlTree.HEADING(Headings.TypeDeclaration.SUMMARY_HEADING, label);
> - labelHeading.add(Contents.SPACE);
> + labelHeading.add(Entity.NO_BREAK_SPACE);
> labelHeading.add(classLink);
> inheritedTree.add(labelHeading);
> }
>
>
>
>
> Testing
> - Local: Test passed on `MacOS 14.4.1`
> - `dev-8222669-11/test/langtools/jdk/javadoc/doclet`: Test results: passed: `191`
> - Pipeline: All checks have passed
> - Testing Machine: SAP nightlies passed on `2024-04-27`
> - jdk/javadoc/doclet/5093723/T5093723.java: SUCCESSFUL GitHub 📊⏲ - [58,605 msec]
> - jdk/javadoc/doclet/AccessAsciiArt/AccessAsciiArt.java: SUCCESSFUL GitHub 📊⏲ - [38,695 msec]
> - jdk/javadoc/doclet/AccessSummary/AccessSummary.java: SUCCESSFUL GitHub 📊⏲ - [44,981 msec]
> - jdk/javadoc/doclet/ValidHtml/ValidHtml.java: SUCCESSFUL GitHub 📊⏲ - [4,706 msec]
> - jdk/javadoc/doclet/VersionNumber/VersionNumber.java: SUCCESSFUL GitHub 📊⏲ - [2,702 msec]
> - ...
This pull request has been closed without being integrated.
-------------
PR: https://git.openjdk.org/jdk11u-dev/pull/2680
More information about the jdk-updates-dev
mailing list