From 2506ddcd61d0c8fd7398c7cf378a09df4a69da38 Mon Sep 17 00:00:00 2001
From: Mark Davis <mark@macchiato.com>
Date: Thu, 30 Mar 2023 11:15:18 -0600
Subject: [PATCH] CLDR-16520 Break lines between cells for diffability (#2819)

---
 .../org/unicode/cldr/tool/TablePrinter.java   | 22 ++++++++++---------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/tools/cldr-code/src/main/java/org/unicode/cldr/tool/TablePrinter.java b/tools/cldr-code/src/main/java/org/unicode/cldr/tool/TablePrinter.java
index 4a99f283c1e..958f5411ea8 100644
--- a/tools/cldr-code/src/main/java/org/unicode/cldr/tool/TablePrinter.java
+++ b/tools/cldr-code/src/main/java/org/unicode/cldr/tool/TablePrinter.java
@@ -15,6 +15,8 @@
 
 public class TablePrinter {
 
+    public static final String LS = System.lineSeparator();
+
     public static void main(String[] args) {
         // quick test;
         TablePrinter tablePrinter = new TablePrinter()
@@ -350,7 +352,7 @@ private String tsvFormat(Comparable value) {
         if (value instanceof Number) {
             int debug = 0;
         }
-        String s = value.toString().replace("\n", " • ");
+        String s = value.toString().replace(LS, " • ");
         return BIDI.containsNone(s) ? s : RLE + s + PDF;
     }
 
@@ -372,7 +374,7 @@ public String toTableInternal(Comparable[][] sortedFlat) {
         if (tableAttributes != null) {
             result.append(' ').append(tableAttributes);
         }
-        result.append(">" + System.lineSeparator());
+        result.append(">" + LS);
 
         if (caption != null) {
             result.append("<caption>").append(caption).append("</caption>");
@@ -401,10 +403,10 @@ public String toTableInternal(Comparable[][] sortedFlat) {
                     }
                 }
                 if (divider) {
-                    result.append("\t<tr><td class='divider' colspan='" + visibleWidth + "'></td></tr>");
+                    result.append("  <tr><td class='divider' colspan='" + visibleWidth + "'></td></tr>" + LS);
                 }
             }
-            result.append("\t<tr>");
+            result.append("  <tr>");
             for (int j = 0; j < sortedFlat[i].length; ++j) {
                 int identical = findIdentical(sortedFlat, i, j);
                 if (identical == 0) continue;
@@ -412,7 +414,7 @@ public String toTableInternal(Comparable[][] sortedFlat) {
                     continue;
                 }
                 patternArgs[0] = sortedFlat[i][j];
-                result.append(columnsFlat[j].isHeader ? "<th" : "<td");
+                result.append(LS + "\t").append(columnsFlat[j].isHeader ? "<th" : "<td");
                 if (columnsFlat[j].cellAttributes != null) {
                     try {
                         result.append(' ').append(columnsFlat[j].cellAttributes.format(patternArgs));
@@ -440,7 +442,7 @@ public String toTableInternal(Comparable[][] sortedFlat) {
                 }
                 result.append(columnsFlat[j].isHeader ? "</th>" : "</td>");
             }
-            result.append("</tr>" + System.lineSeparator());
+            result.append("</tr>" + LS);
         }
         result.append("</table>");
         return result.toString();
@@ -455,24 +457,24 @@ private String format(Comparable comparable) {
         if (comparable == null) {
             return null;
         }
-        String s = comparable.toString().replace("\n", "<br>");
+        String s = comparable.toString().replace(LS, "<br>");
         return BIDI.containsNone(s) ? s : RLE + s + PDF;
     }
 
     private void showHeader(StringBuilder result) {
-        result.append("\t<tr>");
+        result.append("  <tr>");
         for (int j = 0; j < columnsFlat.length; ++j) {
             if (columnsFlat[j].hidden) {
                 continue;
             }
-            result.append("<th");
+            result.append(LS + "\t<th");
             if (columnsFlat[j].headerAttributes != null) {
                 result.append(' ').append(columnsFlat[j].headerAttributes);
             }
             result.append('>').append(columnsFlat[j].header).append("</th>");
 
         }
-        result.append("</tr>" + System.lineSeparator());
+        result.append("</tr>" + LS);
     }
 
     /**