Differential D41324

iflib: Fix white space and reduce some line lengths
ClosedPublic
Actions

Authored by erj on Aug 4 2023, 6:03 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Nov 19, 9:31 AM

	Unknown Object (File)
	Wed, Nov 19, 6:43 AM

	Unknown Object (File)
	Sun, Nov 16, 2:27 PM

	Unknown Object (File)
	Sun, Nov 16, 12:20 AM

	Unknown Object (File)
	Sun, Nov 9, 11:48 PM

	Unknown Object (File)
	Tue, Nov 4, 4:34 AM

	Unknown Object (File)
	Mon, Nov 3, 4:56 AM

	Unknown Object (File)
	Sun, Nov 2, 4:13 PM

Subscribers

Details

Reviewers

shurd
kbowling

Group Reviewers

Commits

rGdadd804ded83: iflib: Fix white space and reduce some line lengths
rG7f527d48124b: iflib: Fix white space and reduce some line lengths

Summary

Signed-off-by: Eric Joyner <erj@FreeBSD.org>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

erj created this revision.Aug 4 2023, 6:03 PM

Herald added a reviewer: shurd. · View Herald TranscriptAug 4 2023, 6:03 PM

Herald added a reviewer: iflib. · View Herald Transcript

Herald added subscribers: glebius, melifaro, ae, imp. · View Herald Transcript

erj requested review of this revision.Aug 4 2023, 6:03 PM

Harbormaster completed remote builds in B52980: Diff 125595.Aug 4 2023, 6:03 PM

erj added a child revision: D41325: iflib: Remove redundant variable.Aug 4 2023, 6:05 PM

erj mentioned this in D39930: iflib: Add subinterface interrupt allocation function.Aug 4 2023, 6:17 PM

kbowling accepted this revision.Aug 4 2023, 6:47 PM

This revision is now accepted and ready to land.Aug 4 2023, 6:47 PM

This change doesn't acutally fix white space violations but introduces additional ones as according to style(9): 'Second level indents are four spaces'.

In D41324#941005, @marius wrote:

This change doesn't acutally fix white space violations but introduces additional ones as according to style(9): 'Second level indents are four spaces'.

It's "fix" in the context of whatever style the existing iflib code uses; if you want style(9) adherence, someone should probably re-write the iflib code to match it.

Closed by commit rG7f527d48124b: iflib: Fix white space and reduce some line lengths (authored by erj). · Explain WhyAug 7 2023, 10:47 PM

This revision was automatically updated to reflect the committed changes.

erj added a commit: rG7f527d48124b: iflib: Fix white space and reduce some line lengths.

erj added a commit: rGdadd804ded83: iflib: Fix white space and reduce some line lengths.Aug 24 2023, 9:16 PM

Revision Contents
Changeset List

Path

Size

sys/

net/

7 lines

18 lines

Diff 125683

sys/net/iflib.h

Loading...

sys/net/iflib.c

Loading...