Merge pull request #1752 from Buddybenj/clean-up

Clean Up

Installer/Installer.nsi

@@ -3,7 +3,7 @@
 !define BASE_DIR "..\Binary\${DOLPHIN_ARCH}"
 
 ; HM NIS Edit Wizard helper defines
-!define PRODUCT_PUBLISHER "Dolphin Development Team"
+!define PRODUCT_PUBLISHER "Dolphin Team"
 !define PRODUCT_WEB_SITE "https://dolphin-emu.org/"
 !define PRODUCT_DIR_REGKEY "Software\Microsoft\Windows\CurrentVersion\App Paths\${PRODUCT_NAME}.exe"
 !define PRODUCT_UNINST_KEY "Software\Microsoft\Windows\CurrentVersion\Uninstall\${PRODUCT_NAME}"

Source/Core/Core/DSPEmulator.cpp

@@ -10,11 +10,7 @@
 DSPEmulator *CreateDSPEmulator(bool HLE)
 {
 	if (HLE)
-	{
 		return new DSPHLE();
-	}
 	else
-	{
 		return new DSPLLE();
 }
-}

Source/Core/Core/HW/CPU.cpp

@@ -42,7 +42,6 @@ void CCPU::Run()
 
 	while (true)
 	{
-reswitch:
 		switch (PowerPC::GetState())
 		{
 		case PowerPC::CPU_RUNNING:
@@ -60,7 +59,7 @@ reswitch:
 			if (PowerPC::GetState() == PowerPC::CPU_POWERDOWN)
 				return;
 			if (PowerPC::GetState() != PowerPC::CPU_STEPPING)
-				goto reswitch;
+				continue;
 
 			//3: do a step
 			PowerPC::SingleStep();

Source/Core/Core/MemTools.cpp

@@ -136,7 +136,7 @@ static void ExceptionThread(mach_port_t port)
 	mach_msg_header_t *send_msg = nullptr;
 	mach_msg_size_t send_size = 0;
 	mach_msg_option_t option = MACH_RCV_MSG;
-	while (1)
+	while (true)
 	{
 		// If this isn't the first run, send the reply message.  Then, receive
 		// a message: either a mach_exception_raise_state RPC due to

Source/Core/Core/NetPlayProto.h

@@ -32,7 +32,7 @@ typedef std::vector<u8> NetWiimote;
 
 #define NETPLAY_VERSION  "Dolphin NetPlay 2014-01-08"
 
-const int NETPLAY_INITIAL_GCTIME = 1272737767;
+static const int NETPLAY_INITIAL_GCTIME = 1272737767;
 
 
 // messages

Source/Core/Core/PowerPC/PPCAnalyst.cpp

@@ -313,7 +313,7 @@ static void FindFunctionsAfterBLR(PPCSymbolDB *func_db)
 	{
 		while (true)
 		{
-			// skip zeroes that sometimes pad function to 16 byte boundary (eg. Donkey Kong Country Returns)
+			// skip zeroes that sometimes pad function to 16 byte boundary (e.g. Donkey Kong Country Returns)
 			while (Memory::Read_Instruction(location) == 0 && ((location & 0xf) != 0))
 				location += 4;
 			if (PPCTables::IsValidInstruction(Memory::Read_Instruction(location)))

Source/Core/Core/State.cpp

@@ -176,7 +176,8 @@ static int GetEmptySlot(std::map<double, int> m)
 				break;
 			}
 		}
-		if (!found) return i;
+		if (!found)
+			return i;
 	}
 	return -1;
 }

Source/Core/DiscIO/WbfsBlob.cpp

@@ -40,10 +40,8 @@ WbfsFileReader::WbfsFileReader(const std::string& filename)
 
 WbfsFileReader::~WbfsFileReader()
 {
-	for (u32 i = 0; i != m_files.size(); ++ i)
+	for (file_entry* entry : m_files)
-	{
+		delete entry;
-		delete m_files[i];
-	}
 
 	delete[] m_wlba_table;
 }
@@ -58,7 +56,7 @@ bool WbfsFileReader::OpenFiles(const std::string& filename)
 
 		// Replace last character with index (e.g. wbfs = wbf1)
 		std::string path = filename;
-		if (0 != m_total_files)
+		if (m_total_files != 0)
 		{
 			path[path.length() - 1] = '0' + m_total_files;
 		}
@@ -90,10 +88,7 @@ bool WbfsFileReader::ReadHeader()
 	m_hd_sector_size = 1ull << m_hd_sector_shift;
 
 	if (m_size != (m_hd_sector_count * m_hd_sector_size))
-	{
-		//printf("File size doesn't match expected size\n");
 		return false;
-	}
 
 	// Read wbfs cluster info
 	m_files[0]->file.ReadBytes(&m_wbfs_sector_shift, 1);
@@ -101,10 +96,7 @@ bool WbfsFileReader::ReadHeader()
 	m_wbfs_sector_count = m_size / m_wbfs_sector_size;
 
 	if (m_wbfs_sector_size < WII_SECTOR_SIZE)
-	{
-		//Setting this too low would case a very large memory allocation
 		return false;
-	}
 
 	m_blocks_per_disc = (WII_SECTOR_COUNT * WII_SECTOR_SIZE) / m_wbfs_sector_size;
 	m_disc_info_size = align(WII_DISC_HEADER_SIZE + m_blocks_per_disc * 2, m_hd_sector_size);
@@ -113,11 +105,8 @@ bool WbfsFileReader::ReadHeader()
 	m_files[0]->file.Seek(2, SEEK_CUR);
 	m_files[0]->file.ReadBytes(m_disc_table, 500);
 
-	if (0 == m_disc_table[0])
+	if (m_disc_table[0] == 0)
-	{
-		//printf("Game must be in 'slot 0'\n");
 		return false;
-	}
 
 	return true;
 }

