piwalker/dolphin
1
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2025-07-24 14:49:42 -06:00
Code Issues Packages Projects Releases Wiki Activity

DSPCodeUtil: Don't return data via an out parameter

Browse Source 
We can just return directly in these cases.
This commit is contained in:
Lioncash
2018-03-25 18:29:11 -04:00
parent 5c83e18fbd
commit 0f25627614
3 changed files with 30 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
Source/Core/Core/DSP/DSPCodeUtil.cpp
View File

@ -102,42 +102,45 @@ bool Compare(const std::vector<u16>& code1, const std::vector<u16>& code2)
return code1.size() == code2.size() && code1.size() == count_equal;
}
void CodeToBinaryStringBE(const std::vector<u16>& code, std::string& str)
std::string CodeToBinaryStringBE(const std::vector<u16>& code)
{
str.resize(code.size() * 2);
std::string str(code.size() * 2, '\0');
for (size_t i = 0; i < code.size(); i++)
{
str[i * 2 + 0] = code[i] >> 8;
str[i * 2 + 1] = code[i] & 0xff;
}
return str;
}
void BinaryStringBEToCode(const std::string& str, std::vector<u16>& code)
std::vector<u16> BinaryStringBEToCode(const std::string& str)
{
code.resize(str.size() / 2);
std::vector<u16> code(str.size() / 2);
for (size_t i = 0; i < code.size(); i++)
{
code[i] = ((u16)(u8)str[i * 2 + 0] << 8) | ((u16)(u8)str[i * 2 + 1]);
}
return code;
}
bool LoadBinary(const std::string& filename, std::vector<u16>& code)
std::optional<std::vector<u16>> LoadBinary(const std::string& filename)
{
std::string buffer;
if (!File::ReadFileToString(filename, buffer))
return false;
return std::nullopt;
BinaryStringBEToCode(buffer, code);
return true;
return std::make_optional(BinaryStringBEToCode(buffer));
}
bool SaveBinary(const std::vector<u16>& code, const std::string& filename)
{
std::string buffer;
CodeToBinaryStringBE(code, buffer);
if (!File::WriteStringToFile(buffer, filename))
return false;
return true;
const std::string buffer = CodeToBinaryStringBE(code);
return File::WriteStringToFile(buffer, filename);
}
bool DumpDSPCode(const u8* code_be, int size_in_bytes, u32 crc)

7
Source/Core/Core/DSP/DSPCodeUtil.h
View File

@ -4,6 +4,7 @@
#pragma once
#include <optional>
#include <string>
#include <vector>
@ -16,11 +17,11 @@ bool Disassemble(const std::vector<u16>& code, bool line_numbers, std::string& t
bool Compare(const std::vector<u16>& code1, const std::vector<u16>& code2);
// Big-endian, for writing straight to file using File::WriteStringToFile.
void CodeToBinaryStringBE(const std::vector<u16>& code, std::string& str);
void BinaryStringBEToCode(const std::string& str, std::vector<u16>& code);
std::string CodeToBinaryStringBE(const std::vector<u16>& code);
std::vector<u16> BinaryStringBEToCode(const std::string& str);
// Load code (big endian binary).
bool LoadBinary(const std::string& filename, std::vector<u16>& code);
std::optional<std::vector<u16>> LoadBinary(const std::string& filename);
bool SaveBinary(const std::vector<u16>& code, const std::string& filename);
bool DumpDSPCode(const u8* code_be, int size_in_bytes, u32 crc);