소스 검색

dpkg: Reduce the scope of a if/else in setforce()

Guillem Jover 15 년 전
부모
커밋
10960fcbe9
1개의 변경된 파일9개의 추가작업 그리고 8개의 파일을 삭제
  1. 9 8
      src/main.c

+ 9 - 8
src/main.c

@@ -472,19 +472,20 @@ static void setforce(const struct cmdinfo *cip, const char *value) {
     l = comma ? (size_t)(comma - value) : strlen(value);
     for (fip=forceinfos; fip->name; fip++)
       if (!strncmp(fip->name,value,l) && strlen(fip->name)==l) break;
+
     if (!fip->name) {
       badusage(_("unknown force/refuse option `%.*s'"),
                (int)min(l, 250), value);
+    } else if (strcmp(fip->name, "all") == 0) {
+      for (fip = forceinfos; fip->name; fip++)
+        if (fip->opt)
+          *fip->opt = cip->arg;
+    } else if (fip->opt) {
+      *fip->opt = cip->arg;
     } else {
-      if (strcmp("all", fip->name) == 0) {
-	for (fip=forceinfos; fip->name; fip++)
-	  if (fip->opt)
-	    *fip->opt= cip->arg;
-      } else if (fip->opt)
-	*fip->opt= cip->arg;
-      else
-	warning(_("obsolete force/refuse option '%s'\n"), fip->name);
+      warning(_("obsolete force/refuse option '%s'\n"), fip->name);
     }
+
     if (!comma) break;
     value= ++comma;
   }