]> granicus.if.org Git - clang/commitdiff
Remove invalid conversion specifiers from format string checking.
authorTed Kremenek <kremenek@apple.com>
Thu, 28 Jan 2010 00:17:51 +0000 (00:17 +0000)
committerTed Kremenek <kremenek@apple.com>
Thu, 28 Jan 2010 00:17:51 +0000 (00:17 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@94707 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Analysis/PrintfFormatString.cpp
lib/Sema/SemaChecking.cpp

index b4dce9ffcc5b1aad13793f72ab7c6138f3c68098..d7b9a21a3dc84258b88699649a7e950db988235a 100644 (file)
@@ -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;
index 0a7b7f0911f2cc13309e36984eca98fb345b76a1..97d30752c7b302110171f1ce8e927da7b1e94cd6 100644 (file)
@@ -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;