test_escape: Fix handling of short options in getopt_long()

michaelpq · michaelpq · commit a68a7594ca3a · 2025-02-19T09:45:54.000+09:00
This addresses two errors in the module, based on the set of options supported: - '-c', for --conninfo, was not listed. - '-f', for --force-unsupported, was not listed. While on it, these are now listed in an alphabetical order. Author: Japin Li Discussion: https://postgr.es/m/ME0P300MB04451FB20CE0346A59C25CADB6FA2@ME0P300MB0445.AUSP300.PROD.OUTLOOK.COM Backpatch-through: 13
diff --git a/src/test/modules/test_escape/test_escape.c b/src/test/modules/test_escape/test_escape.c
@@ -824,7 +824,7 @@ main(int argc, char *argv[])
 		{NULL, 0, NULL, 0},
 	};
 
-	while ((c = getopt_long(argc, argv, "vqh", long_options, &option_index)) != -1)
+	while ((c = getopt_long(argc, argv, "c:fhqv", long_options, &option_index)) != -1)
 	{
 		switch (c)
 		{

Original file line number	Diff line number	Diff line change
`@@ -824,7 +824,7 @@ main(int argc, char *argv[])`
`824`	`824`	`{NULL, 0, NULL, 0},`
`825`	`825`	`};`
`826`	`826`
`827`		`- while ((c = getopt_long(argc, argv, "vqh", long_options, &option_index)) != -1)`
	`827`	`+ while ((c = getopt_long(argc, argv, "c:fhqv", long_options, &option_index)) != -1)`
`828`	`828`	`{`
`829`	`829`	`switch (c)`
`830`	`830`	`{`