Re: RFR: 8266666: Implementation for snippets [v27]

2021-09-21 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request with a new ta

Re: RFR: 8266666: Implementation for snippets [v25]

2021-09-21 Thread Pavel Rappo
On Tue, 21 Sep 2021 01:33:09 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 37 commits: >> >> - Merge branch 'master' into 826 >> - Improve markup error messages >> >> * C

Re: RFR: 8266666: Implementation for snippets [v26]

2021-09-21 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v25]

2021-09-21 Thread Pavel Rappo
On Mon, 20 Sep 2021 17:01:20 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v25]

2021-09-20 Thread Jonathan Gibbons
On Mon, 20 Sep 2021 17:01:20 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v25]

2021-09-20 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request with a new ta

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-15 Thread Pavel Rappo
On Mon, 13 Sep 2021 18:59:48 GMT, Jonathan Gibbons wrote: >> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/taglets/snippet/ParseException.java >> line 50: >> >>> 48: * @param position the approximate position >>> 49: */ >>> 50: public ParseException(String mes

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-15 Thread Pavel Rappo
On Mon, 13 Sep 2021 17:42:20 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove trailing whitespace to satisfy jcheck > > src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommen

Re: RFR: 8266666: Implementation for snippets [v24]

2021-09-14 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-14 Thread Pavel Rappo
On Tue, 14 Sep 2021 09:32:05 GMT, Pavel Rappo wrote: >> src/jdk.compiler/share/classes/com/sun/source/util/SimpleDocTreeVisitor.java >> line 460: >> >>> 458: * @return the result of {@code defaultAction} >>> 459: * @since 18 >>> 460: */ >> >> optional: reformat the latter part

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-14 Thread Pavel Rappo
On Mon, 13 Sep 2021 11:59:06 GMT, Hannes Wallnöfer wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove trailing whitespace to satisfy jcheck > > src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-14 Thread Pavel Rappo
On Mon, 13 Sep 2021 17:32:38 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove trailing whitespace to satisfy jcheck > > src/jdk.compiler/share/classes/com/sun/source/util/SimpleDocTreeVis

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-14 Thread Pavel Rappo
On Mon, 13 Sep 2021 14:23:59 GMT, Hannes Wallnöfer wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove trailing whitespace to satisfy jcheck > > src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-13 Thread Jonathan Gibbons
On Mon, 13 Sep 2021 18:34:59 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove trailing whitespace to satisfy jcheck > > src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-13 Thread Jonathan Gibbons
On Thu, 9 Sep 2021 15:27:34 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> branc

Re: RFR: 8266666: Implementation for snippets [v7]

2021-09-13 Thread Jonathan Gibbons
On Fri, 20 Aug 2021 18:12:23 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v22]

2021-09-13 Thread Hannes Wallnöfer
On Thu, 9 Sep 2021 13:40:20 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> branc

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-13 Thread Hannes Wallnöfer
On Thu, 9 Sep 2021 15:27:34 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> branc

Re: RFR: 8266666: Implementation for snippets [v23]

2021-09-09 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v22]

2021-09-09 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request with a new ta

Re: RFR: 8266666: Implementation for snippets [v21]

2021-08-31 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request with a new ta

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-25 Thread Pavel Rappo
On Wed, 25 Aug 2021 21:50:34 GMT, Jonathan Gibbons wrote: > So you're just suggesting that I should handle the parts related to > tagletPath, right? If so, OK. Yes, I am. I have created a separate bug, where I suggested a patch: https://bugs.openjdk.java.net/browse/JDK-8273001 -

Re: RFR: 8266666: Implementation for snippets [v20]

2021-08-25 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-25 Thread Jonathan Gibbons
On Tue, 24 Aug 2021 21:56:05 GMT, Jonathan Gibbons wrote: >> Generally impressive piece of work. > >> @jonathan-gibbons, do we need to mention the semantic change to >> `com.sun.source.doctree.AttributeTree` in the release notes? Before this PR, >> `AttributeTree` used to only represent HTML at

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-25 Thread Pavel Rappo
On Tue, 24 Aug 2021 21:56:05 GMT, Jonathan Gibbons wrote: >> Generally impressive piece of work. > >> @jonathan-gibbons, do we need to mention the semantic change to >> `com.sun.source.doctree.AttributeTree` in the release notes? Before this PR, >> `AttributeTree` used to only represent HTML at

Re: RFR: 8266666: Implementation for snippets [v19]

2021-08-25 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v18]

2021-08-25 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has refreshed the contents of this pull re

Re: RFR: 8266666: Implementation for snippets [v16]

2021-08-25 Thread Pavel Rappo
On Wed, 25 Aug 2021 16:00:08 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v17]

2021-08-25 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has refreshed the contents of this pull re

Re: RFR: 8266666: Implementation for snippets [v16]

2021-08-25 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v15]

2021-08-24 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request with a new ta

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Pavel Rappo
On Tue, 24 Aug 2021 21:56:05 GMT, Jonathan Gibbons wrote: > > @jonathan-gibbons, do we need to mention the semantic change to > > `com.sun.source.doctree.AttributeTree` in the release notes? Before this > > PR, `AttributeTree` used to only represent HTML attributes. After this PR > > has been

