param.h
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F137404014: D26979.diff
	Sun, Nov 23, 3:21 AM

	Unknown Object (File)
	Fri, Nov 21, 10:55 PM

	Unknown Object (File)
	Thu, Nov 20, 7:59 PM

	Unknown Object (File)
	Thu, Nov 20, 7:58 PM

	Unknown Object (File)
	Thu, Nov 20, 7:58 PM

	Unknown Object (File)
	Thu, Nov 20, 7:57 PM

	Unknown Object (File)
	Thu, Nov 20, 7:56 PM

	Unknown Object (File)
	Thu, Nov 20, 7:56 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