From: Andy Heninger <andy.heninger@gmail.com>
Date: Sat, 17 Feb 2018 22:29:07 +0000 (+0000)
Subject: ICU-13569 Add clarifying comment to some RBBI state table handling code.
X-Git-Tag: release-61-rc~73
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a0c64552ab37caba06e76e76fc73e6318b0b4f5c;p=icu

ICU-13569 Add clarifying comment to some RBBI state table handling code.

X-SVN-Rev: 40948
---

diff --git a/icu4j/main/classes/core/src/com/ibm/icu/text/RBBITableBuilder.java b/icu4j/main/classes/core/src/com/ibm/icu/text/RBBITableBuilder.java
index 44b5e1a05fd..8954b672bbb 100644
--- a/icu4j/main/classes/core/src/com/ibm/icu/text/RBBITableBuilder.java
+++ b/icu4j/main/classes/core/src/com/ibm/icu/text/RBBITableBuilder.java
@@ -1054,7 +1054,10 @@ class RBBITableBuilder {
 
            //
            // Fill in the header fields.
-           //      Annoying because they really want to be ints, not shorts.
+           //      Note that NUMSTATES, ROWLEN and FLAGS are ints, not shorts.
+           //      ICU data created from Java is always big endian format, so
+           //      order the halves of the 32 bit fields into the short[] data accordingly.
+           //      TODO: ticket 13598 restructure so that ints are represented as ints directly.
            //
            // RBBIStateTable.fNumStates
            table[RBBIDataWrapper.NUMSTATES]   = (short)(numStates >>> 16);