Re: RFR: 8266666: Implementation for snippets [v14]

2021-08-24 Thread Pavel Rappo
On Tue, 24 Aug 2021 17:46:51 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Pavel Rappo
On Fri, 20 Aug 2021 18:42:53 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Jonathan Gibbons
On Fri, 20 Aug 2021 18:50:44 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v14]

2021-08-24 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request with a new ta

Re: RFR: 8266666: Implementation for snippets [v13]

2021-08-24 Thread Jonathan Gibbons
On Mon, 26 Jul 2021 14:31:27 GMT, Hannes Wallnöfer wrote: > I would prefer if variables such as this that are used farther than a few > lines from their definition had more meaningful names. It's also confusing that `r` is a `String` but `r1`, `r2` (lines 209-211) are `StyledText` ---

Re: RFR: 8266666: Implementation for snippets [v13]

2021-08-24 Thread Jonathan Gibbons
On Tue, 24 Aug 2021 13:14:57 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Pavel Rappo
On Fri, 20 Aug 2021 18:50:44 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Jonathan Gibbons
On Tue, 24 Aug 2021 11:30:54 GMT, Pavel Rappo wrote: >> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/resources/doclets.properties >> line 360: >> >>> 358: >>> 359: doclet.tag.attribute.repeated=\ >>> 360: repeated attribute: "{0}" >> >> check the resource file for other

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Jonathan Gibbons
On Tue, 24 Aug 2021 10:24:35 GMT, Pavel Rappo wrote: >> Since we are here, is there any langtools convention for when to collapse a >> span of single-import declarations into an import-on-demand (.*) declaration? > > Fixed in a7cd155. > Since we are here, is there any langtools convention for w

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Pavel Rappo
On Fri, 20 Aug 2021 18:11:05 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v13]

2021-08-24 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Pavel Rappo
On Fri, 20 Aug 2021 18:07:30 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Pavel Rappo
On Mon, 23 Aug 2021 18:03:51 GMT, Pavel Rappo wrote: >> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/taglets/SnippetTaglet.java >> line 39: >> >>> 37: import jdk.javadoc.internal.doclets.toolkit.taglets.snippet.Parser; >>> 38: import jdk.javadoc.internal.doclets.toolkit.ta

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-24 Thread Pavel Rappo
On Fri, 20 Aug 2021 18:13:32 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v12]

2021-08-24 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v11]

2021-08-23 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Pavel Rappo
On Fri, 20 Aug 2021 18:09:47 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Jonathan Gibbons
On Fri, 20 Aug 2021 18:46:43 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Jonathan Gibbons
On Mon, 23 Aug 2021 17:36:02 GMT, Jonathan Gibbons wrote: >>> This discussion does not belong here. >> >> I've hoped to specify tag syntax before we push this PR. Until the syntax >> has been specified, the comment has to be stored somewhere. What would be an >> appropriate place to store that

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Jonathan Gibbons
On Mon, 23 Aug 2021 16:01:51 GMT, Pavel Rappo wrote: >> src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommentParser.java >> line 923: >> >>> 921: * probably be considered on a case-by-case basis, for an >>> attribute >>> 922: * value of the {@snippet} tag we do

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Pavel Rappo
On Fri, 20 Aug 2021 18:25:02 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Pavel Rappo
On Fri, 20 Aug 2021 18:03:12 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v10]

2021-08-23 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Pavel Rappo
On Fri, 20 Aug 2021 17:54:37 GMT, Jonathan Gibbons wrote: > This discussion does not belong here. I've hoped to specify tag syntax before we push this PR. Until the syntax has been specified, the comment has to be stored somewhere. What would be an appropriate place to store that comment in?

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Pavel Rappo
On Fri, 20 Aug 2021 17:46:38 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v9]

2021-08-23 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Pavel Rappo
On Fri, 20 Aug 2021 17:49:03 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-23 Thread Pavel Rappo
On Fri, 20 Aug 2021 17:45:21 GMT, Jonathan Gibbons wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Pass through FIXMEs and TODOs >> >> Downgrades FIXMEs that do not mark *feature issues* to TODOs, or removes >

Re: RFR: 8266666: Implementation for snippets [v8]

2021-08-23 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request with a new ta

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-20 Thread Jonathan Gibbons
On Fri, 6 Aug 2021 12:24:06 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> branc

Re: RFR: 8266666: Implementation for snippets [v7]

2021-08-06 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v6]

2021-07-29 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v5]

2021-07-29 Thread Hannes Wallnöfer
On Wed, 28 Jul 2021 14:40:55 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v5]

2021-07-29 Thread Hannes Wallnöfer
On Wed, 28 Jul 2021 14:40:55 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v5]

2021-07-29 Thread Pavel Rappo
On Fri, 23 Jul 2021 20:13:56 GMT, Jonathan Gibbons wrote: > the existing seeTagToContent handles all flavors of @see tag, and {@link}. All but fake ones. The `seeTagToContent` method uses the path (`DocTreePath`) of the `see` tag being passed. In the case of snippet markup, there's no such tag

