Switched to using codePoints in toHtmlEntities

lib/src/main/java/rife/render/EncodeHtmlEntities.java

@@ -44,14 +44,19 @@ public class EncodeHtmlEntities implements ValueRenderer {
      * @param text the String to convert.
      * @return the converted string.
      */
+    @SuppressWarnings("PMD.AvoidReassigningLoopVariables")
     public static String toHtmlEntities(String text) {
-        var buff = new StringBuilder(text.length() * 6);
+        // https://stackoverflow.com/a/6766497/8356718
-
+        var sb = new StringBuilder(text.length() * 6);
         for (var i = 0; i < text.length(); i++) {
-            buff.append("&#").append((int) text.charAt(i)).append(';');
+            var codePoint = text.codePointAt(i);
+            // Skip over the second char in a surrogate pair
+            if (codePoint > 0xffff) {
+                i++;
+            }
+            sb.append(String.format("&#%s;", codePoint));
         }
-
+        return sb.toString();
-        return buff.toString();
     }
 
     /**

lib/src/main/java/rife/render/RenderUtils.java

@@ -33,7 +33,7 @@ public final class RenderUtils {
      *
      * @param template the template
      * @param valueId  the ID of the value to fetch
-     * @return The fetched value.
+     * @return the fetched value.
      */
     public static String fetchValue(Template template, String valueId) {
         Object value = null;

lib/src/main/java/rife/render/Rot13.java

@@ -37,8 +37,8 @@ public class Rot13 implements ValueRenderer {
     /**
      * Translates a String to/from ROT13.
      *
-     * @param src The source String.
+     * @param src the source String.
-     * @return The translated String.
+     * @return the translated String.
      */
     public static String rot13(String src) {
         if (src == null || src.isEmpty()) {