[ https://issues.apache.org/jira/browse/DOXIA-727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17808560#comment-17808560 ]
ASF GitHub Bot commented on DOXIA-727: -------------------------------------- michael-o commented on code in PR #197: URL: https://github.com/apache/maven-doxia/pull/197#discussion_r1458617225 ########## doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/AbstractSinkTest.java: ########## @@ -632,7 +632,7 @@ public void testFigureFromUrl() { if (isXmlSink()) { assertThat(wrapXml(actual), isIdenticalTo(wrapXml(expected))); } else { - assertEquals(actual, expected); + assertEquals(expected, actual); Review Comment: These are quite embrassing... > MarkdownSink: Incorrect escaping inside code spans > -------------------------------------------------- > > Key: DOXIA-727 > URL: https://issues.apache.org/jira/browse/DOXIA-727 > Project: Maven Doxia > Issue Type: Bug > Components: Module - Markdown > Reporter: Konrad Windszus > Assignee: Konrad Windszus > Priority: Major > > According to https://spec.commonmark.org/0.30/#backslash-escapes > bq. Backslash escapes do not work in code blocks, code spans, autolinks, or > raw HTML > Currently the escaping is not disabled between {{Sink.inline(...)}} and > {{Sink.inline(...)}} with "code" semantics (also used from > {{Sink.monospaced()}}) (emits a code span) but only in between > {{Sink.verbatim()}} and {{Sink.verbatim()_}} (emits a code block). > autolinks and raw HTML are currently either never emitted or don't use > escaping -- This message was sent by Atlassian Jira (v8.20.10#820010)