Changeset View
Changeset View
Standalone View
Standalone View
src/wirelessdevice.h
Show First 20 Lines • Show All 123 Lines • ▼ Show 20 Line(s) | 47 | public: | |||
---|---|---|---|---|---|
124 | WirelessDevice::OperationMode mode() const; | 124 | WirelessDevice::OperationMode mode() const; | ||
125 | /** | 125 | /** | ||
126 | * Retrieves the effective bit rate currently attainable by this device. | 126 | * Retrieves the effective bit rate currently attainable by this device. | ||
127 | * | 127 | * | ||
128 | * @return the bitrate in Kbit/s | 128 | * @return the bitrate in Kbit/s | ||
129 | */ | 129 | */ | ||
130 | int bitRate() const; | 130 | int bitRate() const; | ||
131 | /** | 131 | /** | ||
132 | * The time when the last "LastScan" property change was received | ||||
133 | * @since 5.62.0 | ||||
jgrulich: Add something like:
@note will always return invalid QDateTime when runtime NM < 1.12 | |||||
134 | * @note will always return invalid QDateTime when runtime NM < 1.12 | ||||
135 | * @return | ||||
136 | */ | ||||
137 | QDateTime lastScanTime() const; | ||||
138 | /** | ||||
139 | * The time the last RequestScan function was called | ||||
140 | * @since 5.62.0 | ||||
141 | * @note will always return invalid QDateTime when runtime NM < 1.12 | ||||
142 | * @return | ||||
143 | */ | ||||
Thinking about it now, wouldn't be this wording better? QDateTime lastScanRequestTime() const; Maybe even without the "Time" at the end so it's consistent with "lastScan". jgrulich: Thinking about it now, wouldn't be this wording better?
QDateTime lastScanRequestTime() const… | |||||
I chose lastRequestScanTime to have RequestScan in the name, that is the function, it keeps the last time called from. The Time suffix does not feel great indeed. I can make an adjustment to lastRequestScan, I will adjust D23578 then meven: I chose lastRequestScanTime to have RequestScan in the name, that is the function, it keeps the… | |||||
jgrulich: Please do, you can push it directly, it's just a function rename. | |||||
meven: https://cgit.kde.org/networkmanager-qt.git/commit/?id=af98fdba63e32c38008592774436ae6c9b61cd90 | |||||
144 | QDateTime lastRequestScanTime() const; | ||||
145 | /** | ||||
132 | * Retrieves the capabilities of this wifi network. | 146 | * Retrieves the capabilities of this wifi network. | ||
133 | * | 147 | * | ||
134 | * @return the flag set describing the capabilities | 148 | * @return the flag set describing the capabilities | ||
135 | * @see Capabilities | 149 | * @see Capabilities | ||
136 | */ | 150 | */ | ||
137 | WirelessDevice::Capabilities wirelessCapabilities() const; | 151 | WirelessDevice::Capabilities wirelessCapabilities() const; | ||
138 | 152 | | |||
139 | /** | 153 | /** | ||
▲ Show 20 Lines • Show All 65 Lines • ▼ Show 20 Line(s) | 180 | Q_SIGNALS: | |||
205 | /** | 219 | /** | ||
206 | * A wireless network appeared | 220 | * A wireless network appeared | ||
207 | */ | 221 | */ | ||
208 | void networkAppeared(const QString &ssid); | 222 | void networkAppeared(const QString &ssid); | ||
209 | /** | 223 | /** | ||
210 | * A wireless network disappeared | 224 | * A wireless network disappeared | ||
211 | */ | 225 | */ | ||
212 | void networkDisappeared(const QString &ssid); | 226 | void networkDisappeared(const QString &ssid); | ||
227 | /** | ||||
228 | * The LastScanTime property has changed, meaning a scan has just finished | ||||
229 | * @since 5.62.0 | ||||
230 | * @note will never be emitted when runtime NM < 1.12 | ||||
jgrulich: Add something like:
@note will never be emitted when runtime NM < 1.12 | |||||
231 | * @see lastScanTime | ||||
232 | */ | ||||
233 | void lastScanTimeChanged(const QDateTime timestamp); | ||||
jgrulich: const QDateTime ×tamp | |||||
213 | 234 | | |||
214 | private: | 235 | private: | ||
215 | Q_DECLARE_PRIVATE(WirelessDevice) | 236 | Q_DECLARE_PRIVATE(WirelessDevice) | ||
216 | }; | 237 | }; | ||
217 | 238 | | |||
218 | } // namespace NetworkManager | 239 | } // namespace NetworkManager | ||
219 | #endif //NETWORKMANAGERQT_WIRELESSDEVICE_H | 240 | #endif //NETWORKMANAGERQT_WIRELESSDEVICE_H | ||
220 | 241 | |
Add something like:
@note will always return invalid QDateTime when runtime NM < 1.12