From: Andy Heninger <andy.heninger@gmail.com>
Date: Mon, 17 Jul 2017 21:22:53 +0000 (+0000)
Subject: ICU-12507 RBBI data loading, incorporate Markus's review comments.
X-Git-Tag: release-60-rc~220^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4a5b4e4b44d1710a3792863c7f677fa20a9d2709;p=icu

ICU-12507 RBBI data loading, incorporate Markus's review comments.

X-SVN-Rev: 40268
---

diff --git a/icu4c/source/common/rbbidata.cpp b/icu4c/source/common/rbbidata.cpp
index 32f3d32d508..9850be12dfc 100644
--- a/icu4c/source/common/rbbidata.cpp
+++ b/icu4c/source/common/rbbidata.cpp
@@ -66,13 +66,8 @@ RBBIDataWrapper::RBBIDataWrapper(UDataMemory* udm, UErrorCode &status) {
     fUDataMem = udm;
 }
 
-UBool RBBIDataWrapper::isDataVersionAcceptable(const uint8_t version[]) {
-    for (int i=0; i<UPRV_LENGTHOF(RBBI_DATA_FORMAT_VERSION); ++i) {
-        if (RBBI_DATA_FORMAT_VERSION[i] != version[i]) {
-            return false;
-        }
-    }
-    return true;
+UBool RBBIDataWrapper::isDataVersionAcceptable(const UVersionInfo version) {
+    return RBBI_DATA_FORMAT_VERSION[0] == version[0];
 }
 
 
diff --git a/icu4c/source/common/rbbidata.h b/icu4c/source/common/rbbidata.h
index f1608b54285..bd703f9029f 100644
--- a/icu4c/source/common/rbbidata.h
+++ b/icu4c/source/common/rbbidata.h
@@ -51,6 +51,7 @@ ubrk_swap(const UDataSwapper *ds,
 
 #include "unicode/uobject.h"
 #include "unicode/unistr.h"
+#include "unicode/uversion.h"
 #include "umutex.h"
 #include "utrie2.h"
 
@@ -64,12 +65,9 @@ static const uint8_t RBBI_DATA_FORMAT_VERSION[] = {4, 0, 0, 0};
  */
 struct RBBIDataHeader {
     uint32_t         fMagic;           /*  == 0xbla0                                               */
-    uint8_t          fFormatVersion[4]; /* Data Format.  Same as the value in struct UDataInfo      */
+    UVersionInfo     fFormatVersion;   /* Data Format.  Same as the value in struct UDataInfo      */
                                        /*   if there is one associated with this data.             */
                                        /*     (version originates in rbbi, is copied to UDataInfo) */
-                                       /*   For ICU 3.2 and earlier, this field was                */
-                                       /*       uint32_t  fVersion                                 */
-                                       /*   with a value of 1.                                     */
     uint32_t         fLength;          /*  Total length in bytes of this RBBI Data,                */
                                        /*      including all sections, not just the header.        */
     uint32_t         fCatCount;        /*  Number of character categories.                         */
@@ -155,7 +153,7 @@ public:
     RBBIDataWrapper(UDataMemory* udm, UErrorCode &status);
     ~RBBIDataWrapper();
 
-    static UBool          isDataVersionAcceptable(const uint8_t version[]);
+    static UBool          isDataVersionAcceptable(const UVersionInfo version);
 
     void                  init0();
     void                  init(const RBBIDataHeader *data, UErrorCode &status);