From: Ted Kremenek <kremenek@apple.com>
Date: Thu, 28 Jan 2010 00:17:51 +0000 (+0000)
Subject: Remove invalid conversion specifiers from format string checking.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f291976ecf537dc4f2d5537b1664d47ff4b94865;p=clang

Remove invalid conversion specifiers from format string checking.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@94707 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Analysis/PrintfFormatString.cpp b/lib/Analysis/PrintfFormatString.cpp
index b4dce9ffcc..d7b9a21a3d 100644
--- a/lib/Analysis/PrintfFormatString.cpp
+++ b/lib/Analysis/PrintfFormatString.cpp
@@ -191,11 +191,6 @@ static FormatSpecifierResult ParseFormatSpecifier(printf::FormatStringHandler &H
   // Finally, look for the conversion specifier.
   ConversionSpecifier::Kind cs;
   switch (*I) {
-    case 'D':
-    case 'O':
-    case 'U':
-    case 'C':
-    case 'S':
     default:
       H.HandleInvalidConversionSpecifier(I);
       return true;
diff --git a/lib/Sema/SemaChecking.cpp b/lib/Sema/SemaChecking.cpp
index 0a7b7f0911..97d30752c7 100644
--- a/lib/Sema/SemaChecking.cpp
+++ b/lib/Sema/SemaChecking.cpp
@@ -1160,9 +1160,6 @@ void Sema::CheckPrintfString(const StringLiteral *FExpr,
     case 'u':
     case 'x':
     case 'X':
-    case 'D':
-    case 'O':
-    case 'U':
     case 'e':
     case 'E':
     case 'f':
@@ -1172,8 +1169,6 @@ void Sema::CheckPrintfString(const StringLiteral *FExpr,
     case 'a':
     case 'A':
     case 'c':
-    case 'C':
-    case 'S':
     case 's':
     case 'p':
       ++numConversions;