Date: Thu, 6 Jan 2011 08:33:48 +0000 (UTC) From: Edward Tomasz Napierala <trasz@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r217048 - head/share/man/man9 Message-ID: <201101060833.p068Xmrj008299@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: trasz Date: Thu Jan 6 08:33:48 2011 New Revision: 217048 URL: http://svn.freebsd.org/changeset/base/217048 Log: Get rid of bad advice regarding /* NOTREACHED */. Compilers don't really need it (one can use __dead2 instead), and style(9) was not even consistent with itself in this regard. Modified: head/share/man/man9/style.9 Modified: head/share/man/man9/style.9 ============================================================================== --- head/share/man/man9/style.9 Thu Jan 6 08:13:30 2011 (r217047) +++ head/share/man/man9/style.9 Thu Jan 6 08:33:48 2011 (r217048) @@ -470,9 +470,6 @@ statement that cascade should have a .Li FALLTHROUGH comment. Numerical arguments should be checked for accuracy. -Code that cannot be reached should have a -.Li NOTREACHED -comment. .Bd -literal while ((ch = getopt(argc, argv, "abNn:")) != -1) switch (ch) { /* Indent the switch. */ @@ -496,7 +493,6 @@ comment. case '?': default: usage(); - /* NOTREACHED */ } argc -= optind; argv += optind;
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201101060833.p068Xmrj008299>