diff --git a/icu4c/source/test/intltest/apicoll.cpp b/icu4c/source/test/intltest/apicoll.cpp
index 177c3a325ce..5db7ed1422e 100644
--- a/icu4c/source/test/intltest/apicoll.cpp
+++ b/icu4c/source/test/intltest/apicoll.cpp
@@ -1398,7 +1398,6 @@ void CollationAPITest::TestGetLocale() {
   UErrorCode status = U_ZERO_ERROR;
   const char *rules = "&a<x<y<z";
   UChar rlz[256] = {0};
-  uint32_t rlzLen = u_unescape(rules, rlz, 256);
 
   Collator *coll = NULL;
   Locale locale;
@@ -1416,8 +1415,10 @@ void CollationAPITest::TestGetLocale() {
     { "fr_FR_NONEXISTANT", "fr_FR", "fr" }
   };
 
+  u_unescape(rules, rlz, 256);
+
   /* test opening collators for different locales */
-  for(i = 0; i<sizeof(testStruct)/sizeof(testStruct[0]); i++) {
+  for(i = 0; i<(int32_t)(sizeof(testStruct)/sizeof(testStruct[0])); i++) {
     status = U_ZERO_ERROR;
     coll = Collator::createInstance(testStruct[i].requestedLocale, status);
     if(U_FAILURE(status)) {
diff --git a/icu4c/source/test/intltest/biditst.cpp b/icu4c/source/test/intltest/biditst.cpp
index 027e7a83f64..3dec07c4c6b 100644
--- a/icu4c/source/test/intltest/biditst.cpp
+++ b/icu4c/source/test/intltest/biditst.cpp
@@ -689,7 +689,7 @@ static int countRoundtrips=0, countNonRoundtrips=0;
 void BiDiTest::TestInverseBiDi() {
     BiDi bidi;
     UErrorCode errorCode;
-    int i;
+    uint32_t i;
 
     logln("inverse BiDi: testInverseBiDi(L) with %u test cases ---\n", LENGTHOF(testCases));
     for(i=0; i<LENGTHOF(testCases); ++i) {
diff --git a/icu4c/source/test/intltest/canittst.cpp b/icu4c/source/test/intltest/canittst.cpp
index 9668fd566c9..5ba5775fea4 100644
--- a/icu4c/source/test/intltest/canittst.cpp
+++ b/icu4c/source/test/intltest/canittst.cpp
@@ -36,12 +36,12 @@ void CanonicalIteratorTest::runIndexedTest(int32_t index, UBool exec,
 
 /**
  * Convert Java-style strings with \u Unicode escapes into UnicodeString objects
- */
 static UnicodeString str(const char *input)
 {
     UnicodeString str(input, ""); // Invariant conversion
     return str.unescape();
 }
+ */
 
 
 CanonicalIteratorTest::CanonicalIteratorTest() :
@@ -61,7 +61,6 @@ CanonicalIteratorTest::~CanonicalIteratorTest()
 
 void CanonicalIteratorTest::TestExhaustive() {
     UErrorCode status = U_ZERO_ERROR;
-    int counter = 0;
     CanonicalIterator it("", status);
     UChar32 i = 0;
     UnicodeString s, decomp, comp;
@@ -159,7 +158,6 @@ void CanonicalIteratorTest::TestBasic() {
         //logln("Results for: " + name.transliterate(testArray[i]));
         UnicodeString testStr = CharsToUnicodeString(testArray[i][0]);
         it.setSource(testStr, status);
-        int counter = 0;
         set->removeAll();
         while (TRUE) {
             //UnicodeString *result = new UnicodeString(it.next());
diff --git a/icu4c/source/test/intltest/tstnorm.cpp b/icu4c/source/test/intltest/tstnorm.cpp
index c6f93386591..08ed169fea5 100644
--- a/icu4c/source/test/intltest/tstnorm.cpp
+++ b/icu4c/source/test/intltest/tstnorm.cpp
@@ -863,7 +863,7 @@ void BasicNormalizerTest::TestConcatenate() {
     int32_t i;
 
     /* test concatenation */
-    for(i=0; i<sizeof(cases)/sizeof(cases[0]); ++i) {
+    for(i=0; i<(int32_t)(sizeof(cases)/sizeof(cases[0])); ++i) {
         switch(*cases[i][0]) {
         case 'C': mode=UNORM_NFC; break;
         case 'D': mode=UNORM_NFD; break;