Allow consecutive question marks in inline processing instructions

Author: ybai08

commonmark/src/main/java/org/commonmark/internal/inline/HtmlInlineParser.java

@@ -164,17 +164,13 @@ private boolean tryProcessingInstruction(Scanner scanner) {
             return false;
         }
         scanner.next();
-        int found;
-        while ((found = scanner.find('?')) > 0) {
+        while (scanner.find('?') >= 0) {
             scanner.next();
             if (scanner.next('>')) {
                 return true;
             }
         }
-        // The loop also ends when a `?` is found directly at the current position (`found == 0`),
-        // which is not the end of the input, so only remember the miss when the scan really did
-        // reach the end.
-        noProcessingInstructionEnd = found < 0;
+        noProcessingInstructionEnd = true;
         return false;
     }
 

commonmark/src/test/java/org/commonmark/test/HtmlInlineParserTest.java

@@ -21,11 +21,17 @@ public void cdata() {
         assertRendering("inline <![CDATA[ ] ]] ]]>", "<p>inline <![CDATA[ ] ]] ]]></p>\n");
     }
 
+    @Test
+    public void processingInstruction() {
+        assertRendering("inline <??>", "<p>inline <??></p>\n");
+        assertRendering("inline <?a??>", "<p>inline <?a??></p>\n");
+        assertRendering("inline <?? <?php ?>", "<p>inline <?? <?php ?></p>\n");
+    }
+
     @Test
     public void afterFailedAttempt() {
         // A `<` that doesn't start inline HTML must not stop a later one from being parsed
         assertRendering("inline <!- <!-- a -->", "<p>inline &lt;!- <!-- a --></p>\n");
-        assertRendering("inline <?? <?php ?>", "<p>inline &lt;?? <?php ?></p>\n");
         assertRendering(
                 "inline <![x]]> <![CDATA[a]]>", "<p>inline &lt;![x]]&gt; <![CDATA[a]]></p>\n");
         assertRendering("inline <!foo> <!bar baz>", "<p>inline &lt;!foo&gt; <!bar baz></p>\n");