hwpmc_mod.c: whitespace style cleanup
ClosedPublic
Actions

Authored by mhorne on Jun 13 2023, 5:46 PM.

Project Tags

None

Referenced Files

	F176952436: D40514.1793647048.diff
	Sun, Nov 1, 7:17 PM

Subscribers

imp

Details

Reviewers

jkoshy
gnn

Commits

rG693cd30797b8: hwpmc_mod.c: whitespace style cleanup

Summary

Handle a few things related to spacing:

Remove redundant/superfluous blank lines (and add a couple where helpful)
Add spacing around binary operators
Remove spacing after casts and before goto labels
Adjustments for line width of 80 chars
Tab/space character issues

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 52023
Build 48914: arc lint + arc unit

Event Timeline

mhorne created this revision.Jun 13 2023, 5:46 PM

Herald added a reviewer: gnn. · View Herald TranscriptJun 13 2023, 5:46 PM

Herald added a subscriber: imp. · View Herald Transcript

mhorne requested review of this revision.Jun 13 2023, 5:46 PM

Harbormaster completed remote builds in B52023: Diff 123162.Jun 13 2023, 5:46 PM

mhorne added a parent revision: D40513: hwpmc_mod.c: return statement style.Jun 13 2023, 5:46 PM

jkoshy added inline comments.Jun 14 2023, 9:24 AM

sys/dev/hwpmc/hwpmc_mod.c
1792	Consider keeping this blank line, as it visually separates exit processing steps from the rest of the function?

jkoshy accepted this revision.Jun 14 2023, 9:42 AM

jkoshy added inline comments.

sys/dev/hwpmc/hwpmc_mod.c
4999	Consider keeping this blank line to preserve visual separation?
5327	Consider keeping this blank line, for better visual separation of exit processing from the rest of the function?

This revision is now accepted and ready to land.Jun 14 2023, 9:42 AM

Closed by commit rG693cd30797b8: hwpmc_mod.c: whitespace style cleanup. · Explain WhyJun 14 2023, 4:47 PM

This revision was automatically updated to reflect the committed changes.

mhorne marked 3 inline comments as done.

mhorne added a commit: rG693cd30797b8: hwpmc_mod.c: whitespace style cleanup.

mhorne mentioned this in D40289: hwpmc: style(9) pass of hwpmc_mod.c.Jun 14 2023, 4:49 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

hwpmc/

hwpmc_mod.c

463 lines

Diff 123162

View Options

hwpmc_mod.c: whitespace style cleanupClosedPublicActions