Bug 200034 - [patch]: remove ieee80211_ack_rate() (duplicates ieee80211_ctl_rate())
Summary: [patch]: remove ieee80211_ack_rate() (duplicates ieee80211_ctl_rate())
Status: New
Alias: None
Product: Base System
Classification: Unclassified
Component: wireless (show other bugs)
Version: CURRENT
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-wireless (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2015-05-07 20:05 UTC by Andriy Voskoboinyk
Modified: 2015-05-08 03:53 UTC (History)
1 user (show)

See Also:

Attachments
Remove ieee80211_ack_rate() / unused ackrate variables (4.47 KB, patch)
2015-05-07 20:05 UTC, Andriy Voskoboinyk 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-05-07 20:05:52 UTC 
Created attachment 156478 [details]
Remove ieee80211_ack_rate() / unused ackrate variables

At this moment ieee80211_ack_rate():
 - fully duplicates contents of ieee80211_ctl_rate();
 - is not used for anything useful;
so it can be safely removed.

Also, this patch fixes some 'dead assignment' warnings (from scan.freebsd.org)
Comment 1 Adrian Chadd freebsd_committer freebsd_triage 2015-05-08 03:53:57 UTC 
Hm ,that's odd. ACK and control rates can be different.

I'll go look into it some more.