Source/Core/DolphinQt/MainWindow.cpp

@@ -284,7 +284,7 @@ void DMainWindow::OnOpenDocs()
 
 void DMainWindow::OnOpenGitHub()
 {
-	QDesktopServices::openUrl(QUrl(SL("https://github.com/dolphin-emu/dolphin/")));
+	QDesktopServices::openUrl(QUrl(SL("https://github.com/dolphin-emu/dolphin")));
 }
 
 void DMainWindow::OnOpenSystemInfo()

Source/Core/DolphinWX/AboutDolphin.cpp

@@ -41,7 +41,7 @@ AboutDolphin::AboutDolphin(wxWindow *parent, wxWindowID id,
 				"Today Dolphin is an open source project with many\n"
 				"contributors, too many to list.\n"
 				"If interested, just go check out the project page at\n"
-				"https://github.com/dolphin-emu/dolphin/ .\n"
+				"https://github.com/dolphin-emu/dolphin .\n"
 				"\n"
 				"Special thanks to Bushing, Costis, CrowTRobo,\n"
 				"Marcan, Segher, Titanik, or9 and Hotquik for their\n"

Source/Core/DolphinWX/FrameTools.cpp

@@ -1415,7 +1415,7 @@ void CFrame::OnHelp(wxCommandEvent& event)
 		WxUtils::Launch("https://dolphin-emu.org/docs/guides/");
 		break;
 	case IDM_HELP_GITHUB:
-		WxUtils::Launch("https://github.com/dolphin-emu/dolphin/");
+		WxUtils::Launch("https://github.com/dolphin-emu/dolphin");
 		break;
 	}
 }

docs/ActionReplay/GCNCodeTypes.txt

@@ -29,7 +29,7 @@ NEWBIES OR WANNABES. Sorry.
 ***************  If you are an experienced, known (by me or gscentral 
 admins) code hacker, and you don't
                  understand this document, you may try to ask help using the 
-www.gscentral.com forums
+gscentral.com forums
                  (or PM me there).