From duke at openjdk.org Mon Jun 3 06:37:20 2024 From: duke at openjdk.org (psoujany) Date: Mon, 3 Jun 2024 06:37:20 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v24] In-Reply-To: References: Message-ID: > The scrollable element `
` with non-interactive content is not tabbable. Grid columns in the javadoc stylesheet has overflow: auto, which is failing Accessibility checks. > https://bugs.openjdk.org/browse/JDK-8325690 psoujany has updated the pull request incrementally with one additional commit since the last revision: Add tabindex only to elemetns which are not tabbable ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17819/files - new: https://git.openjdk.org/jdk/pull/17819/files/eda64006..95f99497 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=23 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=22-23 Stats: 14 lines in 1 file changed: 1 ins; 10 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/17819.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/17819/head:pull/17819 PR: https://git.openjdk.org/jdk/pull/17819 From duke at openjdk.org Mon Jun 3 06:40:05 2024 From: duke at openjdk.org (psoujany) Date: Mon, 3 Jun 2024 06:40:05 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v23] In-Reply-To: References: <4qdAGPJ4wHfvhkkALNrnsEHdSAu2guRqMxfDYnsz8b4=.c9bbf75b-608e-40ae-b261-d5539eb12708@github.com> Message-ID: On Fri, 31 May 2024 14:20:50 GMT, Hannes Walln?fer wrote: >> psoujany has updated the pull request incrementally with two additional commits since the last revision: >> >> - Add tab order to plain text >> - Add tab order to plain text > > src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/Table.java line 349: > >> 347: * For more information on the `widget_tabbable_single` rule, please refer to the following documentation: >> 348: * widget_tabbable_single >> 349: */ > > I think this new comment is a bit excessive. Can it be boiled down to a single sentence, maybe one or two lines? Also, there's no need to use `/**` doc comment style here. For example (but feel free to improve): > > // Only make element tabbable if it doesn't contain tabbable content. > > Regarding the link, I don't think the content after the `#` is necessary. I would prefer the following form: > > // https://able.ibm.com/rules/archives/latest/doc/en-US/widget_tabbable_single.html I simplified comment as per your suggestion. Thank You. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17819#discussion_r1623851288 From duke at openjdk.org Mon Jun 3 06:46:10 2024 From: duke at openjdk.org (psoujany) Date: Mon, 3 Jun 2024 06:46:10 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v23] In-Reply-To: References: <4qdAGPJ4wHfvhkkALNrnsEHdSAu2guRqMxfDYnsz8b4=.c9bbf75b-608e-40ae-b261-d5539eb12708@github.com> Message-ID: On Fri, 31 May 2024 14:23:06 GMT, Hannes Walln?fer wrote: >> psoujany has updated the pull request incrementally with two additional commits since the last revision: >> >> - Add tab order to plain text >> - Add tab order to plain text > > src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/Table.java line 335: > >> 333: } >> 334: int colIndex = 0; >> 335: String regex = "<(?:a|area|button|input|object|select|textarea)\\b"; > > The only element that will likely occur in a API description is `a`. Is it actually the case that a contained link will make the table cell tabbable and scrollable via keyboard? We initially added all elements which have default tab-order to avoid potential violations. But `` should be sufficient as it's the only potential option possible in API description. Link will only make table cell tabbable via keyboard. As CSS is adding overflow: auto on table cells, making cells to be tabbable. The proposed solution here makes table cells accessible via keyboard by adding tabindex. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17819#discussion_r1623857346 From duke at openjdk.org Mon Jun 3 07:05:31 2024 From: duke at openjdk.org (psoujany) Date: Mon, 3 Jun 2024 07:05:31 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v25] In-Reply-To: References: Message-ID: > The scrollable element `
` with non-interactive content is not tabbable. Grid columns in the javadoc stylesheet has overflow: auto, which is failing Accessibility checks. > https://bugs.openjdk.org/browse/JDK-8325690 psoujany 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 28 additional commits since the last revision: - Merge branch 'openjdk:master' into remove-overflow - Add tabindex only to elemetns which are not tabbable - Add tab order to plain text - Add tab order to plain text - Add tabindex to plain text - Add tabindex to plain text - Remove trailing whitespaces - Add tabindex to grid columns - Add tabindex to grid columns - Add tabindex - ... and 18 more: https://git.openjdk.org/jdk/compare/1d8695f4...cff5ca30 ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17819/files - new: https://git.openjdk.org/jdk/pull/17819/files/95f99497..cff5ca30 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=24 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=23-24 Stats: 724439 lines in 9515 files changed: 182054 ins; 181766 del; 360619 mod Patch: https://git.openjdk.org/jdk/pull/17819.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/17819/head:pull/17819 PR: https://git.openjdk.org/jdk/pull/17819 From duke at openjdk.org Mon Jun 3 07:11:17 2024 From: duke at openjdk.org (psoujany) Date: Mon, 3 Jun 2024 07:11:17 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v26] In-Reply-To: References: Message-ID: > The scrollable element `
` with non-interactive content is not tabbable. Grid columns in the javadoc stylesheet has overflow: auto, which is failing Accessibility checks. > https://bugs.openjdk.org/browse/JDK-8325690 psoujany has updated the pull request incrementally with one additional commit since the last revision: Make only elements which are tabbable ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17819/files - new: https://git.openjdk.org/jdk/pull/17819/files/cff5ca30..268fbfa0 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=25 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=24-25 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.org/jdk/pull/17819.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/17819/head:pull/17819 PR: https://git.openjdk.org/jdk/pull/17819 From jlahoda at openjdk.org Mon Jun 3 07:34:18 2024 From: jlahoda at openjdk.org (Jan Lahoda) Date: Mon, 3 Jun 2024 07:34:18 GMT Subject: RFR: 8325168: JShell should support Markdown comments [v6] In-Reply-To: References: Message-ID: > This is an attempt to add Markdown support in documentation comments to JShell. > > It works by converting the Markdown text to HTML during the process of resolving `{@inheritDoc}` tags. Jan Lahoda has updated the pull request incrementally with one additional commit since the last revision: Reflecting review comments. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17686/files - new: https://git.openjdk.org/jdk/pull/17686/files/d85a2b8a..64c984d6 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17686&range=05 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17686&range=04-05 Stats: 128 lines in 3 files changed: 21 ins; 4 del; 103 mod Patch: https://git.openjdk.org/jdk/pull/17686.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/17686/head:pull/17686 PR: https://git.openjdk.org/jdk/pull/17686 From duke at openjdk.org Mon Jun 3 08:42:19 2024 From: duke at openjdk.org (psoujany) Date: Mon, 3 Jun 2024 08:42:19 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v27] In-Reply-To: References: Message-ID: > The scrollable element `
` with non-interactive content is not tabbable. Grid columns in the javadoc stylesheet has overflow: auto, which is failing Accessibility checks. > https://bugs.openjdk.org/browse/JDK-8325690 psoujany has updated the pull request incrementally with one additional commit since the last revision: Make only elements which are tabbable ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17819/files - new: https://git.openjdk.org/jdk/pull/17819/files/268fbfa0..4b4d829b Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=26 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=25-26 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/17819.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/17819/head:pull/17819 PR: https://git.openjdk.org/jdk/pull/17819 From duke at openjdk.org Mon Jun 3 10:12:25 2024 From: duke at openjdk.org (psoujany) Date: Mon, 3 Jun 2024 10:12:25 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v28] In-Reply-To: References: Message-ID: <_b7a8ZSp08t2fYj-3BlN-gqxoAhwnpNqkni0EtxqHv4=.abedd121-41d0-43f4-b6e5-18f60d48983d@github.com> > The scrollable element `
` with non-interactive content is not tabbable. Grid columns in the javadoc stylesheet has overflow: auto, which is failing Accessibility checks. > https://bugs.openjdk.org/browse/JDK-8325690 psoujany has updated the pull request incrementally with one additional commit since the last revision: Add tabindex to tabbable elements ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17819/files - new: https://git.openjdk.org/jdk/pull/17819/files/4b4d829b..aa84ed70 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=27 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=26-27 Stats: 53 lines in 5 files changed: 0 ins; 0 del; 53 mod Patch: https://git.openjdk.org/jdk/pull/17819.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/17819/head:pull/17819 PR: https://git.openjdk.org/jdk/pull/17819 From duke at openjdk.org Mon Jun 3 12:33:31 2024 From: duke at openjdk.org (psoujany) Date: Mon, 3 Jun 2024 12:33:31 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v29] In-Reply-To: References: Message-ID: > The scrollable element `
` with non-interactive content is not tabbable. Grid columns in the javadoc stylesheet has overflow: auto, which is failing Accessibility checks. > https://bugs.openjdk.org/browse/JDK-8325690 psoujany has updated the pull request incrementally with one additional commit since the last revision: Add tabindex to tabbable elements ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17819/files - new: https://git.openjdk.org/jdk/pull/17819/files/aa84ed70..265ee868 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=28 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17819&range=27-28 Stats: 6 lines in 3 files changed: 0 ins; 0 del; 6 mod Patch: https://git.openjdk.org/jdk/pull/17819.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/17819/head:pull/17819 PR: https://git.openjdk.org/jdk/pull/17819 From hannesw at openjdk.org Mon Jun 3 12:43:00 2024 From: hannesw at openjdk.org (Hannes =?UTF-8?B?V2FsbG7DtmZlcg==?=) Date: Mon, 3 Jun 2024 12:43:00 GMT Subject: RFR: 8332039: Cannot invoke "com.sun.source.util.DocTreePath.getTreePath()" because "path" is null In-Reply-To: References: Message-ID: On Fri, 31 May 2024 20:58:31 GMT, Jonathan Gibbons wrote: > > To clarify, I'm not too concerned with how we call an action whereby a nested class gets its `@since` tag from the enclosing class; but yes, "inheritance" might not be ideal. What I'm concerned with is the fact that the logic originally provided in [`getDefaultBlockTags`](https://github.com/openjdk/jdk/blob/6ee8407758c92d32e18642b0758d2d5c71ad09f5/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/taglets/SimpleTaglet.java#L141-L147) is now duplicated in `getInheritedDocTreePath`. > > Like I said, while it's bad, the new duplication merely replicates the one we already have between `InheritableTaglet.inherit` and `getInheritedDocTreePath`. > > If we can't fix this in the next few days, we should file a cleanup issue to resolve this duplication. I don't quite see the problem with this, or how to improve it substantially (beyond finding better terminology). We have different ways of obtaining doc comment fragments from related elements. The major and well-established one is along the lines of Java type inheritance, and a minor/new one along enclosing/enclosed classes. Since `CommentHelper` provides a means to reverse lookup `DocTree` -> `DocTreePath`, it necessarily needs to know about all of these mechanisms. The code for the enclosing/enclosed classes is actually trivial, so I don't see how this could be simplified further. The code for finding doc comments along type inheritance is more complex, but it already delegates to the `DocFinder` class, which is also used for the primary lookup for inherited doc comments. I certainly don't think the existing/proposed code is perfect, but I don't see huge potential for improvement. Of course I'm open to learn and better understand the issues, and how to solve them. ------------- PR Comment: https://git.openjdk.org/jdk/pull/19363#issuecomment-2145104410 From liach at openjdk.org Mon Jun 3 23:49:01 2024 From: liach at openjdk.org (Chen Liang) Date: Mon, 3 Jun 2024 23:49:01 GMT Subject: RFR: 8331947: Preview creates checkbox for JEP-less preview feature [v4] In-Reply-To: References: <_kx1TJGenvZCMoSaeGX1HdxQQWR8t24OOqnrq0SpOmY=.31389dd4-1e6f-4c18-bf15-273520afb6b8@github.com> Message-ID: On Fri, 31 May 2024 10:01:17 GMT, Hannes Walln?fer wrote: >> Please review a simple patch to exclude preview visitor classes meant to support future preview features from the Preview API page. >> >> The test adds an sample element annotated with the new `PreviewFeature.Feature.LANGUAGE_MODEL` constant (which does not have a `@JEP` annotation) to make sure it is not listed in the Preview API page. The test itself does not have to be modified, as it would fail without the change in `PreviewAPIListBuilder.java`. > > Hannes Walln?fer has updated the pull request incrementally with one additional commit since the last revision: > > Address review feedback Marked as reviewed by liach (Author). ------------- PR Review: https://git.openjdk.org/jdk/pull/19344#pullrequestreview-2095041186 From hannesw at openjdk.org Tue Jun 4 10:26:11 2024 From: hannesw at openjdk.org (Hannes =?UTF-8?B?V2FsbG7DtmZlcg==?=) Date: Tue, 4 Jun 2024 10:26:11 GMT Subject: RFR: 8325690: The scrollable element
with non-interactive content is not tabbable [v29] In-Reply-To: References: Message-ID: On Mon, 3 Jun 2024 12:33:31 GMT, psoujany wrote: >> The scrollable element `
` with non-interactive content is not tabbable. Grid columns in the javadoc stylesheet has overflow: auto, which is failing Accessibility checks. >> https://bugs.openjdk.org/browse/JDK-8325690 > > psoujany has updated the pull request incrementally with one additional commit since the last revision: > > Add tabindex to tabbable elements src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/Table.java line 334: > 332: cell.addStyle(rowStyle); > 333: if (!matchFound) { > 334: cell.put(HtmlAttr.ROLE, "tablist") Looking into this further, I don't think the `role="tablist"` attribute is correct here. According to the [MDN documentation], a `tablist` is defined as a container for a set of `tabs`, which does not apply for table cells. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17819#discussion_r1625750412 From jlahoda at openjdk.org Tue Jun 4 11:57:13 2024 From: jlahoda at openjdk.org (Jan Lahoda) Date: Tue, 4 Jun 2024 11:57:13 GMT Subject: Integrated: 8325168: JShell should support Markdown comments In-Reply-To: References: Message-ID: On Fri, 2 Feb 2024 14:02:28 GMT, Jan Lahoda wrote: > This is an attempt to add Markdown support in documentation comments to JShell. > > It works by converting the Markdown text to HTML during the process of resolving `{@inheritDoc}` tags. This pull request has now been integrated. Changeset: 8d3de45f Author: Jan Lahoda URL: https://git.openjdk.org/jdk/commit/8d3de45f4dfd60dc4e2f210cb0c085fcf6efb8e2 Stats: 2056 lines in 9 files changed: 1209 ins; 842 del; 5 mod 8325168: JShell should support Markdown comments Reviewed-by: jjg ------------- PR: https://git.openjdk.org/jdk/pull/17686 From hannesw at openjdk.org Tue Jun 4 17:24:02 2024 From: hannesw at openjdk.org (Hannes =?UTF-8?B?V2FsbG7DtmZlcg==?=) Date: Tue, 4 Jun 2024 17:24:02 GMT Subject: RFR: 8332039: Cannot invoke "com.sun.source.util.DocTreePath.getTreePath()" because "path" is null In-Reply-To: References: Message-ID: On Thu, 23 May 2024 10:39:52 GMT, Hannes Walln?fer wrote: > Please review a patch to fix a NPE thrown when a `@since` tag inherited by a nested class contains a nested inline tag. The solution is to make `CommentHelper.getDocTreePath(DocTree)` able to handle block tags inherited by nested classes. I filed a follow-up issue to refactor the code for inherited and derived doc comments: https://bugs.openjdk.org/browse/JDK-8333557 ------------- PR Comment: https://git.openjdk.org/jdk/pull/19363#issuecomment-2148040277 From hannesw at openjdk.org Tue Jun 4 17:24:03 2024 From: hannesw at openjdk.org (Hannes =?UTF-8?B?V2FsbG7DtmZlcg==?=) Date: Tue, 4 Jun 2024 17:24:03 GMT Subject: Integrated: 8332039: Cannot invoke "com.sun.source.util.DocTreePath.getTreePath()" because "path" is null In-Reply-To: References: Message-ID: On Thu, 23 May 2024 10:39:52 GMT, Hannes Walln?fer wrote: > Please review a patch to fix a NPE thrown when a `@since` tag inherited by a nested class contains a nested inline tag. The solution is to make `CommentHelper.getDocTreePath(DocTree)` able to handle block tags inherited by nested classes. This pull request has now been integrated. Changeset: a706e35b Author: Hannes Walln?fer URL: https://git.openjdk.org/jdk/commit/a706e35b12addff987b489059be8f240c60fae75 Stats: 68 lines in 2 files changed: 49 ins; 3 del; 16 mod 8332039: Cannot invoke "com.sun.source.util.DocTreePath.getTreePath()" because "path" is null Reviewed-by: jjg ------------- PR: https://git.openjdk.org/jdk/pull/19363 From prappo at openjdk.org Wed Jun 5 11:47:58 2024 From: prappo at openjdk.org (Pavel Rappo) Date: Wed, 5 Jun 2024 11:47:58 GMT Subject: RFR: 8331947: Preview creates checkbox for JEP-less preview feature [v4] In-Reply-To: References: <_kx1TJGenvZCMoSaeGX1HdxQQWR8t24OOqnrq0SpOmY=.31389dd4-1e6f-4c18-bf15-273520afb6b8@github.com> Message-ID: On Fri, 31 May 2024 10:01:17 GMT, Hannes Walln?fer wrote: >> Please review a simple patch to exclude preview visitor classes meant to support future preview features from the Preview API page. >> >> The test adds an sample element annotated with the new `PreviewFeature.Feature.LANGUAGE_MODEL` constant (which does not have a `@JEP` annotation) to make sure it is not listed in the Preview API page. The test itself does not have to be modified, as it would fail without the change in `PreviewAPIListBuilder.java`. > > Hannes Walln?fer has updated the pull request incrementally with one additional commit since the last revision: > > Address review feedback Approved with fuzz. src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/PreviewAPIListBuilder.java line 96: > 94: return true; > 95: } > 96: // Preview features without JEP are not included. Trivial: we don't need to mention "support" near "preview" and "features", right? Just checking. test/langtools/jdk/javadoc/doclet/testPreview/TestPreview.java line 161: > 159: """); > 160: > 161: // 8331947: Support preview features without JEP should not be included in Preview API page Trivial: here it is phrased as "Support preview features", while in the actual test source file it is "Preview support feature". ------------- Marked as reviewed by prappo (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/19344#pullrequestreview-2098808073 PR Review Comment: https://git.openjdk.org/jdk/pull/19344#discussion_r1627545672 PR Review Comment: https://git.openjdk.org/jdk/pull/19344#discussion_r1627530580 From hannesw at openjdk.org Wed Jun 5 12:39:58 2024 From: hannesw at openjdk.org (Hannes =?UTF-8?B?V2FsbG7DtmZlcg==?=) Date: Wed, 5 Jun 2024 12:39:58 GMT Subject: RFR: 8331947: Preview creates checkbox for JEP-less preview feature [v4] In-Reply-To: References: <_kx1TJGenvZCMoSaeGX1HdxQQWR8t24OOqnrq0SpOmY=.31389dd4-1e6f-4c18-bf15-273520afb6b8@github.com> Message-ID: On Wed, 5 Jun 2024 11:04:06 GMT, Pavel Rappo wrote: >> Hannes Walln?fer has updated the pull request incrementally with one additional commit since the last revision: >> >> Address review feedback > > test/langtools/jdk/javadoc/doclet/testPreview/TestPreview.java line 161: > >> 159: """); >> 160: >> 161: // 8331947: Support preview features without JEP should not be included in Preview API page > > Trivial: here it is phrased as "Support preview features", while in the actual test source file it is "Preview support feature". I agree the terminology in comments is somewhat inconsistent. I don't think there is an "official" term for these preview features. [In the CSR](https://bugs.openjdk.org/browse/JDK-8329634) they are summarized as follows: "Add a set of persistent preview visitors to support new language changes in development." I think the essential property for our purpose is the "without JEP". I added the "support" adjective to make their purpose clearer, but I don't think it's essential. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/19344#discussion_r1627680852 From hannesw at openjdk.org Wed Jun 5 12:43:01 2024 From: hannesw at openjdk.org (Hannes =?UTF-8?B?V2FsbG7DtmZlcg==?=) Date: Wed, 5 Jun 2024 12:43:01 GMT Subject: Integrated: 8331947: Preview creates checkbox for JEP-less preview feature In-Reply-To: <_kx1TJGenvZCMoSaeGX1HdxQQWR8t24OOqnrq0SpOmY=.31389dd4-1e6f-4c18-bf15-273520afb6b8@github.com> References: <_kx1TJGenvZCMoSaeGX1HdxQQWR8t24OOqnrq0SpOmY=.31389dd4-1e6f-4c18-bf15-273520afb6b8@github.com> Message-ID: <4-C4fdy0mVJi1DUKAW_ZigRR87uyYRoAqK7Y2xrhSdw=.5601a03c-0345-49a0-8f72-d0d310aea336@github.com> On Wed, 22 May 2024 08:39:12 GMT, Hannes Walln?fer wrote: > Please review a simple patch to exclude preview visitor classes meant to support future preview features from the Preview API page. > > The test adds an sample element annotated with the new `PreviewFeature.Feature.LANGUAGE_MODEL` constant (which does not have a `@JEP` annotation) to make sure it is not listed in the Preview API page. The test itself does not have to be modified, as it would fail without the change in `PreviewAPIListBuilder.java`. This pull request has now been integrated. Changeset: 765ad0e4 Author: Hannes Walln?fer URL: https://git.openjdk.org/jdk/commit/765ad0e40bc522de4b2821ccc60b9139faf7376f Stats: 97 lines in 8 files changed: 46 ins; 11 del; 40 mod 8331947: Preview creates checkbox for JEP-less preview feature Reviewed-by: liach, prappo ------------- PR: https://git.openjdk.org/jdk/pull/19344 From michael.osipov at innomotics.com Wed Jun 5 09:51:28 2024 From: michael.osipov at innomotics.com (Osipov, Michael) Date: Wed, 5 Jun 2024 09:51:28 +0000 Subject: Missing element-list for https://docs.oracle.com/javase/8/docs/jre/api/security/jgss/spec In-Reply-To: References: <5947f22d-0c8d-4063-b721-42b7dd44ba4c@siemens.com> <47A81E8E-1DFB-4CE1-A465-7FDE83F183DA@oracle.com> Message-ID: On 2024-05-31 21:38, Jonathan Gibbons wrote: > Michael, > > There is no `element-list` file for any version of JDK before JDK 9. > Before JDK 9, the appropriate information was in the `package-list` > file. In JDK 9, with the introduction of modules, the format of the file > was updated to include modules, and because this was an incompatible > change, the file was renamed as well, to `element-list`. > > It is an annoying misconfiguration of `docs.oracle.com` that you are > seeing `302 Moved Temporarily` instead of `404 Not Found` which would be > a more semantically accurate response. > > That leaves the question of why whatever you were doing was looking for > `element-list` in JDK 8. To answer that, we need more info, to determine > whether it is just a command-line error on your part, or any error in > the `javadoc` tool itself. What version of JDK and javadoc are you > using; what external libraries were you referencing in `-link` or `- > linkoffline` options; and what source level were you using, with either > the `-source` or `--release` options? John, thanks for the elaboration. Let me better clarify what happens. The code is question with a modification for you is available at: https://github.com/michael-o/tomcatspnegoad/tree/javadoc-issue Class net.sf.michaelo.tomcat.pac.asn1.AdIfRelevantAsn1Parser uses com.sun.security.jgss.AuthorizationDataEntry and others use private Sun classes which does not allow me to use -release for now, only -source. The source is Java 8. When I run javadoc:javadoc with Zulu 8 all links are generated successfully. Running Zulu 11 with extracted Javadoc call (&"C:\Program Files\Zulu\zulu-11\bin\javadoc.exe" -verbose "@options" "@packages") gives me no warning, even in verbose mode. It simply does not link. When trying Temurin 22.0.1 (&"C:\Temp\jdk-22.0.1+8\bin\javadoc.exe" -verbose "@options" "@packages") it gives me: > Warnung: URL https://docs.oracle.com/javase/8/docs/jre/api/security/jgss/spec/element-list wurde umgeleitet an https://docs.oracle.com/en/java/javase/22/ - Aktualisieren Sie die Befehlszeilenoptionen, um diese Warnung zu unterdr?cken. That is the redirect. Either I am misunderstanding or I have hit an edge case for public classes outside of the standard JDK (Java namespaces). Here is the input to Javadoc (@options, @packages) if you cannot try yourself: https://gist.github.com/michael-o/212c6797b000914b9142f1f077b1d9df I have tried: > Apache Maven 3.8.8 (4c87b05d9aedce574290d1acc98575ed5eb6cd39) > Maven home: C:\Entwicklung\Programme\apache-maven-3.8.8 > Java version: 11.0.18, vendor: Azul Systems, Inc., runtime: C:\Program Files\Zulu\zulu-11 > Default locale: de_DE, platform encoding: UTF-8 > OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows" > Apache Maven 3.8.8 (4c87b05d9aedce574290d1acc98575ed5eb6cd39) > Maven home: C:\Entwicklung\Programme\apache-maven-3.8.8 > Java version: 1.8.0_362, vendor: Azul Systems, Inc., runtime: C:\Program Files\Zulu\zulu-8\jre > Default locale: de_DE, platform encoding: Cp1252 > OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows" > Apache Maven 3.8.8 (4c87b05d9aedce574290d1acc98575ed5eb6cd39) > Maven home: C:\Entwicklung\Programme\apache-maven-3.8.8 > Java version: 22.0.1, vendor: Eclipse Adoptium, runtime: C:\Temp\jdk-22.0.1+8 > Default locale: de_DE, platform encoding: UTF-8 > OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows" Note: I have routed the Javadoc traffic through Fiddler and it clearly following the misconfigured docs.oracle.com location: > # Result Protocol Host URL Body Caching Content-Type Process Comments Custom > 10 302 HTTPS docs.oracle.com /javase/8/docs/jre/api/security/jgss/spec/element-list 1 javadoc:24660 > # Result Protocol Host URL Body Caching Content-Type Process Comments Custom > 12 200 HTTPS docs.oracle.com /en/java/javase/22/ 33 431 max-age=19052 text/html javadoc:24660 Regards, Michael From jonathan.gibbons at oracle.com Thu Jun 6 23:11:56 2024 From: jonathan.gibbons at oracle.com (Jonathan Gibbons) Date: Thu, 6 Jun 2024 16:11:56 -0700 Subject: [External] : Re: Missing element-list for https://docs.oracle.com/javase/8/docs/jre/api/security/jgss/spec In-Reply-To: References: <5947f22d-0c8d-4063-b721-42b7dd44ba4c@siemens.com> <47A81E8E-1DFB-4CE1-A465-7FDE83F183DA@oracle.com> Message-ID: <94c6cf1c-21f3-4fb6-9c5a-4bd6cada8e4b@oracle.com> On 6/5/24 2:51 AM, Osipov, Michael wrote: > On 2024-05-31 21:38, Jonathan Gibbons wrote: >> Michael, >> >> There is no `element-list` file for any version of JDK before JDK 9. >> Before JDK 9, the appropriate information was in the `package-list` >> file. In JDK 9, with the introduction of modules, the format of the file >> was updated to include modules, and because this was an incompatible >> change, the file was renamed as well, to `element-list`. >> >> It is an annoying misconfiguration of `docs.oracle.com` that you are >> seeing `302 Moved Temporarily` instead of `404 Not Found` which would be >> a more semantically accurate response. >> >> That leaves the question of why whatever you were doing was looking for >> `element-list` in JDK 8. To answer that, we need more info, to determine >> whether it is just a command-line error on your part, or any error in >> the `javadoc` tool itself. What version of JDK and javadoc are you >> using; what external libraries were you referencing in `-link` or `- >> linkoffline` options; and what source level were you using, with either >> the `-source` or `--release` options? > John, > > thanks for the elaboration. Let me better clarify what happens. > > The code is question with a modification for you is available at: > https://urldefense.com/v3/__https://github.com/michael-o/tomcatspnegoad/tree/javadoc-issue__;!!ACWV5N9M2RV99hQ!L6MibNz6g0M99jnVOo2O_Zs2vP8-4CM-NS4WNUmHmL5dB30i0qaSfDJBHW4S_bxjOhImERJfBWGpStJ6WrwE0WAva10xtV86PQ$ > > Class net.sf.michaelo.tomcat.pac.asn1.AdIfRelevantAsn1Parser uses > com.sun.security.jgss.AuthorizationDataEntry and others use private Sun > classes which does not allow me to use -release for now, only -source. > The source is Java 8. When I run javadoc:javadoc with Zulu 8 all links > are generated successfully. Running Zulu 11 with extracted Javadoc call > (&"C:\Program Files\Zulu\zulu-11\bin\javadoc.exe" -verbose "@options" > "@packages") gives me no warning, even in verbose mode. It simply does > not link. > When trying Temurin 22.0.1 (&"C:\Temp\jdk-22.0.1+8\bin\javadoc.exe" > -verbose "@options" "@packages") it gives me: >> Warnung: URLhttps://docs.oracle.com/javase/8/docs/jre/api/security/jgss/spec/element-list wurde umgeleitet anhttps://docs.oracle.com/en/java/javase/22/ - Aktualisieren Sie die Befehlszeilenoptionen, um diese Warnung zu unterdr?cken. > That is the redirect. Either I am misunderstanding or I have hit an edge > case for public classes outside of the standard JDK (Java namespaces). > Here is the input to Javadoc (@options, @packages) if you cannot try > yourself:https://urldefense.com/v3/__https://gist.github.com/michael-o/212c6797b000914b9142f1f077b1d9df__;!!ACWV5N9M2RV99hQ!L6MibNz6g0M99jnVOo2O_Zs2vP8-4CM-NS4WNUmHmL5dB30i0qaSfDJBHW4S_bxjOhImERJfBWGpStJ6WrwE0WAva12c46QzhA$ > > I have tried: >> Apache Maven 3.8.8 (4c87b05d9aedce574290d1acc98575ed5eb6cd39) >> Maven home: C:\Entwicklung\Programme\apache-maven-3.8.8 >> Java version: 11.0.18, vendor: Azul Systems, Inc., runtime: C:\Program Files\Zulu\zulu-11 >> Default locale: de_DE, platform encoding: UTF-8 >> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows" >> Apache Maven 3.8.8 (4c87b05d9aedce574290d1acc98575ed5eb6cd39) >> Maven home: C:\Entwicklung\Programme\apache-maven-3.8.8 >> Java version: 1.8.0_362, vendor: Azul Systems, Inc., runtime: C:\Program Files\Zulu\zulu-8\jre >> Default locale: de_DE, platform encoding: Cp1252 >> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows" >> Apache Maven 3.8.8 (4c87b05d9aedce574290d1acc98575ed5eb6cd39) >> Maven home: C:\Entwicklung\Programme\apache-maven-3.8.8 >> Java version: 22.0.1, vendor: Eclipse Adoptium, runtime: C:\Temp\jdk-22.0.1+8 >> Default locale: de_DE, platform encoding: UTF-8 >> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows" > Note: I have routed the Javadoc traffic through Fiddler and it clearly > following the misconfigured docs.oracle.com location: >> # Result Protocol Host URL Body Caching Content-Type Process Comments Custom >> 10 302 HTTPS docs.oracle.com /javase/8/docs/jre/api/security/jgss/spec/element-list 1 javadoc:24660 >> # Result Protocol Host URL Body Caching Content-Type Process Comments Custom >> 12 200 HTTPS docs.oracle.com /en/java/javase/22/ 33 431 max-age=19052 text/html javadoc:24660 > > Regards, > > Michael > > Michael, I investigated a bit further, and I may have a solution for you. The redirect is annoying, and sad to say, there is nothing I can do to get it fixed in a timely manner.? But, I think you may be able to work around it.? The answer is based on the content of the files in the gist that you referenced in your message. Look at these lines: -link 'https://docs.oracle.com/javase/8/docs/jre/api/security/jgss/spec' -linkoffline 'https://docs.oracle.com/javase/8/docs/api' 'C:/Entwicklung/workspace-2023-06/tomcatspnegoad/tomcat90/target/javadoc-bundle-options' I'm not sure what is in your `javadoc-bundle-options` file, but it *ought* to be a local copy of the `package-list` found here: https://docs.oracle.com/javase/8/docs/api/package-list For the first option, the `-link` one, try changing it to: -linkoffline 'https://docs.oracle.com/javase/8/docs/jre/api/security/jgss/spec' /path/to/file where `/path/to/file` is a local copy of the file found here: https://docs.oracle.com/javase/8/docs/jre/api/security/jgss/spec/package-list which in reality is just: com.sun.security.jgss In other words, both options should be instances of the `-linkoffline` option, which should be of the form: -linkofflinehttps:/url/to/api /path/to/file where `/path/to/file` is a local copy of the file downloaded from `https:/url/to/api/package-list` Note I am using `.../package-list` in this message because you are using `-source 8`.? If you use a newer version of these APIs, you may need to change `package-list` to `element-list`. (I believe the change happened in JDK 11.) For more details on the `-linkoffline` option, see https://docs.oracle.com/en/java/javase/22/docs/specs/man/javadoc.html#option-linkoffline -- Jon -------------- next part -------------- An HTML attachment was scrubbed... URL: From dholmes at openjdk.org Mon Jun 10 02:38:25 2024 From: dholmes at openjdk.org (David Holmes) Date: Mon, 10 Jun 2024 02:38:25 GMT Subject: RFR: 8330205: Initial troff manpage generation for JDK 24 Message-ID: Sets the version to 24/24-ea and the copyright year to 2025. Content changes related to the start of release (e.g. for removed options in java.1) are handled separately. This initial generation also picks up the unpublished changes from: - JDK-8330807: Update Manpage for obsoletion of ScavengeBeforeFullGC - JDK-8284500: Typo in Supported Named Extensions - BasicContraints - JDK-8324342: Doclet should default @since for a nested class to that of its enclosing class Thanks ------------- Commit messages: - 8330205: Initial troff manpage generation for JDK 24 Changes: https://git.openjdk.org/jdk/pull/19617/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=19617&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8330205 Stats: 66 lines in 28 files changed: 12 ins; 13 del; 41 mod Patch: https://git.openjdk.org/jdk/pull/19617.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/19617/head:pull/19617 PR: https://git.openjdk.org/jdk/pull/19617 From alanb at openjdk.org Mon Jun 10 07:18:11 2024 From: alanb at openjdk.org (Alan Bateman) Date: Mon, 10 Jun 2024 07:18:11 GMT Subject: RFR: 8330205: Initial troff manpage generation for JDK 24 In-Reply-To: References: Message-ID: <3hAM5el3W8wXMgyjuCGXpmRjCxld2dvHMiUbvarBPhU=.62cd1645-6bf1-4da4-b70e-70c12e6377c8@github.com> On Mon, 10 Jun 2024 02:31:00 GMT, David Holmes wrote: > Sets the version to 24/24-ea and the copyright year to 2025. > > Content changes related to the start of release (e.g. for removed options in java.1) are handled separately. > > This initial generation also picks up the unpublished changes from: > > - JDK-8330807: Update Manpage for obsoletion of ScavengeBeforeFullGC > - JDK-8284500: Typo in Supported Named Extensions - BasicContraints > - JDK-8324342: Doclet should default @since for a nested class to that of its enclosing class > > Thanks Marked as reviewed by alanb (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/19617#pullrequestreview-2106832118 From dholmes at openjdk.org Mon Jun 10 08:04:12 2024 From: dholmes at openjdk.org (David Holmes) Date: Mon, 10 Jun 2024 08:04:12 GMT Subject: RFR: 8330205: Initial troff manpage generation for JDK 24 In-Reply-To: <3hAM5el3W8wXMgyjuCGXpmRjCxld2dvHMiUbvarBPhU=.62cd1645-6bf1-4da4-b70e-70c12e6377c8@github.com> References: <3hAM5el3W8wXMgyjuCGXpmRjCxld2dvHMiUbvarBPhU=.62cd1645-6bf1-4da4-b70e-70c12e6377c8@github.com> Message-ID: On Mon, 10 Jun 2024 07:15:51 GMT, Alan Bateman wrote: >> Sets the version to 24/24-ea and the copyright year to 2025. >> >> Content changes related to the start of release (e.g. for removed options in java.1) are handled separately. >> >> This initial generation also picks up the unpublished changes from: >> >> - JDK-8330807: Update Manpage for obsoletion of ScavengeBeforeFullGC >> - JDK-8284500: Typo in Supported Named Extensions - BasicContraints >> - JDK-8324342: Doclet should default `@since` for a nested class to that of its enclosing class >> >> Thanks > > Marked as reviewed by alanb (Reviewer). Thanks for the review @AlanBateman ! ------------- PR Comment: https://git.openjdk.org/jdk/pull/19617#issuecomment-2157606095 From iris at openjdk.org Mon Jun 10 17:30:15 2024 From: iris at openjdk.org (Iris Clark) Date: Mon, 10 Jun 2024 17:30:15 GMT Subject: RFR: 8330205: Initial troff manpage generation for JDK 24 In-Reply-To: References: Message-ID: On Mon, 10 Jun 2024 02:31:00 GMT, David Holmes wrote: > Sets the version to 24/24-ea and the copyright year to 2025. > > Content changes related to the start of release (e.g. for removed options in java.1) are handled separately. > > This initial generation also picks up the unpublished changes from: > > - JDK-8330807: Update Manpage for obsoletion of ScavengeBeforeFullGC > - JDK-8284500: Typo in Supported Named Extensions - BasicContraints > - JDK-8324342: Doclet should default `@since` for a nested class to that of its enclosing class > > Thanks Marked as reviewed by iris (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/19617#pullrequestreview-2108383696 From dnguyen at openjdk.org Mon Jun 10 20:05:33 2024 From: dnguyen at openjdk.org (Damon Nguyen) Date: Mon, 10 Jun 2024 20:05:33 GMT Subject: RFR: 8333827: JDK 23 RDP1 L10n resource files update Message-ID: This issue is responsible for updating the translations of all the localize(able) resources in the JDK. Primarily, the changes between JDK 22 RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated. The translation tool adjusted some definitions, which causes some changes in localized files where the source file had no changes. This causes some words being reverted from localized languages to English, and some had its definitions changed. Alternatively, the diffs are viewable here and was generated using John Gibbon's diff tool for l10n: https://cr.openjdk.org/~dnguyen/output2/ ------------- Commit messages: - Review comment edits - Initial localization changes Changes: https://git.openjdk.org/jdk/pull/19609/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=19609&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8333827 Stats: 239 lines in 39 files changed: 133 ins; 57 del; 49 mod Patch: https://git.openjdk.org/jdk/pull/19609.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/19609/head:pull/19609 PR: https://git.openjdk.org/jdk/pull/19609 From achung at openjdk.org Mon Jun 10 20:05:33 2024 From: achung at openjdk.org (Alisen Chung) Date: Mon, 10 Jun 2024 20:05:33 GMT Subject: RFR: 8333827: JDK 23 RDP1 L10n resource files update In-Reply-To: References: Message-ID: On Fri, 7 Jun 2024 22:46:44 GMT, Damon Nguyen wrote: > This issue is responsible for updating the translations of all the localize(able) resources in the JDK. Primarily, the changes between JDK 22 RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated. > > The translation tool adjusted some definitions, which causes some changes in localized files where the source file had no changes. This causes some words being reverted from localized languages to English, and some had its definitions changed. > > Alternatively, the diffs are viewable here and was generated using John Gibbon's diff tool for l10n: > https://cr.openjdk.org/~dnguyen/output2/ Just a question about the reverted translations, otherwise LGTM > The translation tool adjusted some definitions, which causes some changes in localized files where the source file had no changes. This causes some words being reverted from localized languages to English, and some had its definitions changed. Any particular reason you know that the tool had the definitions adjusted? I see some of the changes look like like command line options not being translated, but it's a bit hard to tell. Do you know if the reverted words should be translated (and possibly will be re-translated in future drops?) or should they be left untranslated? ------------- PR Review: https://git.openjdk.org/jdk/pull/19609#pullrequestreview-2105526180 PR Comment: https://git.openjdk.org/jdk/pull/19609#issuecomment-2155723513 From jlu at openjdk.org Mon Jun 10 20:05:33 2024 From: jlu at openjdk.org (Justin Lu) Date: Mon, 10 Jun 2024 20:05:33 GMT Subject: RFR: 8333827: JDK 23 RDP1 L10n resource files update In-Reply-To: References: Message-ID: <6x05vtOnVTgxlyzPvesvYR9v3NUNHNkinbbaXJtXzxU=.492b5e27-02dd-4457-a725-705a3d6be7f9@github.com> On Fri, 7 Jun 2024 22:46:44 GMT, Damon Nguyen wrote: > This issue is responsible for updating the translations of all the localize(able) resources in the JDK. Primarily, the changes between JDK 22 RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated. > > The translation tool adjusted some definitions, which causes some changes in localized files where the source file had no changes. This causes some words being reverted from localized languages to English, and some had its definitions changed. > > Alternatively, the diffs are viewable here and was generated using John Gibbon's diff tool for l10n: > https://cr.openjdk.org/~dnguyen/output2/ Cannot speak to the translations themselves, but in general the changes LGTM pending the comments. src/java.desktop/share/classes/sun/print/resources/serviceui_de.properties line 1: > 1: # It looks like these .properties files need to have their copyright years bumped. I believe since the original was not bumped, the translation tool did not reflect the correct year in the localized versions. src/java.desktop/share/classes/sun/print/resources/serviceui_zh_CN.properties line 66: > 64: label.size=??(&Z): > 65: label.source=??(&C): > 66: label.outputbins=????(&P)? Looks like the translation tool changed this from a colon (U+003A) to a full width colon (U+FF1A). There are l10n rules when it comes to punctuation, I am not sure if this is a result of that. It looks like the surrounding key/values are simply using (U+003A) colons. If we decide to revert this colon change, we need to also probably file a bug against the translation tool. src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/msg/XMLMessages_de.properties line 278: > 276: UndeclaredElementInContentSpec = Contentmodell des Elements "{0}" verweist auf das nicht deklarierte Element "{1}". > 277: UniqueNotationName = Deklaration f?r die Notation "{0}" ist nicht eindeutig. Ein jeweiliger Name darf nicht in mehreren Notationsdeklarationen deklariert werden. > 278: ENTITYFailedInitializeGrammar = ENTITYDatatype-Validator: Nicht erfolgreich. Initialisierungsmethode muss mit einer g?ltigen Grammatikreferenz aufgerufen werden. \t It looks like the _zh_CN_ file should also have the `\t` removed, but such changes are done by the translation tool. I think in this case, we should manually remove it, and then file a bug against the translation tool. src/jdk.jconsole/share/classes/sun/tools/jconsole/resources/messages_de.properties line 285: > 283: VIRTUAL_MACHINE=Virtuelle Maschine > 284: VM_ARGUMENTS=VM-Argumente > 285: VMINTERNAL_FRAME_ACCESSIBLE_DESCRIPTION=Innerer Frame f?r das Monitoring einer Java Virtual Machine This one-off change is consistent with the change in _src/jdk.jdi/share/classes/com/sun/tools/example/debug/tty/TTYResources_de.java_. src/jdk.jshell/share/classes/jdk/internal/jshell/tool/resources/l10n_de.properties line 183: > 181: jshell.fix.wrong.shortcut =Unerwartetes Zeichen nach Umschalt+Tab.\nVerwenden Sie "I" f?r automatischen Import, "V" zur Variablenerstellung oder "M" zur Methodenerstellung.\nWeitere Informationen finden Sie unter:\n/help shortcuts > 182: > 183: help.usage = Verwendung: jshell