NetPlayServer: remove goto statements

This commit is contained in:
Michael M 2017-08-06 23:05:51 -07:00
parent dbb26a2b54
commit 2abf13ae36
1 changed files with 6 additions and 8 deletions

View File

@ -959,17 +959,15 @@ void NetPlayServer::TryPortmapping(u16 port)
void NetPlayServer::mapPortThread(const u16 port) void NetPlayServer::mapPortThread(const u16 port)
{ {
if (!m_upnp_inited) if (!m_upnp_inited)
if (!initUPnP()) initUPnP();
goto fail;
if (!UPnPMapPort(m_upnp_ourip, port)) if (m_upnp_inited && UPnPMapPort(m_upnp_ourip, port))
goto fail; {
NOTICE_LOG(NETPLAY, "Successfully mapped port %d to %s.", port, m_upnp_ourip.c_str());
return;
}
NOTICE_LOG(NETPLAY, "Successfully mapped port %d to %s.", port, m_upnp_ourip.c_str());
return;
fail:
WARN_LOG(NETPLAY, "Failed to map port %d to %s.", port, m_upnp_ourip.c_str()); WARN_LOG(NETPLAY, "Failed to map port %d to %s.", port, m_upnp_ourip.c_str());
return;
} }
// UPnP thread: try to unmap a port // UPnP thread: try to unmap a port