Cross-reference style(9) and style.mdoc(5)

Suggested by: yuripv
MFC after: 3 days

Merge ^/head r358269 through r358399.

_Static_assert is to be preferred to CTASSERT.

Document the existing prefernce that _Static_assert be used in preference to the
old CTASSERT we used to use for compile time assertions.

style.9: update C99 commentary

Make style.9 read as a current statement of C99 preferences, rather than a
description of ongoing changes to our preferred style. Alsu use the short
form "ISO C99" on

style.9: update C99 commentary

Make style.9 read as a current statement of C99 preferences, rather than a
description of ongoing changes to our preferred style. Alsu use the short
form "ISO C99" on the 2nd and later instances rather than repeating the
unwieldy `ISO/IEC 9899:1999 ("ISO C99")` each time.

Reviewed by: cem, imp, jhb, kib
MFC after: 1 month
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D23648

show more ...

Better copyright advice

Document the common practices around copyrights with "all rights reserved" in
them as new copyright notices get added.

It's an open question qhether to point people at the f

Better copyright advice

Document the common practices around copyrights with "all rights reserved" in
them as new copyright notices get added.

It's an open question qhether to point people at the fact that since the Berne
convention was ratified, All rights reserved is largely obsolete.
https://en.wikipedia.org/wiki/All_rights_reserved#Obsolescence has the
details. The committer's guide will be revised shortly, and it's likely that's a
better place for this discussion. If not, I'll add a blurb here.

Reviewed by: jhb@, brooks@
Differential Review: https://reviews.freebsd.org/D22800

show more ...

MFHead @348740

Sponsored by: The FreeBSD Foundation

style.9: Codify tolerance for eliding blank lines

Consensus seems to be that eliding blank lines for functions with no local
variables is acceptable. Codify that explicitly in the style document.

style.9: Codify tolerance for eliding blank lines

Consensus seems to be that eliding blank lines for functions with no local
variables is acceptable. Codify that explicitly in the style document.

Reported by: jhb
Reviewed by: delphij, imp, vangyzen (earlier version); rgrimes
With feedback from: kib
Differential Revision: https://reviews.freebsd.org/D20448

show more ...

style.9: Correct usage's definition to match its declaration

Suggested by: emaste
Reviewed by: delphij, imp, rgrimes, vangyzen (earlier version)
Sponsored by: Dell EMC Isilon
Differential Revision:

style.9: Correct usage's definition to match its declaration

Suggested by: emaste
Reviewed by: delphij, imp, rgrimes, vangyzen (earlier version)
Sponsored by: Dell EMC Isilon
Differential Revision: (part of D20448)

show more ...

Merge ^/head r339813 through r340125.

style: remove "All rights reserved" from comment

imp removed it from the template in r333391, so remove it from the
example license header in style.9 as well.

Sponsored by: The FreeBSD Foundation

style(9): add some additional useful FILES/xref information

Submitted by: 0mp
Reviewed by: imp
Differential Revision: https://reviews.freebsd.org/D15196

Remove reference to lint. We no longer support it, so exceptions for
it are no longer relevant.

Update to include information about using SPDX-License-Identifier
lines.

Remove K&R stuff: never use it in new code.

Remove how to format K&R stuff. The project hasn't been using it in
new code for a long time. It's so obsolete, we don't need a statement
to never use it.

Remove K&R stuff: never use it in new code.

Remove how to format K&R stuff. The project hasn't been using it in
new code for a long time. It's so obsolete, we don't need a statement
to never use it. Add a statement requesting that comments about
parameters be preserved when converting to ASNI style, per Kirk.

Differential Revision: https://reviews.freebsd.org/D14051

show more ...

Bump .Dd date for c99 change

Add info about c99 designationed initializers.

Differential Revision: https://reviews.freebsd.org/D13975

No need for the zero width space now that I put the comma in.

Make igor happier with this file:
o Don't use contractions.
o Add common after e.g. where needed

Merge ^/head r325842 through r325998.

Bump dates after lint removal.

Noted by: wblock
Sponsored by: The FreeBSD Foundation

Remove xlint(1).

xlint is currently a fossil. We have much more useful and alive tools
to do now what xlint did twenty years ago.

I did not cleared some stuff which makes lint operational, in
sys/

Remove xlint(1).

xlint is currently a fossil. We have much more useful and alive tools
to do now what xlint did twenty years ago.

I did not cleared some stuff which makes lint operational, in
sys/x86/include and sys/sys, but I might do it as followup. The
x86/include/ucontext.h and _types.h hacks made to please lint was the
main reason for my initial proposal to classify xlint as obsolete and
to remove it.

Also I do not intend to clear sccs ids.

Reviewed by: bapt, brooks, emaste, jhb, pfg
Sponsored by: The FreeBSD Foundation
Differential revision: https://reviews.freebsd.org/D13015

show more ...

Merge ^/head r306412 through r306905.

style(9): Some additional clarification

Prompted by an email from bde@.

Reviewed by: emaste, imp (earlier version)
With input from: wblock
Sponsored by: Dell EMC Isilon
Differential Revision: https

style(9): Some additional clarification

Prompted by an email from bde@.

Reviewed by: emaste, imp (earlier version)
With input from: wblock
Sponsored by: Dell EMC Isilon
Differential Revision: https://reviews.freebsd.org/D7983

show more ...

Merge ^/head r305892 through r306302.

Spell alphabetically correctly both in the commit message AND in the
actual man page. Sigh.

Submitted by: David A Bright and Pedro Giffuni