From feb5f37aa3ea27fa94ae0d74189f3e429900de0f Mon Sep 17 00:00:00 2001 From: Philippe Teuwen Date: Sun, 22 Sep 2013 02:14:08 +0200 Subject: [PATCH] nfc-read-forum-tag3: remove redundant error switch case was redundant as getopt was already telling the issue: nfc-read-forum-tag3: option requires an argument -- 'o' Option -o requires an argument. This fixes also a problem reported by Coverity about missing break: CID 1090330 (#1 of 1): Missing break in switch (MISSING_BREAK) unterminated_case: This case (value 63) is not terminated by a 'break' statement. --- utils/nfc-read-forum-tag3.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/utils/nfc-read-forum-tag3.c b/utils/nfc-read-forum-tag3.c index 934c8d2..9d8a06d 100644 --- a/utils/nfc-read-forum-tag3.c +++ b/utils/nfc-read-forum-tag3.c @@ -178,9 +178,6 @@ main(int argc, char *argv[]) case 'o': ndef_output = optarg; break; - case '?': - if (optopt == 'o') - fprintf(stderr, "Option -%c requires an argument.\n", optopt); default: print_usage(argv[0]); exit(EXIT_FAILURE);