colin/NetworkManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

manager: reorder conditions in unmanaged_to_disconnected() to check cheaper condition first

Browse Source 
Getting nm_device_get_state() is cheap, contrary to nm_device_is_available().
Reorder the checks.
This commit is contained in:
Thomas Haller
2018-02-05 15:01:48 +01:00
parent fc0430b1ab
commit 6075348f0f
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/nm-manager.c
View File

@@ -3561,8 +3561,8 @@ unmanaged_to_disconnected (NMDevice *device)
NM_DEVICE_STATE_REASON_USER_REQUESTED);
}
if ( nm_device_is_available (device, NM_DEVICE_CHECK_DEV_AVAILABLE_FOR_USER_REQUEST)
&& (nm_device_get_state (device) == NM_DEVICE_STATE_UNAVAILABLE)) {
if ( nm_device_get_state (device) == NM_DEVICE_STATE_UNAVAILABLE
&& nm_device_is_available (device, NM_DEVICE_CHECK_DEV_AVAILABLE_FOR_USER_REQUEST)) {
nm_device_state_changed (device,
NM_DEVICE_STATE_DISCONNECTED,
NM_DEVICE_STATE_REASON_USER_REQUESTED);