Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-16 Thread via GitHub
michael-o commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1428911493 ## doxia-core/src/main/java/org/apache/maven/doxia/index/IndexEntry.java: ## @@ -304,7 +310,7 @@ public String toString(int depth) { message.append(",

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-16 Thread via GitHub
michael-o commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1428900306 ## doxia-core/src/main/java/org/apache/maven/doxia/index/IndexingSink.java: ## @@ -104,15 +106,21 @@ private IndexingSink(IndexEntry rootEntry, Sink delegate) {

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-16 Thread via GitHub
michael-o commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1428795972 ## doxia-core/src/main/java/org/apache/maven/doxia/parser/AbstractParser.java: ## @@ -189,6 +229,25 @@ protected boolean isSecondParsing() { return secondP

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-16 Thread via GitHub
michael-o commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1428794860 ## doxia-core/src/main/java/org/apache/maven/doxia/index/IndexEntry.java: ## @@ -105,6 +110,10 @@ protected void setId(String id) { this.id = id; }

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-16 Thread via GitHub
michael-o commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1428794572 ## doxia-core/src/main/java/org/apache/maven/doxia/parser/Parser.java: ## @@ -83,4 +85,34 @@ public interface Parser { * @return true (default value) if comme

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-15 Thread via GitHub
kwin commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1428394870 ## doxia-core/src/main/java/org/apache/maven/doxia/parser/Parser.java: ## @@ -83,4 +85,34 @@ public interface Parser { * @return true (default value) if comment Do

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-15 Thread via GitHub
kwin commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1428392892 ## doxia-core/src/main/java/org/apache/maven/doxia/parser/Parser.java: ## @@ -83,4 +85,34 @@ public interface Parser { * @return true (default value) if comment Do

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-15 Thread via GitHub
michael-o commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1428238514 ## doxia-core/src/main/java/org/apache/maven/doxia/parser/Parser.java: ## @@ -83,4 +85,34 @@ public interface Parser { * @return true (default value) if comme

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-03 Thread via GitHub
michael-o commented on PR #180: URL: https://github.com/apache/maven-doxia/pull/180#issuecomment-1837574750 > > I don't see need to modify/intercept the sink at all, a parser can do this perfectly with some buffering. TOCMacro does this buffering with a fresh parser as well. > > The

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-03 Thread via GitHub
kwin commented on PR #180: URL: https://github.com/apache/maven-doxia/pull/180#issuecomment-1837439220 > I don't see need to modify/intercept the sink at all, a parser can do this perfectly with some buffering. TOCMacro does this buffering with a fresh parser as well. The point is th

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-12-02 Thread via GitHub
michael-o commented on PR #180: URL: https://github.com/apache/maven-doxia/pull/180#issuecomment-1837260464 > > This seems like a reimplementation of RandomAccessSink: The RandomAccessSink provides the ability to create a {https://github.com/link Sink} with hooks. Why not evaluate it if it

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-11-29 Thread via GitHub
kwin commented on PR #180: URL: https://github.com/apache/maven-doxia/pull/180#issuecomment-1831556494 > This seems like a reimplementation of RandomAccessSink: The RandomAccessSink provides the ability to create a {https://github.com/link Sink} with hooks. Why not evaluate it if it there.

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-11-28 Thread via GitHub
michael-o commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1408359647 ## doxia-core/src/main/java/org/apache/maven/doxia/macro/MacroRequest.java: ## @@ -40,15 +40,23 @@ public class MacroRequest { /** A map of parameters. */

Re: [PR] [DOXIASITETOOLS-320] Auto-generate anchor for TOC entries [maven-doxia]

2023-11-28 Thread via GitHub
kwin commented on code in PR #180: URL: https://github.com/apache/maven-doxia/pull/180#discussion_r1408134053 ## doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/NamesForIndexEntriesSinkWrapper.java: ## @@ -0,0 +1,136 @@ +/* + * Licensed to the Apache Software Foundatio