From 3b1ec5032c640c360bb8f796db51fa23b85547e2 Mon Sep 17 00:00:00 2001
From: Victor Chang <vichang@google.com>
Date: Tue, 12 Jan 2021 11:59:13 +0000
Subject: [PATCH] ICU-21116 Fix ListFormatterTest on Android
Android ignores assert statement by default.
The test also failed when running with ant and on OpenJDK runtime
on release-67-1. But no longer fails on master branch.
---
.../src/com/ibm/icu/impl/FormattedValueStringBuilderImpl.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/icu4j/main/classes/core/src/com/ibm/icu/impl/FormattedValueStringBuilderImpl.java b/icu4j/main/classes/core/src/com/ibm/icu/impl/FormattedValueStringBuilderImpl.java
index 0c82d263254..43f0c4df176 100644
--- a/icu4j/main/classes/core/src/com/ibm/icu/impl/FormattedValueStringBuilderImpl.java
+++ b/icu4j/main/classes/core/src/com/ibm/icu/impl/FormattedValueStringBuilderImpl.java
@@ -144,7 +144,8 @@ public class FormattedValueStringBuilderImpl {
int end = i - self.zero;
// Handle span fields; don't trim them
if (currField instanceof SpanFieldPlaceholder) {
- assert handleSpan(currField, cfpos, fieldStart, end);
+ boolean handleResult = handleSpan(currField, cfpos, fieldStart, end);
+ assert handleResult;
return true;
}
// Grouping separators can be whitespace; don't throw them out!
--
2.40.0