Revert "Merge pull request #49 from Parlane/sprintf_tidy"

Change broke the build on Debian stable.

This reverts commit 28755439b3, reversing
changes made to 64e01ec763.

Source/Core/Common/CDUtils.cpp

@@ -2,7 +2,6 @@
 
 #include "CDUtils.h"
 #include "Common.h"
-#include "StringUtil.h"
 
 #include <memory> // 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<std::string> cdio_get_devices ()
 {
+	unsigned int i;
+	char drive[40];
 	std::vector<std::string> 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);
+			sprintf(drive, checklist[i].format, j);
-			if ( (is_cdrom(drive.c_str(), NULL)) > 0 )
+			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<std::string> 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;
+}
+

Source/Core/Common/IniFile.h

@@ -140,10 +140,6 @@ public:
 	void SetLines(const char* sectionName, const std::vector<std::string> &lines);
 	bool GetLines(const char* sectionName, std::vector<std::string>& 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);
 

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
 

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",
+	static const char level_to_char[7] = "-NEWID";
+	sprintf(msg, "%s %s:%u %c[%s]: %s\n",
 		Common::Timer::GetTimeFormatted().c_str(),
-	                                   file, line, 
+		file, line, level_to_char[(int)level],
-	                                   LogTypes::LOG_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()

Source/Core/Common/MemArena.cpp

@@ -6,7 +6,6 @@
 
 #include "MemoryUtil.h"
 #include "MemArena.h"
-#include "StringUtil.h"
 
 #ifdef _WIN32
 #include <windows.h>
@@ -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);
+		sprintf(fn, "dolphinmem.%d", i);
-		fd = shm_open(file_name.c_str(), O_RDWR | O_CREAT | O_EXCL, 0600);
+		fd = shm_open(fn, O_RDWR | O_CREAT | O_EXCL, 0600);
 		if (fd != -1)
-		{
-			shm_unlink(file_name.c_str());
 			break;
-		}
+		if (errno != EEXIST)
-		else 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

Source/Core/Common/NandPaths.cpp

@@ -12,16 +12,20 @@ namespace Common
 
 std::string GetTicketFileName(u64 _titleID)
 {
-	return StringFromFormat("%sticket/%08x/%08x.tik",
+	char TicketFilename[1024];
-			File::GetUserPath(D_WIIUSER_IDX).c_str(), 
+	sprintf(TicketFilename, "%sticket/%08x/%08x.tik",
-			(u32)(_titleID >> 32), (u32)_titleID);
+			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/",
+	char path[1024];
-			File::GetUserPath(D_WIIUSER_IDX).c_str(),
+	sprintf(path, "%stitle/%08x/%08x/data/",
-			(u32)(_titleID >> 32), (u32)_titleID);
+			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/",
+	char ContentPath[1024];
-			File::GetUserPath(D_WIIUSER_IDX).c_str(), 
+	sprintf(ContentPath, "%stitle/%08x/%08x/content/",
-			(u32)(_titleID >> 32), (u32)_titleID);
+			File::GetUserPath(D_WIIUSER_IDX).c_str(), (u32)(_titleID >> 32), (u32)_titleID);
+
+	return ContentPath;
 }
 
 bool CheckTitleTMD(u64 _titleID)

Source/Core/Common/Timer.cpp

@@ -167,23 +167,27 @@ u64 Timer::GetLocalTimeSinceJan1970()
 std::string Timer::GetTimeFormatted()
 {
 	time_t sysTime;
-	time(&sysTime);
+	struct tm * gmTime;
-	
+	char formattedTime[13];
-	struct tm * gmTime = localtime(&sysTime);
-
 	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

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);
 
 	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, "");
+				ini.Get("General", tmp, &tmpPath, "");
-				m_ISOFolder.push_back(std::move(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);

Source/Core/Core/DSP/DSPCodeUtil.cpp

@@ -110,6 +110,8 @@ void CodeToHeader(const std::vector<u16> &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<u16> &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<u16> *codes, const std::vector<std::string>
 				   u32 numCodes, const char *name, std::string &header)
 {
 	std::vector<std::vector<u16> > codes_padded;
+	char buffer[1024];
 	u32 reserveSize = 0;
 	for(u32 i = 0; i < numCodes; i++)
 	{
@@ -144,16 +148,20 @@ void CodesToHeader(const std::vector<u16> *codes, const std::vector<std::string>
 
 		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<u16> *codes, const std::vector<std::string>
 		{
 			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");
 	}

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;

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());

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

Source/Core/VideoCommon/VertexLoader.cpp

@@ -600,10 +600,9 @@ void VertexLoader::CompileVertexTranslator()
 
 		if (pFunc == 0)
 		{
-			Host_SysMessage(
+			char temp[256];
-				StringFromFormat("VertexLoader_Normal::GetFunction(%i %i %i %i) returned zero!", 
+			sprintf(temp,"%i %i %i %i", m_VtxDesc.Normal, m_VtxAttr.NormalFormat, m_VtxAttr.NormalElements, m_VtxAttr.NormalIndex3);
-				m_VtxDesc.Normal, m_VtxAttr.NormalFormat, 
+			Host_SysMessage("VertexLoader_Normal::GetFunction returned zero!");
-				m_VtxAttr.NormalElements, m_VtxAttr.NormalIndex3).c_str());
 		}
 		WriteCall(pFunc);