Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1500)

Issue 313560043: Final updates to correct doxygen warnings for the wifi module (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 years, 7 months ago by ammo6818-vandals.uidaho.edu
Modified:
2 years, 8 months ago
Reviewers:
S. Deronne
CC:
ns-3-reviews_googlegroups.com
Visibility:
Public.

Description

Final updates to correct doxygen warnings for the wifi module

Patch Set 1 #

Patch Set 2 : Updates to correct doxygen warnings for latest external merge #

Total comments: 91

Patch Set 3 : Doxygen warning correction #

Patch Set 4 : Additional doxygen corrections #

Patch Set 5 : Additional doxygen corrections #

Total comments: 24

Patch Set 6 : Additional doxygen corrections for the latest updates #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -1 line) Patch
M src/wifi/model/wifi-mac-queue.h View 1 2 3 4 5 1 chunk +12 lines, -1 line 0 comments Download

Messages

Total messages: 6
S. Deronne
https://codereview.appspot.com/313560043/diff/20001/src/wifi/helper/athstats-helper.h File src/wifi/helper/athstats-helper.h (right): https://codereview.appspot.com/313560043/diff/20001/src/wifi/helper/athstats-helper.h#newcode50 src/wifi/helper/athstats-helper.h:50: * \param filename the fiel name file https://codereview.appspot.com/313560043/diff/20001/src/wifi/helper/qos-wifi-mac-helper.h File ...
3 years, 7 months ago (2017-02-11 09:07:18 UTC) #1
S. Deronne
No doxygen updates needed for wifi-phy-configuration example? How can I check doxygen warnings to be ...
3 years, 7 months ago (2017-02-11 09:21:31 UTC) #2
S. Deronne
https://codereview.appspot.com/313560043/diff/20001/src/wifi/test/dcf-manager-test.cc File src/wifi/test/dcf-manager-test.cc (right): https://codereview.appspot.com/313560043/diff/20001/src/wifi/test/dcf-manager-test.cc#newcode58 src/wifi/test/dcf-manager-test.cc:58: typedef std::list<ExpectedGrant> ExpectedGrants; //!< the collection of expected grants ...
3 years, 7 months ago (2017-02-11 09:51:20 UTC) #3
S. Deronne
https://codereview.appspot.com/313560043/diff/20001/src/wifi/model/amsdu-subframe-header.h File src/wifi/model/amsdu-subframe-header.h (right): https://codereview.appspot.com/313560043/diff/20001/src/wifi/model/amsdu-subframe-header.h#newcode52 src/wifi/model/amsdu-subframe-header.h:52: * Set destination addresss function address https://codereview.appspot.com/313560043/diff/20001/src/wifi/model/amsdu-subframe-header.h#newcode72 src/wifi/model/amsdu-subframe-header.h:72: * ...
3 years, 7 months ago (2017-02-11 18:22:44 UTC) #4
S. Deronne
https://codereview.appspot.com/313560043/diff/80001/src/wifi/model/ampdu-tag.h File src/wifi/model/ampdu-tag.h (right): https://codereview.appspot.com/313560043/diff/80001/src/wifi/model/ampdu-tag.h#newcode51 src/wifi/model/ampdu-tag.h:51: * \param supported the A-MPDU suppoted flag s/suppoted/supported https://codereview.appspot.com/313560043/diff/80001/src/wifi/model/block-ack-manager.h ...
3 years, 3 months ago (2017-05-28 19:19:50 UTC) #5
S. Deronne
3 years, 3 months ago (2017-05-28 19:23:12 UTC) #6
https://codereview.appspot.com/313560043/diff/80001/src/wifi/helper/qos-wifi-...
File src/wifi/helper/qos-wifi-mac-helper.h (right):

https://codereview.appspot.com/313560043/diff/80001/src/wifi/helper/qos-wifi-...
src/wifi/helper/qos-wifi-mac-helper.h:56: * \returns  a default QosWifiMcHelper
extra space
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b