ICU-461.12.tar.gz

[apple/icu.git] / icuSources / test / intltest / fldset.cpp

diff --git a/icuSources/test/intltest/fldset.cpp b/icuSources/test/intltest/fldset.cpp
index aa3aa28479629ea9325f224b2f5767d4ef2dfc95..b289ad60169308d5818588b3d02a85f68af85a7c 100644 (file)
--- a/icuSources/test/intltest/fldset.cpp
+++ b/icuSources/test/intltest/fldset.cpp
@@ -1,6 +1,6 @@
 /*
 ************************************************************************
-* Copyright (c) 2007, International Business Machines
+* Copyright (c) 2007-2008, International Business Machines
 * Corporation and others.  All Rights Reserved.
 ************************************************************************
 */
@@ -189,7 +189,11 @@ int32_t FieldsSet::get(int32_t field) const {
 
 int32_t FieldsSet::handleParseName(const FieldsSet* /* inheritFrom */, const UnicodeString& name, const UnicodeString& /* substr*/ , UErrorCode& status) {
     if(fEnum > -1) {
-        return udbg_enumByString(fEnum, name);
+        int32_t which = udbg_enumByString(fEnum, name);
+        if(which == UDBG_INVALID_ENUM) {
+            status = U_UNSUPPORTED_ERROR;
+        }
+        return which;
     } else {
         status = U_UNSUPPORTED_ERROR;
         return -1;
@@ -254,7 +258,6 @@ void CalendarFieldsSet::setOnCalendar(Calendar *cal, UErrorCode& /*status*/) con
     for (int i=0; i<UDAT_FIELD_COUNT; i++) {
         if (isSet((UCalendarDateFields)i)) {
             int32_t value = get((UCalendarDateFields)i);
-            //fprintf(stderr, "Setting: %s#%d=%d\n",udbg_enumName(UDBG_UCalendarDateFields,i),i,value);            
             cal->set((UCalendarDateFields)i, value);
         }
     }
@@ -266,8 +269,9 @@ void CalendarFieldsSet::setOnCalendar(Calendar *cal, UErrorCode& /*status*/) con
 UBool CalendarFieldsSet::matches(Calendar *cal, CalendarFieldsSet &diffSet,
         UErrorCode& status) const {
     UBool match = TRUE;
-    if (U_FAILURE(status))
+    if (U_FAILURE(status)) {
         return FALSE;
+    }
     for (int i=0; i<UDAT_FIELD_COUNT; i++) {
         if (isSet((UCalendarDateFields)i)) {
             int32_t calVal = cal->get((UCalendarDateFields)i, status);
@@ -284,10 +288,13 @@ UBool CalendarFieldsSet::matches(Calendar *cal, CalendarFieldsSet &diffSet,
 }
 
 
-enum {
-    DTS_DATE = 0,
-    DTS_TIME,
-    DTS_COUNT
+/**
+ * DateTimeStyleSet has two 'fields' -- date, and time.
+ */
+enum DateTimeStyleSetFields {
+    DTS_DATE = 0,  /** Field one: the date (long, medium, short, etc). */
+    DTS_TIME,      /** Field two: the time (long, medium, short, etc). */
+    DTS_COUNT      /** The number of fields */
 };
 
 /**
@@ -295,7 +302,6 @@ enum {
  * */
 DateTimeStyleSet::DateTimeStyleSet() :
     FieldsSet(DTS_COUNT) {
-    
 }
 
 DateTimeStyleSet::~DateTimeStyleSet() {
@@ -320,8 +326,6 @@ UDateFormatStyle DateTimeStyleSet::getTimeStyle() const {
 }
 
 void DateTimeStyleSet::handleParseValue(const FieldsSet* inheritFrom, int32_t field, const UnicodeString& substr, UErrorCode& status) {
-//    int32_t value = udbg_enumByString(UDBG_UDateFormatStyle, substr);
-//    fprintf(stderr, " HPV: %d -> %d\n", field, value);
     UnicodeString kRELATIVE_("RELATIVE_");
     if(substr.startsWith(kRELATIVE_)) {
         UnicodeString relativeas(substr,kRELATIVE_.length());