[
https://issues.apache.org/jira/browse/DOXIA-709?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17796867#comment-17796867
]
ASF GitHub Bot commented on DOXIA-709:
--------------------------------------
kwin commented on code in PR #182:
URL: https://github.com/apache/maven-doxia/pull/182#discussion_r1427098622
##########
doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkEventTestingSink.java:
##########
@@ -803,11 +488,6 @@ public void division(SinkEventAttributes attributes) {
addEvent("division", new Object[] {attributes});
}
- @Override
- public void verbatim() {
- addEvent("verbatim");
- }
-
@Override
public void verbatim(SinkEventAttributes attributes) {
addEvent("verbatim", new Object[] {attributes});
Review Comment:
No, the events are only added through the methods with the
SinkEventAttributes arguments. The other ones just internally delegate to those
(via the AbstractSink).
> Clarify usage of overlapping Sink methods
> -----------------------------------------
>
> Key: DOXIA-709
> URL: https://issues.apache.org/jira/browse/DOXIA-709
> Project: Maven Doxia
> Issue Type: Improvement
> Components: Sink API
> Reporter: Konrad Windszus
> Priority: Major
>
> With Doxia 1.1 new methods have been added taking an additional
> {{SinkEventAttributes}} parameter. Semantically there is now an overlap, e.g.
> between {{Sink.sectionTitle1}} and {{Sink.sectionTitle(1, null)}}
> It should be clear from the javadoc when which one is supposed to be called,
> and also which one is the preferred method now.
> E.g. in {{SinkAdapter}} all new Doxia 1.1 methods taking a
> {{SinkEventAttributes}} just forward to the equivalent method not taking it.
> OTOH {{Xhtml5BaseSink}} delegates the other way around (e.g. for
> {{link(String)}}) or does not delegate at all (e.g. for {{sectionTitle(int,
> SinkEventAttributes)}} and {{sectionTitle1()}}).
> That leads to the fact that things are often not rendered consistently
> although from their javadoc they should lead to the same result.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
