[ 
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)

Reply via email to