[
https://issues.apache.org/jira/browse/DOXIA-723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17806029#comment-17806029
]
ASF GitHub Bot commented on DOXIA-723:
--------------------------------------
michael-o commented on code in PR #194:
URL: https://github.com/apache/maven-doxia/pull/194#discussion_r1450351991
##########
doxia-core/src/main/java/org/apache/maven/doxia/parser/AbstractXmlParser.java:
##########
@@ -194,6 +215,7 @@ protected SinkEventAttributeSet
getAttributesFromParser(XmlPullParser parser) {
* @throws org.apache.maven.doxia.macro.MacroExecutionException if there's
a problem executing a macro
*/
private void parseXml(XmlPullParser parser, Sink sink) throws
XmlPullParserException, MacroExecutionException {
+ sink.setDocumentLocator(new XmlPullParserLocator(parser, null));
Review Comment:
See: https://issues.apache.org/jira/browse/DOXIA-224 and
https://github.com/apache/maven-doxia/commit/70642f0fd1dbc7d0fb7780d347b87e1fd9b2d1b0.
It seems to be a logical reference omitting the format. I think it is similar
to `generator` and `document`:
https://github.com/apache/maven-doxia-sitetools/blob/5fe4a4c5359e6a23b78f385e15f77767cadaee99/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DocumentRenderingContext.java#L66
See how I have been using it throughout.
> Optionally expose document location in Sink
> -------------------------------------------
>
> Key: DOXIA-723
> URL: https://issues.apache.org/jira/browse/DOXIA-723
> Project: Maven Doxia
> Issue Type: Improvement
> Components: Sink API
> Reporter: Konrad Windszus
> Assignee: Konrad Windszus
> Priority: Major
>
> Similar to http://www.saxproject.org/apidoc/org/xml/sax/Locator.html the Sink
> API should provide means to figure out the document location. This should be
> available whenever the Sink events are emitted from parsing a file.
> The locator should expose file name, line number and column number.
> This can be used enhance warning/errors as otherwise it is very hard to
> figure out the root cause of messages like
> https://github.com/apache/maven-doxia/blob/e01880801ca1283b86205e2f7064b9b4dbc84d54/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSink.java#L930.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