Re: RFR: 8266666: Implementation for snippets [v5]

2021-07-28 Thread Pavel Rappo
On Wed, 28 Jul 2021 15:31:11 GMT, liach wrote: >> Pavel Rappo has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Change AnnotatedText to StyledText >> >>Renames the class, removes its generic parameter, and changes the related >

Re: RFR: 8266666: Implementation for snippets [v5]

2021-07-28 Thread liach
On Wed, 28 Jul 2021 14:40:55 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v5]

2021-07-28 Thread Pavel Rappo
On Mon, 26 Jul 2021 12:55:17 GMT, Hannes Wallnöfer wrote: >> Pavel Rappo has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Change AnnotatedText to StyledText >> >>Renames the class, removes its generic parameter, and changes the

Re: RFR: 8266666: Implementation for snippets [v5]

2021-07-28 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v4]

2021-07-27 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v4]

2021-07-27 Thread Pavel Rappo
On Mon, 26 Jul 2021 15:25:03 GMT, Hannes Wallnöfer wrote: >> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/resources/stylesheet.css >> line 881: >> >>> 879: .bold { >>> 880: font-weight: bold; >>> 881: } >> >> Both the name and definition seem very global. Maybe it wo

Re: RFR: 8266666: Implementation for snippets [v3]

2021-07-27 Thread Hannes Wallnöfer
On Tue, 27 Jul 2021 15:59:00 GMT, Pavel Rappo wrote: >> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/taglets/snippet/parser/Parser.java >> line 275: >> >>> 273: throw new ParseException("Unexpected >>> attributes", t.lineSourceOffset + t.markupLine

Re: RFR: 8266666: Implementation for snippets [v3]

2021-07-27 Thread Pavel Rappo
On Mon, 26 Jul 2021 17:05:45 GMT, Hannes Wallnöfer wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Improve comments >> >> This commit adds trivial commentary to some of the Java declarations. This >> commit al

Re: RFR: 8266666: Implementation for snippets [v3]

2021-07-27 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets [v2]

2021-07-27 Thread Pavel Rappo
On Tue, 27 Jul 2021 11:06:35 GMT, Pavel Rappo wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >> bran

Re: RFR: 8266666: Implementation for snippets [v2]

2021-07-27 Thread Pavel Rappo
> This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch. Pavel Rappo has updated the pull request incrementally

Re: RFR: 8266666: Implementation for snippets

2021-07-26 Thread Hannes Wallnöfer
On Fri, 23 Jul 2021 22:35:38 GMT, Jonathan Gibbons wrote: >> This PR implements JEP 413 "Code Snippets in Java API Documentation", which >> hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge >> of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets >>

Re: RFR: 8266666: Implementation for snippets

2021-07-26 Thread Hannes Wallnöfer
On Thu, 15 Jul 2021 14:13:16 GMT, Pavel Rappo wrote: > This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch.

Re: RFR: 8266666: Implementation for snippets

2021-07-26 Thread Jonathan Gibbons
The comments don't need to be of core-libs quality, but at least some of the problems we have had in the code of late has been discerning the original intent, so at least some amount of comment/documentation is useful, even if we don't run it through javadoc or even doclint. Think of the comme

Re: RFR: 8266666: Implementation for snippets

2021-07-26 Thread Pavel Rappo
On Fri, 23 Jul 2021 18:48:27 GMT, Jonathan Gibbons wrote: > More feedback; > > 1. It seems reasonable to introduce the `...taglets.snippets` package. It > seems excessive to have three sub packages within that, for `actions`, > `parser` and `text`, especially when the last contains just a sin

Re: RFR: 8266666: Implementation for snippets

2021-07-26 Thread Pavel Rappo
On Thu, 15 Jul 2021 14:13:16 GMT, Pavel Rappo wrote: > This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch.

Re: RFR: 8266666: Implementation for snippets

2021-07-23 Thread Jonathan Gibbons
On Thu, 15 Jul 2021 14:13:16 GMT, Pavel Rappo wrote: > This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch.

Re: RFR: 8266666: Implementation for snippets

2021-07-23 Thread Jonathan Gibbons
On Thu, 15 Jul 2021 14:13:16 GMT, Pavel Rappo wrote: > This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch.

Re: RFR: 8266666: Implementation for snippets

2021-07-23 Thread Jonathan Gibbons
On Thu, 15 Jul 2021 14:13:16 GMT, Pavel Rappo wrote: > This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch.

Re: RFR: 8266666: Implementation for snippets

2021-07-23 Thread Jonathan Gibbons
On Thu, 15 Jul 2021 14:13:16 GMT, Pavel Rappo wrote: > This PR implements JEP 413 "Code Snippets in Java API Documentation", which > hasn't been yet proposed to target JDK 18. The PR starts as a squashed merge > of the https://github.com/openjdk/jdk-sandbox/tree/jdk.javadoc/snippets > branch.