diff options
| author | Léo Lam <leo@innovatetechnologi.es> | 2018-04-28 11:13:03 +0200 |
|---|---|---|
| committer | Léo Lam <leo@innovatetechnologi.es> | 2018-04-28 11:13:03 +0200 |
| commit | 4f427de9247c5e581c5b2cf1f3d3003fac93e20e (patch) | |
| tree | a4f4a0ce416e37ac687774188e9fcf1d1595487c /Source/Core | |
| parent | cf1adfbe201a5d5a1b66957a392a82401078c015 (diff) | |
IOS/SO: Clean up variable declarations
Move them closer to their usage and fix the naming.
Diffstat (limited to 'Source/Core')
| -rw-r--r-- | Source/Core/Core/IOS/Network/IP/Top.cpp | 38 |
1 files changed, 18 insertions, 20 deletions
diff --git a/Source/Core/Core/IOS/Network/IP/Top.cpp b/Source/Core/Core/IOS/Network/IP/Top.cpp index 81f6a1a5d7..b5d850672c 100644 --- a/Source/Core/Core/IOS/Network/IP/Top.cpp +++ b/Source/Core/Core/IOS/Network/IP/Top.cpp @@ -167,34 +167,32 @@ struct DefaultInterface static std::optional<DefaultInterface> GetSystemDefaultInterface() { #ifdef _WIN32 - DWORD forwardTableSize, ipTableSize, result; - NET_IFINDEX ifIndex = NET_IFINDEX_UNSPECIFIED; - std::unique_ptr<MIB_IPFORWARDTABLE> forwardTable; - std::unique_ptr<MIB_IPADDRTABLE> ipTable; - - forwardTableSize = 0; - if (GetIpForwardTable(nullptr, &forwardTableSize, FALSE) == ERROR_INSUFFICIENT_BUFFER) + std::unique_ptr<MIB_IPFORWARDTABLE> forward_table; + DWORD forward_table_size = 0; + if (GetIpForwardTable(nullptr, &forward_table_size, FALSE) == ERROR_INSUFFICIENT_BUFFER) { - forwardTable = - std::unique_ptr<MIB_IPFORWARDTABLE>((PMIB_IPFORWARDTABLE) operator new(forwardTableSize)); + forward_table = + std::unique_ptr<MIB_IPFORWARDTABLE>((PMIB_IPFORWARDTABLE) operator new(forward_table_size)); } - ipTableSize = 0; - if (GetIpAddrTable(nullptr, &ipTableSize, FALSE) == ERROR_INSUFFICIENT_BUFFER) + std::unique_ptr<MIB_IPADDRTABLE> ip_table; + DWORD ip_table_size = 0; + if (GetIpAddrTable(nullptr, &ip_table_size, FALSE) == ERROR_INSUFFICIENT_BUFFER) { - ipTable = std::unique_ptr<MIB_IPADDRTABLE>((PMIB_IPADDRTABLE) operator new(ipTableSize)); + ip_table = std::unique_ptr<MIB_IPADDRTABLE>((PMIB_IPADDRTABLE) operator new(ip_table_size)); } // find the interface IP used for the default route and use that - result = GetIpForwardTable(forwardTable.get(), &forwardTableSize, FALSE); + NET_IFINDEX ifIndex = NET_IFINDEX_UNSPECIFIED; + DWORD result = GetIpForwardTable(forward_table.get(), &forward_table_size, FALSE); // can return ERROR_MORE_DATA on XP even after the first call while (result == NO_ERROR || result == ERROR_MORE_DATA) { - for (DWORD i = 0; i < forwardTable->dwNumEntries; ++i) + for (DWORD i = 0; i < forward_table->dwNumEntries; ++i) { - if (forwardTable->table[i].dwForwardDest == 0) + if (forward_table->table[i].dwForwardDest == 0) { - ifIndex = forwardTable->table[i].dwForwardIfIndex; + ifIndex = forward_table->table[i].dwForwardIfIndex; break; } } @@ -202,15 +200,15 @@ static std::optional<DefaultInterface> GetSystemDefaultInterface() if (result == NO_ERROR || ifIndex != NET_IFINDEX_UNSPECIFIED) break; - result = GetIpForwardTable(forwardTable.get(), &forwardTableSize, FALSE); + result = GetIpForwardTable(forward_table.get(), &forward_table_size, FALSE); } if (ifIndex != NET_IFINDEX_UNSPECIFIED && - GetIpAddrTable(ipTable.get(), &ipTableSize, FALSE) == NO_ERROR) + GetIpAddrTable(ip_table.get(), &ip_table_size, FALSE) == NO_ERROR) { - for (DWORD i = 0; i < ipTable->dwNumEntries; ++i) + for (DWORD i = 0; i < ip_table->dwNumEntries; ++i) { - const auto& entry = ipTable->table[i]; + const auto& entry = ip_table->table[i]; if (entry.dwIndex == ifIndex) return DefaultInterface{entry.dwAddr, entry.dwMask, entry.dwBCastAddr}; } |
