Skip 0 length ranges when splitting

Expensify · Dec 18, 2024 · 8b86916 · 8b86916
1 parent 62d2771
commit 8b86916
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/android/src/main/java/com/expensify/livemarkdown/RangeSplitter.java b/android/src/main/java/com/expensify/livemarkdown/RangeSplitter.java
@@ -2,11 +2,14 @@
 
 import androidx.annotation.NonNull;
 
+import com.facebook.systrace.Systrace;
+
 import java.util.ArrayList;
 import java.util.List;
 
 public class RangeSplitter {
   public static ArrayList<MarkdownRange> splitRangesOnEmojis(@NonNull List<MarkdownRange> markdownRanges, @NonNull String type) {
+    Systrace.beginSection(0, "splitRangesOnEmojis");
     ArrayList<MarkdownRange> emojiRanges = new ArrayList<>();
     ArrayList<MarkdownRange> oldRanges = new ArrayList<>(markdownRanges);
     ArrayList<MarkdownRange> newRanges = new ArrayList<>();
@@ -41,13 +44,16 @@ public static ArrayList<MarkdownRange> splitRangesOnEmojis(@NonNull List<Markdow
           MarkdownRange newRange = new MarkdownRange(currentRange.getType(), currentStart, emojiStart - currentStart, currentRange.getDepth());
           currentRange = new MarkdownRange(currentRange.getType(), emojiEnd, currentEnd - emojiEnd, currentRange.getDepth());
 
-          newRanges.add(newRange);
+          if (newRange.getLength() > 0) {
+           newRanges.add(newRange);
+          }
         }
         j += 1;
       }
       newRanges.add(currentRange);
       i += 1;
     }
+    Systrace.endSection(0);
     return newRanges;
   }
 }