ShuriZma
/
dolphin
mirror of https://github.com/dolphin-emu/dolphin.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #7580 from Techjar/netplay-padmapping-type 

NetPlay: Remove PadMapping type
This commit is contained in:
Léo Lam 2019-01-16 21:57:38 +01:00 committed by GitHub
parent fd3ef7ebc5 529796bd59
commit c7373701cc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 37 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
Source/Core/Core/NetPlayClient.cpp
View File

@ -398,7 +398,7 @@ unsigned int NetPlayClient::OnData(sf::Packet& packet)
case NP_MSG_PAD_MAPPING:
{
for (PadMapping& mapping : m_pad_map)
for (PlayerId& mapping : m_pad_map)
{
packet >> mapping;
}
@ -411,7 +411,7 @@ unsigned int NetPlayClient::OnData(sf::Packet& packet)
case NP_MSG_WIIMOTE_MAPPING:
{
for (PadMapping& mapping : m_wiimote_map)
for (PlayerId& mapping : m_wiimote_map)
{
packet >> mapping;
}
@ -424,7 +424,7 @@ unsigned int NetPlayClient::OnData(sf::Packet& packet)
{
while (!packet.endOfPacket())
{
PadMapping map;
PadIndex map;
packet >> map;
GCPadStatus pad;
@ -441,7 +441,7 @@ unsigned int NetPlayClient::OnData(sf::Packet& packet)
case NP_MSG_WIIMOTE_DATA:
{
PadMapping map = 0;
PadIndex map;
NetWiimote nw;
u8 size;
packet >> map >> size;
@ -470,7 +470,7 @@ unsigned int NetPlayClient::OnData(sf::Packet& packet)
case NP_MSG_PAD_FIRST_RECEIVED:
{
PadMapping map;
PadIndex map;
packet >> map;
packet >> m_first_pad_status_received[map];
m_first_pad_status_received_event.Set();
@ -1318,7 +1318,7 @@ void NetPlayClient::SendChatMessage(const std::string& msg)
void NetPlayClient::AddPadStateToPacket(const int in_game_pad, const GCPadStatus& pad,
sf::Packet& packet)
{
packet << static_cast<PadMapping>(in_game_pad);
packet << static_cast<PadIndex>(in_game_pad);
packet << pad.button << pad.analogA << pad.analogB << pad.stickX << pad.stickY << pad.substickX
<< pad.substickY << pad.triggerLeft << pad.triggerRight << pad.isConnected;
}
@ -1328,7 +1328,7 @@ void NetPlayClient::SendWiimoteState(const int in_game_pad, const NetWiimote& nw
{
sf::Packet packet;
packet << static_cast<MessageId>(NP_MSG_WIIMOTE_DATA);
packet << static_cast<PadMapping>(in_game_pad);
packet << static_cast<PadIndex>(in_game_pad);
packet << static_cast<u8>(nw.size());
for (auto it : nw)
{
@ -1882,7 +1882,7 @@ bool NetPlayClient::PollLocalPad(const int local_pad, sf::Packet& packet)
return data_added;
}
void NetPlayClient::SendPadHostPoll(const PadMapping pad_num)
void NetPlayClient::SendPadHostPoll(const PadIndex pad_num)
{
if (!m_local_player->IsHost())
return;
@ -1903,7 +1903,7 @@ void NetPlayClient::SendPadHostPoll(const PadMapping pad_num)
}
}
}
else if (m_pad_map[pad_num] > 0)
else if (m_pad_map[pad_num] != 0)
{
while (!m_first_pad_status_received[pad_num])
{
@ -1977,7 +1977,7 @@ void NetPlayClient::SendPowerButtonEvent()
// called from ---GUI--- thread
bool NetPlayClient::LocalPlayerHasControllerMapped() const
{
const auto mapping_matches_player_id = [this](const PadMapping& mapping) {
const auto mapping_matches_player_id = [this](const PlayerId& mapping) {
return mapping == m_local_player->pid;
};

2
Source/Core/Core/NetPlayClient.h
View File

@ -210,7 +210,7 @@ private:
std::optional<std::vector<u8>> DecompressPacketIntoBuffer(sf::Packet& packet);
bool PollLocalPad(int local_pad, sf::Packet& packet);
void SendPadHostPoll(PadMapping pad_num);
void SendPadHostPoll(PadIndex pad_num);
void UpdateDevices();
void AddPadStateToPacket(int in_game_pad, const GCPadStatus& np, sf::Packet& packet);

4
Source/Core/Core/NetPlayProto.h
View File

@ -196,8 +196,8 @@ using NetWiimote = std::vector<u8>;
using MessageId = u8;
using PlayerId = u8;
using FrameNum = u32;
using PadMapping = s8;
using PadMappingArray = std::array<PadMapping, 4>;
using PadIndex = s8;
using PadMappingArray = std::array<PlayerId, 4>;
bool IsNetPlayRunning();
// Precondition: A netplay client instance must be present. In other words,

40
Source/Core/Core/NetPlayServer.cpp
View File

@ -107,8 +107,8 @@ NetPlayServer::NetPlayServer(const u16 port, const bool forward_port,
PanicAlertT("Enet Didn't Initialize");
}
m_pad_map.fill(-1);
m_wiimote_map.fill(-1);
m_pad_map.fill(0);
m_wiimote_map.fill(0);
if (traversal_config.use_traversal)
{
@ -325,9 +325,9 @@ unsigned int NetPlayServer::OnConnect(ENetPeer* socket, sf::Packet& rpac)
rpac >> player.name;
// try to automatically assign new user a pad
for (PadMapping& mapping : m_pad_map)
for (PlayerId& mapping : m_pad_map)
{
if (mapping == -1)
if (mapping == 0)
{
mapping = player.pid;
break;
@ -420,7 +420,7 @@ unsigned int NetPlayServer::OnDisconnect(const Client& player)
if (m_is_running)
{
for (PadMapping mapping : m_pad_map)
for (PlayerId& mapping : m_pad_map)
{
if (mapping == pid && pid != 1)
{
@ -430,7 +430,7 @@ unsigned int NetPlayServer::OnDisconnect(const Client& player)
sf::Packet spac;
spac << (MessageId)NP_MSG_DISABLE_GAME;
// this thread doesn't need players lock
SendToClients(spac, static_cast<PlayerId>(-1));
SendToClients(spac);
break;
}
}
@ -450,20 +450,20 @@ unsigned int NetPlayServer::OnDisconnect(const Client& player)
// alert other players of disconnect
SendToClients(spac);
for (PadMapping& mapping : m_pad_map)
for (PlayerId& mapping : m_pad_map)
{
if (mapping == pid)
{
mapping = -1;
mapping = 0;
UpdatePadMapping();
}
}
for (PadMapping& mapping : m_wiimote_map)
for (PlayerId& mapping : m_wiimote_map)
{
if (mapping == pid)
{
mapping = -1;
mapping = 0;
UpdateWiimoteMapping();
}
}
@ -501,7 +501,7 @@ void NetPlayServer::UpdatePadMapping()
{
sf::Packet spac;
spac << (MessageId)NP_MSG_PAD_MAPPING;
for (PadMapping mapping : m_pad_map)
for (PlayerId mapping : m_pad_map)
{
spac << mapping;
}
@ -513,7 +513,7 @@ void NetPlayServer::UpdateWiimoteMapping()
{
sf::Packet spac;
spac << (MessageId)NP_MSG_WIIMOTE_MAPPING;
for (PadMapping mapping : m_wiimote_map)
for (PlayerId mapping : m_wiimote_map)
{
spac << mapping;
}
@ -656,7 +656,7 @@ unsigned int NetPlayServer::OnData(sf::Packet& packet, Client& player)
while (!packet.endOfPacket())
{
PadMapping map;
PadIndex map;
packet >> map;
// If the data is not from the correct player,
@ -695,7 +695,7 @@ unsigned int NetPlayServer::OnData(sf::Packet& packet, Client& player)
case NP_MSG_PAD_HOST_POLL:
{
PadMapping pad_num;
PadIndex pad_num;
packet >> pad_num;
sf::Packet spac;
@ -705,16 +705,16 @@ unsigned int NetPlayServer::OnData(sf::Packet& packet, Client& player)
{
for (size_t i = 0; i < m_pad_map.size(); i++)
{
if (m_pad_map[i] == -1)
if (m_pad_map[i] == 0)
continue;
const GCPadStatus& pad = m_last_pad_status[i];
spac << static_cast<PadMapping>(i) << pad.button << pad.analogA << pad.analogB << pad.stickX
spac << static_cast<PadIndex>(i) << pad.button << pad.analogA << pad.analogB << pad.stickX
<< pad.stickY << pad.substickX << pad.substickY << pad.triggerLeft << pad.triggerRight
<< pad.isConnected;
}
}
else if (m_pad_map.at(pad_num) != -1)
else if (m_pad_map.at(pad_num) != 0)
{
const GCPadStatus& pad = m_last_pad_status[pad_num];
spac << pad_num << pad.button << pad.analogA << pad.analogB << pad.stickX << pad.stickY
@ -732,7 +732,7 @@ unsigned int NetPlayServer::OnData(sf::Packet& packet, Client& player)
if (player.current_game != m_current_game)
break;
PadMapping map = 0;
PadIndex map;
u8 size;
packet >> map >> size;
std::vector<u8> data(size);
@ -853,7 +853,7 @@ unsigned int NetPlayServer::OnData(sf::Packet& packet, Client& player)
return pair.second == timebases[0].second;
}))
{
int pid_to_blame = -1;
int pid_to_blame = 0;
for (auto pair : timebases)
{
if (std::all_of(timebases.begin(), timebases.end(), [&](std::pair<PlayerId, u64> other) {
@ -1718,7 +1718,7 @@ bool NetPlayServer::CompressBufferIntoPacket(const std::vector<u8>& in_buffer, s
return true;
}
void NetPlayServer::SendFirstReceivedToHost(const PadMapping map, const bool state)
void NetPlayServer::SendFirstReceivedToHost(const PadIndex map, const bool state)
{
sf::Packet pac;
pac << static_cast<MessageId>(NP_MSG_PAD_FIRST_RECEIVED);

2
Source/Core/Core/NetPlayServer.h
View File

@ -118,7 +118,7 @@ private:
void CheckSyncAndStartGame();
bool CompressFileIntoPacket(const std::string& file_path, sf::Packet& packet);
bool CompressBufferIntoPacket(const std::vector<u8>& in_buffer, sf::Packet& packet);
void SendFirstReceivedToHost(PadMapping map, bool state);
void SendFirstReceivedToHost(PadIndex map, bool state);
u64 GetInitialNetPlayRTC() const;

6
Source/Core/DolphinQt/NetPlay/PadMappingDialog.cpp
View File

@ -90,7 +90,7 @@ int PadMappingDialog::exec()
const auto index = gc ? m_pad_mapping[i] : m_wii_mapping[i];
combo->setCurrentIndex(index == -1 ? 0 : index);
combo->setCurrentIndex(index);
}
}
@ -114,7 +114,7 @@ void PadMappingDialog::OnMappingChanged()
int gc_id = m_gc_boxes[i]->currentIndex();
int wii_id = m_wii_boxes[i]->currentIndex();
m_pad_mapping[i] = gc_id > 0 ? m_players[gc_id - 1]->pid : -1;
m_wii_mapping[i] = wii_id > 0 ? m_players[wii_id - 1]->pid : -1;
m_pad_mapping[i] = gc_id > 0 ? m_players[gc_id - 1]->pid : 0;
m_wii_mapping[i] = wii_id > 0 ? m_players[wii_id - 1]->pid : 0;
}
}