Condense "shut up gcc" switch-cases to just default no action case
authorPanu Matilainen <pmatilai@redhat.com>
Wed, 4 Mar 2009 13:12:02 +0000 (15:12 +0200)
committerPanu Matilainen <pmatilai@redhat.com>
Wed, 4 Mar 2009 13:12:02 +0000 (15:12 +0200)
lib/package.c
lib/rpmchecksig.c
lib/signature.c

index 4d5e087..9a56e49 100644 (file)
@@ -779,15 +779,7 @@ rpmRC rpmReadPackageFile(rpmts ts, FD_t fd, const char * fn, Header * hdrp)
 
        fdStealDigest(fd, dig);
        break;
-    /* shut up gcc */
-    case RPMSIGTAG_LEMD5_1:
-    case RPMSIGTAG_LEMD5_2:
-    case RPMSIGTAG_BADSHA1_1:
-    case RPMSIGTAG_BADSHA1_2:
-    case RPMSIGTAG_SIZE:
-    case RPMSIGTAG_LONGSIZE:
-    case RPMSIGTAG_PAYLOADSIZE:
-    case RPMSIGTAG_LONGARCHIVESIZE:
+    default:
        break;
     }
 
index f80222d..485be84 100644 (file)
@@ -282,17 +282,7 @@ static int rpmReSign(rpmts ts, QVA_t qva, ARGV_const_t argv)
            case RPMSIGTAG_PGP:
                xx = headerDel(sigh, RPMSIGTAG_RSA);
                break;
-           /* shut up gcc */
-           case RPMSIGTAG_SHA1:
-           case RPMSIGTAG_MD5:
-           case RPMSIGTAG_LEMD5_1:
-           case RPMSIGTAG_LEMD5_2:
-           case RPMSIGTAG_BADSHA1_1:
-           case RPMSIGTAG_BADSHA1_2:
-           case RPMSIGTAG_PAYLOADSIZE:
-           case RPMSIGTAG_LONGARCHIVESIZE:
-           case RPMSIGTAG_SIZE:
-           case RPMSIGTAG_LONGSIZE:
+           default:
                break;
            }
 
index fba0027..545ab4d 100644 (file)
@@ -598,11 +598,6 @@ static int makeGPGSignature(const char * file, rpmSigTag * sigTagp,
     sigp = &dig->signature;
 
     switch (*sigTagp) {
-    case RPMSIGTAG_SIZE:
-    case RPMSIGTAG_LONGSIZE:
-    case RPMSIGTAG_MD5:
-    case RPMSIGTAG_SHA1:
-       break;
     case RPMSIGTAG_GPG:
        /* XXX check MD5 hash too? */
        if (sigp->pubkey_algo == PGPPUBKEYALGO_RSA)
@@ -622,13 +617,7 @@ static int makeGPGSignature(const char * file, rpmSigTag * sigTagp,
        if (sigp->pubkey_algo == PGPPUBKEYALGO_DSA)
            *sigTagp = RPMSIGTAG_DSA;
        break;
-    /* shut up gcc */
-    case RPMSIGTAG_LEMD5_1:
-    case RPMSIGTAG_LEMD5_2:
-    case RPMSIGTAG_BADSHA1_1:
-    case RPMSIGTAG_BADSHA1_2:
-    case RPMSIGTAG_PAYLOADSIZE:
-    case RPMSIGTAG_LONGARCHIVESIZE:
+    default:
        break;
     }
 
@@ -662,14 +651,6 @@ static int makeHDRSignature(Header sigh, const char * file, rpmSigTag sigTag,
     int ret = -1;      /* assume failure. */
 
     switch (sigTag) {
-    case RPMSIGTAG_SIZE:
-    case RPMSIGTAG_LONGSIZE:
-    case RPMSIGTAG_MD5:
-    case RPMSIGTAG_PGP5:       /* XXX legacy */
-    case RPMSIGTAG_PGP:
-    case RPMSIGTAG_GPG:
-       goto exit;
-       break;
     case RPMSIGTAG_SHA1:
        fd = Fopen(file, "r.fdio");
        if (fd == NULL || Ferror(fd))
@@ -744,13 +725,8 @@ static int makeHDRSignature(Header sigh, const char * file, rpmSigTag sigTag,
            goto exit;
        ret = 0;
        break;
-    /* shut up gcc */
-    case RPMSIGTAG_LEMD5_1:
-    case RPMSIGTAG_LEMD5_2:
-    case RPMSIGTAG_BADSHA1_1:
-    case RPMSIGTAG_BADSHA1_2:
-    case RPMSIGTAG_PAYLOADSIZE:
-    case RPMSIGTAG_LONGARCHIVESIZE:
+    default:
+       goto exit;
        break;
     }
 
@@ -824,13 +800,7 @@ int rpmAddSignature(Header sigh, const char * file, rpmSigTag sigTag,
     case RPMSIGTAG_SHA1:
        ret = makeHDRSignature(sigh, file, sigTag, passPhrase);
        break;
-    /* shut up gcc */
-    case RPMSIGTAG_LEMD5_1:
-    case RPMSIGTAG_LEMD5_2:
-    case RPMSIGTAG_BADSHA1_1:
-    case RPMSIGTAG_BADSHA1_2:
-    case RPMSIGTAG_PAYLOADSIZE:
-    case RPMSIGTAG_LONGARCHIVESIZE:
+    default:
        break;
     }
     free(pkt);