param.h
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F168686638: D26979.id78826.diff
	Sat, Aug 29, 3:20 PM

	Unknown Object (File)
	Wed, Aug 26, 11:43 AM

	Unknown Object (File)
	Wed, Aug 26, 10:43 AM

	Unknown Object (File)
	Tue, Aug 25, 9:27 AM

	Unknown Object (File)
	Mon, Aug 24, 11:33 AM

	Unknown Object (File)
	Mon, Aug 24, 8:59 AM

	Unknown Object (File)
	Sun, Aug 23, 3:21 PM

	Unknown Object (File)
	Thu, Aug 20, 10:02 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