diff --git a/Source/Core/Common/CDUtils.cpp b/Source/Core/Common/CDUtils.cpp index 7e1ebebd27..569ceb3c3b 100644 --- a/Source/Core/Common/CDUtils.cpp +++ b/Source/Core/Common/CDUtils.cpp @@ -2,7 +2,6 @@ #include "CDUtils.h" #include "Common.h" -#include "StringUtil.h" #include // for std::unique_ptr #ifdef _WIN32 @@ -151,10 +150,10 @@ static struct }; // Returns true if a device is a block or char device and not a symbolic link -bool is_device(const std::string& source_name) +bool is_device(const char *source_name) { struct stat buf; - if (0 != lstat(source_name.c_str(), &buf)) + if (0 != lstat(source_name, &buf)) return false; return ((S_ISBLK(buf.st_mode) || S_ISCHR(buf.st_mode)) && @@ -162,15 +161,17 @@ bool is_device(const std::string& source_name) } // Check a device to see if it is a DVD/CD-ROM drive -static bool is_cdrom(const std::string& drive, char *mnttype) +static bool is_cdrom(const char *drive, char *mnttype) { + bool is_cd=false; + int cdfd; + // Check if the device exists if (!is_device(drive)) return(false); - bool is_cd=false; // If it does exist, verify that it is a cdrom/dvd drive - int cdfd = open(drive.c_str(), (O_RDONLY|O_NONBLOCK), 0); + cdfd = open(drive, (O_RDONLY|O_NONBLOCK), 0); if ( cdfd >= 0 ) { #ifdef __linux__ @@ -185,16 +186,21 @@ static bool is_cdrom(const std::string& drive, char *mnttype) // Returns a pointer to an array of strings with the device names std::vector cdio_get_devices () { + unsigned int i; + char drive[40]; std::vector drives; + // Scan the system for DVD/CD-ROM drives. - for (unsigned int i = 0; checklist[i].format; ++i) + for ( i=0; checklist[i].format; ++i ) { - for (unsigned int j = checklist[i].num_min; j <= checklist[i].num_max; ++j) + unsigned int j; + for ( j=checklist[i].num_min; j<=checklist[i].num_max; ++j ) { - std::string drive = StringFromFormat(checklist[i].format, j); - if ( (is_cdrom(drive.c_str(), NULL)) > 0 ) + sprintf(drive, checklist[i].format, j); + if ( (is_cdrom(drive, NULL)) > 0 ) { - drives.push_back(std::move(drive)); + std::string str = drive; + drives.push_back(str); } } } @@ -216,5 +222,17 @@ bool cdio_is_cdrom(std::string device) #endif std::vector devices = cdio_get_devices(); - return std::find(devices.begin(), devices.end(), device) != devices.end();; + bool res = false; + for (auto& odevice : devices) + { + if (strncmp(odevice.c_str(), device.c_str(), MAX_PATH) == 0) + { + res = true; + break; + } + } + + devices.clear(); + return res; } + diff --git a/Source/Core/Common/IniFile.h b/Source/Core/Common/IniFile.h index 005f866f65..abfb58b0a6 100644 --- a/Source/Core/Common/IniFile.h +++ b/Source/Core/Common/IniFile.h @@ -140,10 +140,6 @@ public: void SetLines(const char* sectionName, const std::vector &lines); bool GetLines(const char* sectionName, std::vector& lines, const bool remove_comments = true) const; - inline bool DeleteKey(const char* sectionName, const std::string& key) - { - return DeleteKey(sectionName, key.c_str()); - } bool DeleteKey(const char* sectionName, const char* key); bool DeleteSection(const char* sectionName); diff --git a/Source/Core/Common/Log.h b/Source/Core/Common/Log.h index 8c7a1eb331..b0f3519579 100644 --- a/Source/Core/Common/Log.h +++ b/Source/Core/Common/Log.h @@ -77,8 +77,6 @@ enum LOG_LEVELS LDEBUG = DEBUG_LEVEL, }; -static const char LOG_LEVEL_TO_CHAR[7] = "-NEWID"; - #define LOGTYPES_LEVELS LogTypes::LOG_LEVELS #define LOGTYPES_TYPE LogTypes::LOG_TYPE diff --git a/Source/Core/Common/LogManager.cpp b/Source/Core/Common/LogManager.cpp index 5fe218b2f4..50c6ab400a 100644 --- a/Source/Core/Common/LogManager.cpp +++ b/Source/Core/Common/LogManager.cpp @@ -114,6 +114,7 @@ void LogManager::Log(LogTypes::LOG_LEVELS level, LogTypes::LOG_TYPE type, const char *file, int line, const char *format, va_list args) { char temp[MAX_MSGLEN]; + char msg[MAX_MSGLEN * 2]; LogContainer *log = m_Log[type]; if (!log->IsEnabled() || level > log->GetLevel() || ! log->HasListeners()) @@ -121,15 +122,15 @@ void LogManager::Log(LogTypes::LOG_LEVELS level, LogTypes::LOG_TYPE type, CharArrayFromFormatV(temp, MAX_MSGLEN, format, args); - std::string msg = StringFromFormat("%s %s:%u %c[%s]: %s\n", - Common::Timer::GetTimeFormatted().c_str(), - file, line, - LogTypes::LOG_LEVEL_TO_CHAR[(int)level], - log->GetShortName(), temp); + static const char level_to_char[7] = "-NEWID"; + sprintf(msg, "%s %s:%u %c[%s]: %s\n", + Common::Timer::GetTimeFormatted().c_str(), + file, line, level_to_char[(int)level], + log->GetShortName(), temp); #ifdef ANDROID - Host_SysMessage(msg.c_str()); + Host_SysMessage(msg); #endif - log->Trigger(level, msg.c_str()); + log->Trigger(level, msg); } void LogManager::Init() diff --git a/Source/Core/Common/MemArena.cpp b/Source/Core/Common/MemArena.cpp index 699005dd52..e7f3911e91 100644 --- a/Source/Core/Common/MemArena.cpp +++ b/Source/Core/Common/MemArena.cpp @@ -6,7 +6,6 @@ #include "MemoryUtil.h" #include "MemArena.h" -#include "StringUtil.h" #ifdef _WIN32 #include @@ -58,21 +57,20 @@ void MemArena::GrabLowMemSpace(size_t size) return; } #else + char fn[64]; for (int i = 0; i < 10000; i++) { - std::string file_name = StringFromFormat("dolphinmem.%d", i); - fd = shm_open(file_name.c_str(), O_RDWR | O_CREAT | O_EXCL, 0600); + sprintf(fn, "dolphinmem.%d", i); + fd = shm_open(fn, O_RDWR | O_CREAT | O_EXCL, 0600); if (fd != -1) - { - shm_unlink(file_name.c_str()); break; - } - else if (errno != EEXIST) + if (errno != EEXIST) { ERROR_LOG(MEMMAP, "shm_open failed: %s", strerror(errno)); return; } } + shm_unlink(fn); if (ftruncate(fd, size) < 0) ERROR_LOG(MEMMAP, "Failed to allocate low memory space"); #endif diff --git a/Source/Core/Common/NandPaths.cpp b/Source/Core/Common/NandPaths.cpp index 52e3904536..3ca5550d40 100644 --- a/Source/Core/Common/NandPaths.cpp +++ b/Source/Core/Common/NandPaths.cpp @@ -12,16 +12,20 @@ namespace Common std::string GetTicketFileName(u64 _titleID) { - return StringFromFormat("%sticket/%08x/%08x.tik", - File::GetUserPath(D_WIIUSER_IDX).c_str(), - (u32)(_titleID >> 32), (u32)_titleID); + char TicketFilename[1024]; + sprintf(TicketFilename, "%sticket/%08x/%08x.tik", + File::GetUserPath(D_WIIUSER_IDX).c_str(), (u32)(_titleID >> 32), (u32)_titleID); + + return TicketFilename; } std::string GetTitleDataPath(u64 _titleID) { - return StringFromFormat("%stitle/%08x/%08x/data/", - File::GetUserPath(D_WIIUSER_IDX).c_str(), - (u32)(_titleID >> 32), (u32)_titleID); + char path[1024]; + sprintf(path, "%stitle/%08x/%08x/data/", + File::GetUserPath(D_WIIUSER_IDX).c_str(), (u32)(_titleID >> 32), (u32)_titleID); + + return path; } std::string GetTMDFileName(u64 _titleID) @@ -30,9 +34,11 @@ std::string GetTMDFileName(u64 _titleID) } std::string GetTitleContentPath(u64 _titleID) { - return StringFromFormat("%stitle/%08x/%08x/content/", - File::GetUserPath(D_WIIUSER_IDX).c_str(), - (u32)(_titleID >> 32), (u32)_titleID); + char ContentPath[1024]; + sprintf(ContentPath, "%stitle/%08x/%08x/content/", + File::GetUserPath(D_WIIUSER_IDX).c_str(), (u32)(_titleID >> 32), (u32)_titleID); + + return ContentPath; } bool CheckTitleTMD(u64 _titleID) diff --git a/Source/Core/Common/Timer.cpp b/Source/Core/Common/Timer.cpp index 91a2a2d3ad..267755fb17 100644 --- a/Source/Core/Common/Timer.cpp +++ b/Source/Core/Common/Timer.cpp @@ -167,23 +167,27 @@ u64 Timer::GetLocalTimeSinceJan1970() std::string Timer::GetTimeFormatted() { time_t sysTime; - time(&sysTime); - - struct tm * gmTime = localtime(&sysTime); - + struct tm * gmTime; + char formattedTime[13]; char tmp[13]; + + time(&sysTime); + gmTime = localtime(&sysTime); + strftime(tmp, 6, "%M:%S", gmTime); // Now tack on the milliseconds #ifdef _WIN32 struct timeb tp; (void)::ftime(&tp); - return StringFromFormat("%s:%03i", tmp, tp.millitm); + sprintf(formattedTime, "%s:%03i", tmp, tp.millitm); #else struct timeval t; (void)gettimeofday(&t, NULL); - return StringFromFormat("%s:%03d", tmp, (int)(t.tv_usec / 1000)); + sprintf(formattedTime, "%s:%03d", tmp, (int)(t.tv_usec / 1000)); #endif + + return std::string(formattedTime); } // Returns a timestamp with decimals for precise time comparisons diff --git a/Source/Core/Core/ConfigManager.cpp b/Source/Core/Core/ConfigManager.cpp index afeeaca842..f28dfa5d21 100644 --- a/Source/Core/Core/ConfigManager.cpp +++ b/Source/Core/Core/ConfigManager.cpp @@ -151,14 +151,18 @@ void SConfig::SaveSettings() ini.Get("General", "GCMPathes", &oldPaths, 0); for (int i = numPaths; i < oldPaths; i++) { - ini.DeleteKey("General", StringFromFormat("GCMPath%i", i)); + char tmp[16]; + sprintf(tmp, "GCMPath%i", i); + ini.DeleteKey("General", tmp); } - ini.Set("General", "GCMPathes", numPaths); + ini.Set("General", "GCMPathes", numPaths); for (int i = 0; i < numPaths; i++) { - ini.Set("General", StringFromFormat("GCMPath%i", i).c_str(), m_ISOFolder[i]); + char tmp[16]; + sprintf(tmp, "GCMPath%i", i); + ini.Set("General", tmp, m_ISOFolder[i]); } ini.Set("General", "RecursiveGCMPaths", m_RecursiveISOFolder); @@ -245,10 +249,13 @@ void SConfig::SaveSettings() ini.Set("Core", "SlotB", m_EXIDevice[1]); ini.Set("Core", "SerialPort1", m_EXIDevice[2]); ini.Set("Core", "BBA_MAC", m_bba_mac); + char sidevicenum[16]; for (int i = 0; i < 4; ++i) { - ini.Set("Core", StringFromFormat("SIDevice%i", i).c_str(), m_SIDevice[i]); + sprintf(sidevicenum, "SIDevice%i", i); + ini.Set("Core", sidevicenum, m_SIDevice[i]); } + ini.Set("Core", "WiiSDCard", m_WiiSDCard); ini.Set("Core", "WiiKeyboard", m_WiiKeyboard); ini.Set("Core", "WiimoteContinuousScanning", m_WiimoteContinuousScanning); @@ -300,9 +307,11 @@ void SConfig::LoadSettings() { for (int i = 0; i < numGCMPaths; i++) { + char tmp[16]; + sprintf(tmp, "GCMPath%i", i); std::string tmpPath; - ini.Get("General", StringFromFormat("GCMPath%i", i).c_str(), &tmpPath, ""); - m_ISOFolder.push_back(std::move(tmpPath)); + ini.Get("General", tmp, &tmpPath, ""); + m_ISOFolder.push_back(tmpPath); } } @@ -401,10 +410,13 @@ void SConfig::LoadSettings() ini.Get("Core", "BBA_MAC", &m_bba_mac); ini.Get("Core", "TimeProfiling",&m_LocalCoreStartupParameter.bJITILTimeProfiling, false); ini.Get("Core", "OutputIR", &m_LocalCoreStartupParameter.bJITILOutputIR, false); + char sidevicenum[16]; for (int i = 0; i < 4; ++i) { - ini.Get("Core", StringFromFormat("SIDevice%i", i).c_str(), (u32*)&m_SIDevice[i], (i == 0) ? SIDEVICE_GC_CONTROLLER : SIDEVICE_NONE); + sprintf(sidevicenum, "SIDevice%i", i); + ini.Get("Core", sidevicenum, (u32*)&m_SIDevice[i], (i == 0) ? SIDEVICE_GC_CONTROLLER : SIDEVICE_NONE); } + ini.Get("Core", "WiiSDCard", &m_WiiSDCard, false); ini.Get("Core", "WiiKeyboard", &m_WiiKeyboard, false); ini.Get("Core", "WiimoteContinuousScanning", &m_WiimoteContinuousScanning, false); diff --git a/Source/Core/Core/DSP/DSPCodeUtil.cpp b/Source/Core/Core/DSP/DSPCodeUtil.cpp index 53e47b71cc..afaaa7d22e 100644 --- a/Source/Core/Core/DSP/DSPCodeUtil.cpp +++ b/Source/Core/Core/DSP/DSPCodeUtil.cpp @@ -110,6 +110,8 @@ void CodeToHeader(const std::vector &code, std::string _filename, // Pad with nops to 32byte boundary while (code_padded.size() & 0x7f) code_padded.push_back(0); + + char buffer[1024]; header.clear(); header.reserve(code_padded.size() * 4); header.append("#define NUM_UCODES 1\n\n"); @@ -123,7 +125,8 @@ void CodeToHeader(const std::vector &code, std::string _filename, { if (j && ((j & 15) == 0)) header.append("\n\t\t"); - header.append(StringFromFormat("0x%04x, ", code_padded[j])); + sprintf(buffer, "0x%04x, ", code_padded[j]); + header.append(buffer); } header.append("\n\t},\n"); @@ -134,6 +137,7 @@ void CodesToHeader(const std::vector *codes, const std::vector u32 numCodes, const char *name, std::string &header) { std::vector > codes_padded; + char buffer[1024]; u32 reserveSize = 0; for(u32 i = 0; i < numCodes; i++) { @@ -144,16 +148,20 @@ void CodesToHeader(const std::vector *codes, const std::vector reserveSize += (u32)codes_padded.at(i).size(); } + + header.clear(); header.reserve(reserveSize * 4); - header.append(StringFromFormat("#define NUM_UCODES %u\n\n", numCodes)); + sprintf(buffer, "#define NUM_UCODES %u\n\n", numCodes); + header.append(buffer); header.append("const char* UCODE_NAMES[NUM_UCODES] = {\n"); for (u32 i = 0; i < numCodes; i++) { std::string filename; if (! SplitPath(filenames->at(i), NULL, &filename, NULL)) filename = filenames->at(i); - header.append(StringFromFormat("\t\"%s\",\n", filename.c_str())); + sprintf(buffer, "\t\"%s\",\n", filename.c_str()); + header.append(buffer); } header.append("};\n\n"); header.append("const unsigned short dsp_code[NUM_UCODES][0x1000] = {\n"); @@ -168,7 +176,8 @@ void CodesToHeader(const std::vector *codes, const std::vector { if (j && ((j & 15) == 0)) header.append("\n\t\t"); - header.append(StringFromFormat("0x%04x, ", codes_padded.at(i).at(j))); + sprintf(buffer, "0x%04x, ", codes_padded.at(i).at(j)); + header.append(buffer); } header.append("\n\t},\n"); } diff --git a/Source/Core/Core/Debugger/Debugger_SymbolMap.cpp b/Source/Core/Core/Debugger/Debugger_SymbolMap.cpp index 3aa9294ef2..d4f1a4044f 100644 --- a/Source/Core/Core/Debugger/Debugger_SymbolMap.cpp +++ b/Source/Core/Core/Debugger/Debugger_SymbolMap.cpp @@ -151,7 +151,9 @@ void PrintDataBuffer(LogTypes::LOG_TYPE type, u8* _pData, size_t _Size, const ch std::string Temp; for (int i = 0; i < 16; i++) { - Temp.append(StringFromFormat("%02x ", _pData[j++])); + char Buffer[128]; + sprintf(Buffer, "%02x ", _pData[j++]); + Temp.append(Buffer); if (j >= _Size) break; diff --git a/Source/Core/Core/IPC_HLE/WII_IPC_HLE_Device.h b/Source/Core/Core/IPC_HLE/WII_IPC_HLE_Device.h index 9838716ae6..49249a2053 100644 --- a/Source/Core/Core/IPC_HLE/WII_IPC_HLE_Device.h +++ b/Source/Core/Core/IPC_HLE/WII_IPC_HLE_Device.h @@ -192,7 +192,9 @@ protected: std::string Temp; for (u32 j = 0; j < InBufferSize; j++) { - Temp += StringFromFormat("%02x ", Memory::Read_U8(InBuffer+j)); + char Buffer[128]; + sprintf(Buffer, "%02x ", Memory::Read_U8(InBuffer+j)); + Temp.append(Buffer); } GENERIC_LOG(LogType, LogTypes::LDEBUG, " Buffer: %s", Temp.c_str()); diff --git a/Source/Core/DiscIO/VolumeGC.cpp b/Source/Core/DiscIO/VolumeGC.cpp index f2d97462e9..325e200a34 100644 --- a/Source/Core/DiscIO/VolumeGC.cpp +++ b/Source/Core/DiscIO/VolumeGC.cpp @@ -54,7 +54,9 @@ std::string CVolumeGC::GetUniqueID() const std::string CVolumeGC::GetRevisionSpecificUniqueID() const { - return GetUniqueID() + StringFromFormat("r%d", GetRevision()); + char rev[16]; + sprintf(rev, "r%d", GetRevision()); + return GetUniqueID() + rev; } IVolume::ECountry CVolumeGC::GetCountry() const diff --git a/Source/Core/VideoCommon/VertexLoader.cpp b/Source/Core/VideoCommon/VertexLoader.cpp index d3806aeaa7..772bb1f1f8 100644 --- a/Source/Core/VideoCommon/VertexLoader.cpp +++ b/Source/Core/VideoCommon/VertexLoader.cpp @@ -600,10 +600,9 @@ void VertexLoader::CompileVertexTranslator() if (pFunc == 0) { - Host_SysMessage( - StringFromFormat("VertexLoader_Normal::GetFunction(%i %i %i %i) returned zero!", - m_VtxDesc.Normal, m_VtxAttr.NormalFormat, - m_VtxAttr.NormalElements, m_VtxAttr.NormalIndex3).c_str()); + char temp[256]; + sprintf(temp,"%i %i %i %i", m_VtxDesc.Normal, m_VtxAttr.NormalFormat, m_VtxAttr.NormalElements, m_VtxAttr.NormalIndex3); + Host_SysMessage("VertexLoader_Normal::GetFunction returned zero!"); } WriteCall(pFunc);