Home | History | Annotate | Line # | Download | only in lint1
msg_338.c revision 1.6
      1  1.6  rillig /*	$NetBSD: msg_338.c,v 1.6 2021/10/09 13:57:55 rillig Exp $	*/
      2  1.1  rillig # 3 "msg_338.c"
      3  1.1  rillig 
      4  1.1  rillig // Test for message: option '%c' should be handled in the switch [338]
      5  1.1  rillig 
      6  1.1  rillig int getopt(int, char *const *, const char *);
      7  1.1  rillig extern char *optarg;
      8  1.1  rillig 
      9  1.1  rillig int
     10  1.1  rillig main(int argc, char **argv)
     11  1.1  rillig {
     12  1.1  rillig 	int o;
     13  1.1  rillig 
     14  1.3  rillig 	while ((o = getopt(argc, argv, "a:bc:d")) != -1) { /* expect: 338 *//* expect: 338 */
     15  1.1  rillig 		switch (o) {
     16  1.1  rillig 		case 'a':
     17  1.1  rillig 			break;
     18  1.1  rillig 		case 'b':
     19  1.1  rillig 			/*
     20  1.1  rillig 			 * The following while loop must not finish the check
     21  1.1  rillig 			 * for the getopt options.
     22  1.1  rillig 			 */
     23  1.1  rillig 			while (optarg[0] != '\0')
     24  1.1  rillig 				optarg++;
     25  1.1  rillig 			break;
     26  1.1  rillig 		case 'e':	/* expect: option 'e' should be listed */
     27  1.1  rillig 			break;
     28  1.1  rillig 		case 'f':	/* expect: option 'f' should be listed */
     29  1.1  rillig 			/*
     30  1.1  rillig 			 * The case labels in nested switch statements are
     31  1.1  rillig 			 * ignored by the check for getopt options.
     32  1.1  rillig 			 */
     33  1.1  rillig 			switch (optarg[0]) {
     34  1.1  rillig 			case 'X':
     35  1.1  rillig 				break;
     36  1.1  rillig 			}
     37  1.1  rillig 			break;
     38  1.1  rillig 		case '?':
     39  1.1  rillig 		default:
     40  1.1  rillig 			break;
     41  1.1  rillig 		}
     42  1.1  rillig 	}
     43  1.1  rillig 
     44  1.2  rillig 	/* A while loop that is not related to getopt is simply skipped. */
     45  1.2  rillig 	while (o != 0) {
     46  1.2  rillig 		switch (o) {
     47  1.2  rillig 		case '?':
     48  1.2  rillig 			o = ':';
     49  1.2  rillig 		}
     50  1.2  rillig 	}
     51  1.2  rillig 
     52  1.1  rillig 	return 0;
     53  1.1  rillig }
     54  1.4  rillig 
     55  1.4  rillig void usage(void);
     56  1.4  rillig 
     57  1.5  rillig /*
     58  1.5  rillig  * Before ckgetopt.c 1.11 from 2021-08-23, lint wrongly warned about a
     59  1.5  rillig  * missing '?' in the switch statement, even though it was there.
     60  1.5  rillig  *
     61  1.5  rillig  * Seen in usr.bin/ftp/main.c 1.127 from 2020-07-18.
     62  1.5  rillig  */
     63  1.4  rillig int
     64  1.4  rillig question_option(int argc, char **argv)
     65  1.4  rillig {
     66  1.4  rillig 	int c;
     67  1.4  rillig 
     68  1.4  rillig 	while ((c = getopt(argc, argv, "?x")) != -1) {
     69  1.4  rillig 		switch (c) {
     70  1.4  rillig 		case 'x':
     71  1.4  rillig 			break;
     72  1.4  rillig 		case '?':
     73  1.4  rillig 			usage();
     74  1.4  rillig 			return 0;
     75  1.4  rillig 		default:
     76  1.4  rillig 			usage();
     77  1.4  rillig 			return 1;
     78  1.4  rillig 		}
     79  1.4  rillig 	}
     80  1.4  rillig 	return 0;
     81  1.4  rillig }
     82  1.6  rillig 
     83  1.6  rillig /*
     84  1.6  rillig  * If the first character of the options string is ':', getopt does not print
     85  1.6  rillig  * its own error messages. Getopt returns ':' if an option is missing its
     86  1.6  rillig  * argument; that is handled by the 'default:' already.
     87  1.6  rillig  */
     88  1.6  rillig int
     89  1.6  rillig suppress_errors(int argc, char **argv)
     90  1.6  rillig {
     91  1.6  rillig 	int c;
     92  1.6  rillig 
     93  1.6  rillig 	/* expect+1: warning: option 'o' should be handled in the switch [338] */
     94  1.6  rillig 	while ((c = getopt(argc, argv, ":b:o")) != -1) {
     95  1.6  rillig 		switch (c) {
     96  1.6  rillig 		case 'b':
     97  1.6  rillig 			return 'b';
     98  1.6  rillig 		default:
     99  1.6  rillig 			usage();
    100  1.6  rillig 		}
    101  1.6  rillig 	}
    102  1.6  rillig 	return 0;
    103  1.6  rillig }
    104  1.6  rillig 
    105  1.6  rillig /*
    106  1.6  rillig  * If the first character of the options string is ':', getopt returns ':'
    107  1.6  rillig  * if an option is missing its argument. This condition can be handled
    108  1.6  rillig  * separately from '?', which getopt returns for unknown options.
    109  1.6  rillig  */
    110  1.6  rillig int
    111  1.6  rillig missing_argument(int argc, char **argv)
    112  1.6  rillig {
    113  1.6  rillig 	int c;
    114  1.6  rillig 
    115  1.6  rillig 	/* expect+1: warning: option 'o' should be handled in the switch [338] */
    116  1.6  rillig 	while ((c = getopt(argc, argv, ":b:o")) != -1) {
    117  1.6  rillig 		switch (c) {
    118  1.6  rillig 		case 'b':
    119  1.6  rillig 			return 'b';
    120  1.6  rillig 		case ':':
    121  1.6  rillig 			return 'm';
    122  1.6  rillig 		default:
    123  1.6  rillig 			usage();
    124  1.6  rillig 		}
    125  1.6  rillig 	}
    126  1.6  rillig 	return 0;
    127  1.6  rillig }
    128  1.6  rillig 
    129  1.6  rillig /*
    130  1.6  rillig  * Getopt only returns ':' if ':' is the first character in the options
    131  1.6  rillig  * string. Everywhere else, a ':' marks the preceding option as having a
    132  1.6  rillig  * required argument. In theory, if the options string contained "a::x",
    133  1.6  rillig  * that could be interpreted as '-a argument', followed by '-:' and '-x',
    134  1.6  rillig  * but nobody does that.
    135  1.6  rillig  */
    136  1.6  rillig int
    137  1.6  rillig unreachable_colon(int argc, char **argv)
    138  1.6  rillig {
    139  1.6  rillig 	int c;
    140  1.6  rillig 
    141  1.6  rillig 	/* expect+1: warning: option 'b' should be handled in the switch [338] */
    142  1.6  rillig 	while ((c = getopt(argc, argv, "b:")) != -1) {
    143  1.6  rillig 		switch (c) {
    144  1.6  rillig 		/* TODO: expect+1: warning: option ':' should be listed in the options string [339] */
    145  1.6  rillig 		case ':':
    146  1.6  rillig 			return 'm';
    147  1.6  rillig 		default:
    148  1.6  rillig 			usage();
    149  1.6  rillig 		}
    150  1.6  rillig 	}
    151  1.6  rillig 	return 0;
    152  1.6  rillig }
    153