Page MenuHomeFreeBSD
Differential D36411

net80211: simplify an #ifdef INET/INET6 block
ClosedPublic
Actions

Authored by bz on Aug 31 2022, 10:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 25, 11:09 PM
Unknown Object (File)
Fri, Oct 24, 6:57 AM
Unknown Object (File)
Mon, Oct 20, 4:33 AM
Unknown Object (File)
Sat, Oct 18, 4:03 PM
Unknown Object (File)
Wed, Oct 8, 8:25 AM
Unknown Object (File)
Sep 27 2025, 1:06 AM
Unknown Object (File)
Sep 26 2025, 7:33 AM
Unknown Object (File)
Sep 18 2025, 9:35 PM
View All Files
Subscribers
glebius
imp
melifaro
zlei

Details

Reviewers
zlei
Group Reviewers
wireless
Commits
rGb1a57b105ce6: net80211: simplify an #ifdef INET/INET6 block
rG8def06d5afdc: net80211: simplify an #ifdef INET/INET6 block
Summary

I got lost in the #ifdef #endif here so I checked the code to a
switch block with two non-overlapping #ifdef parts and a default
which makes it a lot easier to read.

No functional changes.

Sponsored by: The FreeBSD Foundation
MFC after: 5 days

Test Plan

I would highly appreciate if people would carefully
double-check that I manually change-ifdefed it correctly.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 47213
Build 44100: arc lint + arc unit

Event Timeline

bz created this revision.Aug 31 2022, 10:39 PM
Herald added subscribers: glebius, melifaro, imp. · View Herald TranscriptAug 31 2022, 10:39 PM
bz requested review of this revision.Aug 31 2022, 10:39 PM
Harbormaster completed remote builds in B47213: Diff 110028.Aug 31 2022, 10:39 PM
zlei added a subscriber: zlei.Sep 2 2022, 1:12 AM
zlei added inline comments.
sys/net80211/ieee80211_output.c
1289

Missing break here.

1305

ditto

bz updated this revision to Diff 110071.Sep 2 2022, 1:57 AM

Add missing breaks. Thank you for care fully checking! :)

Harbormaster completed remote builds in B47228: Diff 110071.Sep 2 2022, 1:57 AM
bz marked 2 inline comments as done.Sep 2 2022, 1:57 AM
zlei accepted this revision.Sep 2 2022, 3:58 AM

Looks good to me.

This revision is now accepted and ready to land.Sep 2 2022, 3:58 AM
Closed by commit rG8def06d5afdc: net80211: simplify an #ifdef INET/INET6 block (authored by bz). · Explain WhySep 3 2022, 9:02 PM
This revision was automatically updated to reflect the committed changes.
bz added a commit: rG8def06d5afdc: net80211: simplify an #ifdef INET/INET6 block.
bz added a commit: rGb1a57b105ce6: net80211: simplify an #ifdef INET/INET6 block.Sep 21 2022, 2:04 PM

Revision Contents
Changeset List

PathSize
sys/
net80211/
24 lines

Diff 110028

View Options

sys/net80211/ieee80211_output.c

Loading...