param.h
ClosedPublic
Actions

Authored by arichardson on Oct 27 2020, 4:52 PM.

Tags

None

Referenced Files

	F161687162: D26979.id78826.diff
	Sun, Jul 5, 11:13 PM

	Unknown Object (File)
	Sat, Jul 4, 11:49 AM

	Unknown Object (File)
	Fri, Jul 3, 6:16 AM

	Unknown Object (File)
	Thu, Jul 2, 10:14 PM

	Unknown Object (File)
	Wed, Jul 1, 7:26 AM

	Unknown Object (File)
	Mon, Jun 22, 5:32 AM

	Unknown Object (File)
	Wed, Jun 17, 4:12 AM

	Unknown Object (File)
	Mon, Jun 8, 9:39 PM

View All Files

Subscribers

imp

Details

Reviewers

emaste
kib

Commits

rS367099: clang-format: place sys/systm.h immediately after sys/param.h

Summary

Should fix one of the objections to r366993

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 34423
Build 31527: arc lint + arc unit

Event Timeline

arichardson requested review of this revision.Oct 27 2020, 4:52 PM

arichardson created this revision.

Harbormaster completed remote builds in B34423: Diff 78826.Oct 27 2020, 4:52 PM

LGTM. I found a few examples in sys/ that don't follow this but most do, and I discussed w/ kib in email that we should describe this in style(9) too.

This revision is now accepted and ready to land.Oct 27 2020, 8:35 PM

Closed by commit rS367099: clang-format: place sys/systm.h immediately after sys/param.h (authored by arichardson). · Explain WhyOct 28 2020, 11:54 AM

This revision was automatically updated to reflect the committed changes.

arichardson added a commit: rS367099: clang-format: place sys/systm.h immediately after sys/param.h.

Herald added a subscriber: imp. · View Herald TranscriptOct 28 2020, 11:54 AM

Revision Contents
Changeset List

Path

Size

.clang-format

5 lines

Diff 78826

View Options

clang-format: place sys/systm.h immediately after sys/param.hClosedPublicActions