Skip to content

Commit

Permalink
nfc-read-forum-tag3: remove redundant error
Browse files Browse the repository at this point in the history
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 (nfc-tools#1 of 1): Missing break in switch (MISSING_BREAK)
  unterminated_case: This case (value 63) is not terminated by a 'break' statement.
  • Loading branch information
doegox committed Sep 22, 2013
1 parent 70048a0 commit feb5f37
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions utils/nfc-read-forum-tag3.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down

0 comments on commit feb5f37

Please sign in to comment.