Differential D29798

ixgbe: Clarify variable index usage in ixgbe_mc_filter_apply
ClosedPublic
Actions

Authored by kbowling on Apr 16 2021, 8:50 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, May 18, 9:47 PM

	Unknown Object (File)
	Thu, May 4, 3:51 AM

	Unknown Object (File)
	Apr 3 2023, 9:51 PM

	Unknown Object (File)
	Mar 12 2023, 10:19 PM

	Unknown Object (File)
	Mar 4 2023, 9:35 PM

	Unknown Object (File)
	Feb 15 2023, 6:45 AM

	Unknown Object (File)
	Jan 12 2023, 8:18 AM

	Unknown Object (File)
	Jan 5 2023, 10:42 PM

View All 15 Files

Subscribers

Details

Reviewers

Group Reviewers

Intel Networking

Commits

rG2d8adf9cc000: ixgbe: Clarify index name in ixgbe_mc_filter_apply
rG8209a085c701: ixgbe: Clarify index name in ixgbe_mc_filter_apply
rG21afed4b1d18: ixgbe: Clarify index name in ixgbe_mc_filter_apply

Summary

From @erj "It looks like it would be less confusing to rename "count" to something like "idx", since that's what it's used for in this function."

Applies the same variable name change as e1000 in D29789

Test Plan

N/A

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kbowling created this revision.Apr 16 2021, 8:50 PM

Herald added a subscriber: ae. · View Herald TranscriptApr 16 2021, 8:50 PM

kbowling requested review of this revision.Apr 16 2021, 8:50 PM

erj accepted this revision.Apr 16 2021, 11:14 PM

This revision is now accepted and ready to land.Apr 16 2021, 11:14 PM

Closed by commit rG21afed4b1d18: ixgbe: Clarify index name in ixgbe_mc_filter_apply (authored by Kevin Bowling <kbowling@FreeBSD.org>). · Explain WhyApr 17 2021, 1:21 AM

This revision was automatically updated to reflect the committed changes.

Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG21afed4b1d18: ixgbe: Clarify index name in ixgbe_mc_filter_apply.

Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG8209a085c701: ixgbe: Clarify index name in ixgbe_mc_filter_apply.Apr 25 2021, 7:44 AM

Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG2d8adf9cc000: ixgbe: Clarify index name in ixgbe_mc_filter_apply.Apr 25 2021, 7:47 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

ixgbe/

8 lines

Diff 87599

sys/dev/ixgbe/if_ix.c

Loading...