Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Unified Diff: source/test/intltest/restest.cpp

Issue 845603002: Update ICU to 54.1 step 1 (Closed) Base URL: https://chromium.googlesource.com/chromium/deps/icu.git@master
Patch Set: remove unusued directories Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « source/test/intltest/reldatefmttest.cpp ('k') | source/test/intltest/restsnew.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: source/test/intltest/restest.cpp
diff --git a/source/test/intltest/restest.cpp b/source/test/intltest/restest.cpp
index 49f4de96648d63e5cf5e0032332d8ed8aab3fd83..530fbb7ee19b37e78561e73c92a8b18ce8ed55a0 100644
--- a/source/test/intltest/restest.cpp
+++ b/source/test/intltest/restest.cpp
@@ -1,6 +1,6 @@
/********************************************************************
* COPYRIGHT:
- * Copyright (c) 1997-2009, International Business Machines Corporation and
+ * Copyright (c) 1997-2014, International Business Machines Corporation and
* others. All Rights Reserved.
********************************************************************/
@@ -21,7 +21,6 @@
static const UChar kErrorUChars[] = { 0x45, 0x52, 0x52, 0x4f, 0x52, 0 };
static const int32_t kErrorLength = 5;
-static const int32_t kERROR_COUNT = -1234567;
//***************************************************************************************
@@ -613,11 +612,12 @@ ResourceBundleTest::TestGetLocaleByType(void)
status = U_ZERO_ERROR;
continue;
}
-
+
locale = res.getLocale(ULOC_REQUESTED_LOCALE, status);
- if(locale != Locale::getDefault()) {
+ if(U_SUCCESS(status) && locale != Locale::getDefault()) {
err("Expected requested locale to be %s. Got %s\n", test[i].requestedLocale, locale.getName());
}
+ status = U_ZERO_ERROR;
locale = res.getLocale(ULOC_VALID_LOCALE, status);
if(strcmp(locale.getName(), test[i].validLocale) != 0) {
err("Expected valid locale to be %s. Got %s\n", test[i].requestedLocale, locale.getName());
@@ -628,6 +628,3 @@ ResourceBundleTest::TestGetLocaleByType(void)
}
}
}
-
-//eof
-
« no previous file with comments | « source/test/intltest/reldatefmttest.cpp ('k') | source/test/intltest/restsnew.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698