style(9) - Prefer a lack of case '?' fallthrough for getopt

* This also coincides with the example in getopt(3)

Discussed-with: swildner@

diff --git a/share/man/man9/style.9 b/share/man/man9/style.9
index c06e70a..aa28138 100644 (file)
--- a/share/man/man9/style.9
+++ b/share/man/man9/style.9
@@ -339,9 +339,7 @@ Elements in a
 .Ic switch
 statement that cascade should have a
 .Li FALLTHROUGH
-comment, unless they contain no code of their own, as in the
-.Ic case '?'
-element in the example below.
+comment, unless they contain no code of their own.
 Numerical arguments should be checked for accuracy.
 Code that cannot be reached should have a
 .Li NOTREACHED
@@ -363,7 +361,6 @@ comment.
                                usage();
                        }
                        break;
-               case '?':
                default:
                        usage();
                        /* NOTREACHED */

diff --git a/sys/platform/vkernel/platform/init.c b/sys/platform/vkernel/platform/init.c
index ea500ec..7e49187 100644 (file)
--- a/sys/platform/vkernel/platform/init.c
+++ b/sys/platform/vkernel/platform/init.c
@@ -313,7 +313,6 @@ main(int ac, char **av)
                case 'h':
                        usage_help(true);
                        break;
-               case '?':
                default:
                        usage_help(false);
                }

diff --git a/sys/platform/vkernel64/platform/init.c b/sys/platform/vkernel64/platform/init.c
index bcd8d64..7218859 100644 (file)
--- a/sys/platform/vkernel64/platform/init.c
+++ b/sys/platform/vkernel64/platform/init.c
@@ -311,7 +311,6 @@ main(int ac, char **av)
                case 'h':
                        usage_help(true);
                        break;
-               case '?':
                default:
                        usage_help(false);
                }