Update release notes wrt #86 fix

FasterXML · Jun 7, 2024 · 4bc3e9b · 4bc3e9b
1 parent 14b58b9
commit 4bc3e9b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/release-notes/VERSION b/release-notes/VERSION
@@ -6,6 +6,9 @@ Project: aalto-xml
 
 1.3.3 (not yet released)
 
+#86: Multi-byte characters are split in writeCData() if first byte sits
+  right at the end of the buffer
+ (reported, fix contributed by @tatsel)
 #90: Update stax2-api dep to 4.2.2 (from 4.2)
 
 1.3.2 (25-Apr-2022)

diff --git a/src/main/java/com/fasterxml/aalto/out/ByteXmlWriter.java b/src/main/java/com/fasterxml/aalto/out/ByteXmlWriter.java
@@ -794,17 +794,15 @@ protected int writeCDataContents(char[] cbuf, int offset, int len)
     {
         if (_surrogate != 0) {
             outputSurrogates(_surrogate, cbuf[offset]);
-//           reset the temporary surrogate storage
+            // reset the temporary surrogate storage
             _surrogate = 0;
             ++offset;
             --len;
         }
-        /* Unlike with writeCharacters() and fastWriteName(), let's not
-         * worry about split buffers here: this is unlikely to become
-         * performance bottleneck. This allows keeping it simple; and
-         * should it matter, we could start doing fast version here
-         * as well.
-         */
+        // Unlike with writeCharacters() and fastWriteName(), let's not
+        // worry about split buffers here: this is unlikely to become
+        // performance bottleneck. This allows keeping it simple; and
+        // should it matter, we could start doing fast version here as well.
         len += offset; // now marks the end
 
         main_loop: