|
Bugzilla – Full Text Bug Listing |
| Summary: | suggest to add ATTR_GET to WifiRemoteStationManager::IsLowLatency attribute | ||
|---|---|---|---|
| Product: | ns-3 | Reporter: | Tom Henderson <tomh> |
| Component: | wifi | Assignee: | sebastien.deronne |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ns-bugs |
| Priority: | P5 | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
(In reply to Tom Henderson from comment #0) > The following attribute of WifiRemoteStationManager is inoperative and > misleading; suggest to remove it. > Actually, my mistake; I see how it works now, but it should instead use the TypeId::ATTR_GET, flag to preclude set operations. See e.g. WaypointMobilityModel .AddAttribute ("WaypointsLeft", "The number of waypoints remaining.", TypeId::ATTR_GET, UintegerValue (0), MakeUintegerAccessor (&WaypointMobilityModel::WaypointsLeft), MakeUintegerChecker<uint32_t> ()) Tom, I agree to change it as suggested. fixed in changeset 11744:d6a6751315f9 |
The following attribute of WifiRemoteStationManager is inoperative and misleading; suggest to remove it. .AddAttribute ("IsLowLatency", "If true, we attempt to modelize a so-called low-latency device: a device" " where decisions about tx parameters can be made on a per-packet basis and feedback about the" " transmission of each packet is obtained before sending the next. Otherwise, we modelize a " " high-latency device, that is a device where we cannot update our decision about tx parameters" " after every packet transmission.", BooleanValue (true), //this value is ignored because there is no setter MakeBooleanAccessor (&WifiRemoteStationManager::IsLowLatency), MakeBooleanChecker ()) Perhaps move the documentation string to the doxygen of WifiRemoteStationManager::IsLowLatency()