SysConf: const-correctness

This commit is contained in:
Lioncash 2017-03-04 19:09:44 -05:00
parent 1ebd2cd7c3
commit 0e5a367ca3
3 changed files with 19 additions and 19 deletions

View File

@ -53,13 +53,13 @@ struct SSysConfEntry
std::vector<u8> data; std::vector<u8> data;
template <class T> template <class T>
T GetData() T GetData() const
{ {
T extracted_data; T extracted_data;
std::memcpy(&extracted_data, data.data(), sizeof(T)); std::memcpy(&extracted_data, data.data(), sizeof(T));
return extracted_data; return extracted_data;
} }
bool GetArrayData(u8* dest, u16 destSize) bool GetArrayData(u8* dest, u16 destSize) const
{ {
if (dest && destSize >= dataLength) if (dest && destSize >= dataLength)
{ {
@ -68,7 +68,7 @@ struct SSysConfEntry
} }
return false; return false;
} }
bool SetArrayData(u8* buffer, u16 bufferSize) bool SetArrayData(const u8* buffer, u16 bufferSize)
{ {
if (buffer) if (buffer)
{ {
@ -85,9 +85,9 @@ public:
SysConf(Common::FromWhichRoot root_type); SysConf(Common::FromWhichRoot root_type);
~SysConf(); ~SysConf();
bool IsValid() { return m_IsValid; } bool IsValid() const { return m_IsValid; }
template <class T> template <class T>
T GetData(const char* sectionName) T GetData(const char* sectionName) const
{ {
if (!m_IsValid) if (!m_IsValid)
{ {
@ -95,13 +95,13 @@ public:
return 0; return 0;
} }
std::vector<SSysConfEntry>::iterator index = m_Entries.begin(); auto index = m_Entries.cbegin();
for (; index < m_Entries.end() - 1; ++index) for (; index < m_Entries.cend() - 1; ++index)
{ {
if (strcmp(index->name, sectionName) == 0) if (strcmp(index->name, sectionName) == 0)
break; break;
} }
if (index == m_Entries.end() - 1) if (index == m_Entries.cend() - 1)
{ {
PanicAlertT("Section %s not found in SYSCONF", sectionName); PanicAlertT("Section %s not found in SYSCONF", sectionName);
return 0; return 0;
@ -110,35 +110,35 @@ public:
return index->GetData<T>(); return index->GetData<T>();
} }
bool GetArrayData(const char* sectionName, u8* dest, u16 destSize) bool GetArrayData(const char* sectionName, u8* dest, u16 destSize) const
{ {
if (!m_IsValid) if (!m_IsValid)
{ {
PanicAlertT("Trying to read from invalid SYSCONF"); PanicAlertT("Trying to read from invalid SYSCONF");
return 0; return false;
} }
std::vector<SSysConfEntry>::iterator index = m_Entries.begin(); auto index = m_Entries.cbegin();
for (; index < m_Entries.end() - 1; ++index) for (; index < m_Entries.cend() - 1; ++index)
{ {
if (strcmp(index->name, sectionName) == 0) if (strcmp(index->name, sectionName) == 0)
break; break;
} }
if (index == m_Entries.end() - 1) if (index == m_Entries.cend() - 1)
{ {
PanicAlertT("Section %s not found in SYSCONF", sectionName); PanicAlertT("Section %s not found in SYSCONF", sectionName);
return 0; return false;
} }
return index->GetArrayData(dest, destSize); return index->GetArrayData(dest, destSize);
} }
bool SetArrayData(const char* sectionName, u8* buffer, u16 bufferSize) bool SetArrayData(const char* sectionName, const u8* buffer, u16 bufferSize)
{ {
if (!m_IsValid) if (!m_IsValid)
return false; return false;
std::vector<SSysConfEntry>::iterator index = m_Entries.begin(); auto index = m_Entries.begin();
for (; index < m_Entries.end() - 1; ++index) for (; index < m_Entries.end() - 1; ++index)
{ {
if (strcmp(index->name, sectionName) == 0) if (strcmp(index->name, sectionName) == 0)
@ -159,7 +159,7 @@ public:
if (!m_IsValid) if (!m_IsValid)
return false; return false;
std::vector<SSysConfEntry>::iterator index = m_Entries.begin(); auto index = m_Entries.begin();
for (; index < m_Entries.end() - 1; ++index) for (; index < m_Entries.end() - 1; ++index)
{ {
if (strcmp(index->name, sectionName) == 0) if (strcmp(index->name, sectionName) == 0)

View File

@ -22,7 +22,7 @@ namespace HLE
{ {
constexpr u16 BT_INFO_SECTION_LENGTH = 0x460; constexpr u16 BT_INFO_SECTION_LENGTH = 0x460;
void BackUpBTInfoSection(SysConf* sysconf) void BackUpBTInfoSection(const SysConf* sysconf)
{ {
const std::string filename = File::GetUserPath(D_SESSION_WIIROOT_IDX) + DIR_SEP WII_BTDINF_BACKUP; const std::string filename = File::GetUserPath(D_SESSION_WIIROOT_IDX) + DIR_SEP WII_BTDINF_BACKUP;
if (File::Exists(filename)) if (File::Exists(filename))

View File

@ -18,7 +18,7 @@ namespace IOS
{ {
namespace HLE namespace HLE
{ {
void BackUpBTInfoSection(SysConf* sysconf); void BackUpBTInfoSection(const SysConf* sysconf);
void RestoreBTInfoSection(SysConf* sysconf); void RestoreBTInfoSection(SysConf* sysconf);
namespace Device namespace Device