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)
Sun, Sep 22, 2:28 PM
Unknown Object (File)
Sun, Sep 22, 8:57 AM
Unknown Object (File)
Sat, Sep 21, 3:05 PM
Unknown Object (File)
Sat, Sep 21, 1:35 AM
Unknown Object (File)
Thu, Sep 19, 4:47 AM
Unknown Object (File)
Tue, Sep 17, 11:56 PM
Unknown Object (File)
Tue, Sep 17, 8:25 PM
Unknown Object (File)
Tue, Sep 17, 5:52 AM
View All 42 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...