diff --git a/icu4c/source/test/cintltst/callcoll.c b/icu4c/source/test/cintltst/callcoll.c
index 9f00f2bed60..1c63d096011 100644
--- a/icu4c/source/test/cintltst/callcoll.c
+++ b/icu4c/source/test/cintltst/callcoll.c
@@ -1166,7 +1166,7 @@ static char *safeGetICUDataDirectory() {
 
 static void
 TestFCDCrash(void) {
-    static char *test[] = {
+    static const char *test[] = {
     "Gr\\u00F6\\u00DFe",
     "Grossist"
     };
diff --git a/icu4c/source/test/cintltst/cloctst.c b/icu4c/source/test/cintltst/cloctst.c
index ea9bcee20a8..6608a383256 100644
--- a/icu4c/source/test/cintltst/cloctst.c
+++ b/icu4c/source/test/cintltst/cloctst.c
@@ -2789,9 +2789,6 @@ static void TestDisplayKeywordValues(void){
     {
         /* Test non existent keywords */
         UErrorCode status = U_ZERO_ERROR;
-        const char* keyword =NULL;
-        int32_t keywordLen = 0;
-        int32_t keywordCount = 0;
         const char* localeID = "es";
         const char* displayLocale = "de";
         UChar *displayKeywordValue = NULL;
diff --git a/icu4c/source/test/cintltst/cnmdptst.c b/icu4c/source/test/cintltst/cnmdptst.c
index 5be7bf0a2e4..20607bf414d 100644
--- a/icu4c/source/test/cintltst/cnmdptst.c
+++ b/icu4c/source/test/cintltst/cnmdptst.c
@@ -469,7 +469,7 @@ static void TestCurrencyPreEuro(void)
     for(i=0; i < 19; i++)
     {
         char curID[256] = {0};
-        int len = uloc_canonicalize(locale[i], curID, 256, &status);
+        uloc_canonicalize(locale[i], curID, 256, &status);
         if(U_FAILURE(status)){
             log_err("Could not canonicalize %s. Error: %s \n", locale[i], u_errorName(status));
             continue;
diff --git a/icu4c/source/test/intltest/numfmtst.cpp b/icu4c/source/test/intltest/numfmtst.cpp
index e9dcdd806a9..710cab650fa 100644
--- a/icu4c/source/test/intltest/numfmtst.cpp
+++ b/icu4c/source/test/intltest/numfmtst.cpp
@@ -629,14 +629,14 @@ NumberFormatTest::TestCurrency(void)
     if (U_FAILURE(status))
         errln((UnicodeString)"FAIL: Status " + (int32_t)status);
     
-    for(int i=0; i < (sizeof(testCases)/sizeof(testCases[i])); i++){
+    for(int i=0; i < (int)(sizeof(testCases)/sizeof(testCases[i])); i++){
         status = U_ZERO_ERROR;
         const char *localeID = testCases[i][0];
         UnicodeString expected(testCases[i][1]);
         expected = expected.unescape();
         s.truncate(0);
         char loc[256]={0};
-        int len = uloc_canonicalize(localeID, loc, 256, &status);
+        uloc_canonicalize(localeID, loc, 256, &status);
         currencyFmt = NumberFormat::createCurrencyInstance(Locale(loc), status);
         if(U_FAILURE(status)){
             errln("Could not create currency formatter for locale %s",localeID);
diff --git a/icu4c/source/test/intltest/numrgts.cpp b/icu4c/source/test/intltest/numrgts.cpp
index 7f8387b18e9..7db8555b377 100644
--- a/icu4c/source/test/intltest/numrgts.cpp
+++ b/icu4c/source/test/intltest/numrgts.cpp
@@ -922,7 +922,7 @@ void NumberFormatRegressionTest::Test4071014 (void)
 
     UErrorCode status = U_ZERO_ERROR;
     char loc[256]={0};
-    int len = uloc_canonicalize("de_DE_PREEURO", loc, 256, &status);
+    uloc_canonicalize("de_DE_PREEURO", loc, 256, &status);
     formatter = NumberFormat::createInstance(Locale(loc), status);
     failure(status, "NumberFormat::createNumberInstance");
     tempString.remove();
@@ -985,7 +985,7 @@ void NumberFormatRegressionTest::Test4071859 (void)
 
     UErrorCode status = U_ZERO_ERROR;
     char loc[256]={0};
-    int len = uloc_canonicalize("it_IT_PREEURO", loc, 256, &status);
+    uloc_canonicalize("it_IT_PREEURO", loc, 256, &status);
     formatter = NumberFormat::createInstance(Locale(loc), status);
     failure(status, "NumberFormat::createNumberInstance");
     tempString = formatter->format (-5789.9876, tempString);