mirror of
https://github.com/izzy2lost/dolphin.git
synced 2026-06-19 01:16:48 -07:00
WII_IPC_HLE_Device_usb: const correctness for function parameters
The data passed in isn't modified in these functions Also normalizes variables with prefixed underscores in the modified functions (and normalizes outliers to our current coding style), as single-underscore followed by any lowercased/uppercased character is reserved for use in the global namespace (it's a common misconception this is assumed to only be the case for underscores followed by a capital character, but this is only the case in C, not C++).
This commit is contained in:
@@ -145,17 +145,17 @@ void PrintCallstack(LogTypes::LOG_TYPE type, LogTypes::LOG_LEVELS level)
|
||||
});
|
||||
}
|
||||
|
||||
void PrintDataBuffer(LogTypes::LOG_TYPE type, u8* _pData, size_t _Size, const std::string& _title)
|
||||
void PrintDataBuffer(LogTypes::LOG_TYPE type, const u8* data, size_t size, const std::string& title)
|
||||
{
|
||||
GENERIC_LOG(type, LogTypes::LDEBUG, "%s", _title.c_str());
|
||||
for (u32 j = 0; j < _Size;)
|
||||
GENERIC_LOG(type, LogTypes::LDEBUG, "%s", title.c_str());
|
||||
for (u32 j = 0; j < size;)
|
||||
{
|
||||
std::string hex_line = "";
|
||||
for (int i = 0; i < 16; i++)
|
||||
{
|
||||
hex_line += StringFromFormat("%02x ", _pData[j++]);
|
||||
hex_line += StringFromFormat("%02x ", data[j++]);
|
||||
|
||||
if (j >= _Size)
|
||||
if (j >= size)
|
||||
break;
|
||||
}
|
||||
GENERIC_LOG(type, LogTypes::LDEBUG, " Data: %s", hex_line.c_str());
|
||||
|
||||
@@ -22,7 +22,7 @@ struct CallstackEntry
|
||||
bool GetCallstack(std::vector<CallstackEntry> &output);
|
||||
void PrintCallstack();
|
||||
void PrintCallstack(LogTypes::LOG_TYPE type, LogTypes::LOG_LEVELS level);
|
||||
void PrintDataBuffer(LogTypes::LOG_TYPE _Log, u8* _pData, size_t _Size, const std::string& _title);
|
||||
void PrintDataBuffer(LogTypes::LOG_TYPE type, const u8* data, size_t size, const std::string& title);
|
||||
void AddAutoBreakpoints();
|
||||
|
||||
|
||||
|
||||
File diff suppressed because it is too large
Load Diff
@@ -63,7 +63,7 @@ public:
|
||||
static void EnqueueReply(u32 CommandAddress);
|
||||
|
||||
// Send ACL data back to Bluetooth stack
|
||||
void SendACLPacket(u16 _ConnectionHandle, u8* _pData, u32 _Size);
|
||||
void SendACLPacket(u16 connection_handle, const u8* data, u32 size);
|
||||
|
||||
bool RemoteDisconnect(u16 _connectionHandle);
|
||||
|
||||
@@ -199,7 +199,7 @@ private:
|
||||
// Events
|
||||
void AddEventToQueue(const SQueuedEvent& _event);
|
||||
bool SendEventCommandStatus(u16 _Opcode);
|
||||
void SendEventCommandComplete(u16 _OpCode, void* _pData, u32 _DataSize);
|
||||
void SendEventCommandComplete(u16 opcode, const void* data, u32 data_size);
|
||||
bool SendEventInquiryResponse();
|
||||
bool SendEventInquiryComplete();
|
||||
bool SendEventRemoteNameReq(const bdaddr_t& _bd);
|
||||
@@ -221,49 +221,49 @@ private:
|
||||
void ExecuteHCICommandMessage(const SHCICommandMessage& _rCtrlMessage);
|
||||
|
||||
// OGF 0x01 - Link control commands and return parameters
|
||||
void CommandWriteInquiryMode(u8* _Input);
|
||||
void CommandWritePageScanType(u8* _Input);
|
||||
void CommandHostBufferSize(u8* _Input);
|
||||
void CommandInquiryCancel(u8* _Input);
|
||||
void CommandRemoteNameReq(u8* _Input);
|
||||
void CommandCreateCon(u8* _Input);
|
||||
void CommandAcceptCon(u8* _Input);
|
||||
void CommandReadClockOffset(u8* _Input);
|
||||
void CommandReadRemoteVerInfo(u8* _Input);
|
||||
void CommandReadRemoteFeatures(u8* _Input);
|
||||
void CommandAuthenticationRequested(u8* _Input);
|
||||
void CommandInquiry(u8* _Input);
|
||||
void CommandDisconnect(u8* _Input);
|
||||
void CommandLinkKeyNegRep(u8* _Input);
|
||||
void CommandLinkKeyRep(u8* _Input);
|
||||
void CommandDeleteStoredLinkKey(u8* _Input);
|
||||
void CommandChangeConPacketType(u8* _Input);
|
||||
void CommandWriteInquiryMode(const u8* input);
|
||||
void CommandWritePageScanType(const u8* input);
|
||||
void CommandHostBufferSize(const u8* input);
|
||||
void CommandInquiryCancel(const u8* input);
|
||||
void CommandRemoteNameReq(const u8* input);
|
||||
void CommandCreateCon(const u8* input);
|
||||
void CommandAcceptCon(const u8* input);
|
||||
void CommandReadClockOffset(const u8* input);
|
||||
void CommandReadRemoteVerInfo(const u8* input);
|
||||
void CommandReadRemoteFeatures(const u8* input);
|
||||
void CommandAuthenticationRequested(const u8* input);
|
||||
void CommandInquiry(const u8* input);
|
||||
void CommandDisconnect(const u8* input);
|
||||
void CommandLinkKeyNegRep(const u8* input);
|
||||
void CommandLinkKeyRep(const u8* input);
|
||||
void CommandDeleteStoredLinkKey(const u8* input);
|
||||
void CommandChangeConPacketType(const u8* input);
|
||||
|
||||
// OGF 0x02 - Link policy commands and return parameters
|
||||
void CommandWriteLinkPolicy(u8* _Input);
|
||||
void CommandSniffMode(u8* _Input);
|
||||
void CommandWriteLinkPolicy(const u8* input);
|
||||
void CommandSniffMode(const u8* input);
|
||||
|
||||
// OGF 0x03 - Host Controller and Baseband commands and return parameters
|
||||
void CommandReset(u8* _Input);
|
||||
void CommandWriteLocalName(u8* _Input);
|
||||
void CommandWritePageTimeOut(u8* _Input);
|
||||
void CommandWriteScanEnable(u8* _Input);
|
||||
void CommandWriteUnitClass(u8* _Input);
|
||||
void CommandReadStoredLinkKey(u8* _Input);
|
||||
void CommandWritePinType(u8* _Input);
|
||||
void CommandSetEventFilter(u8* _Input);
|
||||
void CommandWriteInquiryScanType(u8* _Input);
|
||||
void CommandWriteLinkSupervisionTimeout(u8* _Input);
|
||||
void CommandReset(const u8* input);
|
||||
void CommandWriteLocalName(const u8* input);
|
||||
void CommandWritePageTimeOut(const u8* input);
|
||||
void CommandWriteScanEnable(const u8* input);
|
||||
void CommandWriteUnitClass(const u8* input);
|
||||
void CommandReadStoredLinkKey(const u8* input);
|
||||
void CommandWritePinType(const u8* input);
|
||||
void CommandSetEventFilter(const u8* input);
|
||||
void CommandWriteInquiryScanType(const u8* input);
|
||||
void CommandWriteLinkSupervisionTimeout(const u8* input);
|
||||
|
||||
// OGF 0x04 - Informational commands and return parameters
|
||||
void CommandReadBufferSize(u8* _Input);
|
||||
void CommandReadLocalVer(u8* _Input);
|
||||
void CommandReadLocalFeatures(u8* _Input);
|
||||
void CommandReadBDAdrr(u8* _Input);
|
||||
void CommandReadBufferSize(const u8* input);
|
||||
void CommandReadLocalVer(const u8* input);
|
||||
void CommandReadLocalFeatures(const u8* input);
|
||||
void CommandReadBDAdrr(const u8* input);
|
||||
|
||||
// OGF 0x3F - Vendor specific
|
||||
void CommandVendorSpecific_FC4C(u8* _Input, u32 _Size);
|
||||
void CommandVendorSpecific_FC4F(u8* _Input, u32 _Size);
|
||||
void CommandVendorSpecific_FC4C(const u8* input, u32 size);
|
||||
void CommandVendorSpecific_FC4F(const u8* input, u32 size);
|
||||
|
||||
static void DisplayDisconnectMessage(const int wiimoteNumber, const int reason);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user