refactor: Remove redundant void argument from functions (2)

Core/GameEngine/Include/GameClient/View.h

@@ -408,8 +408,8 @@ class ViewDummy : public View
 	}
 	virtual void screenToTerrain( const ICoord2D *screen, Coord3D *world ) override {}
 	virtual void screenToWorldAtZ( const ICoord2D *s, Coord3D *w, Real z ) override {}
-	virtual void drawView( void ) override {}
-	virtual void updateView(void) override {}
+	virtual void drawView() override {}
+	virtual void updateView() override {}
 	virtual void stepView() override {}
 	virtual void setGuardBandBias( const Coord2D *gb ) override {}
 	virtual Bool isDoingScriptedCamera() override { return false; }

Core/GameEngine/Source/GameNetwork/GameSpy/Thread/PeerThread.cpp

@@ -586,7 +586,7 @@ void GameSpyPeerMessageQueue::addRequest( const PeerRequest& req )
 	m_requests.push(req);
 }
 
-//PeerRequest GameSpyPeerMessageQueue::getRequest( void )
+//PeerRequest GameSpyPeerMessageQueue::getRequest()
 Bool GameSpyPeerMessageQueue::getRequest( PeerRequest& req )
 {
 	MutexClass::LockClass m(m_requestMutex, 0);
@@ -612,7 +612,7 @@ void GameSpyPeerMessageQueue::addResponse( const PeerResponse& resp )
 	m_responses.push(resp);
 }
 
-//PeerResponse GameSpyPeerMessageQueue::getResponse( void )
+//PeerResponse GameSpyPeerMessageQueue::getResponse()
 Bool GameSpyPeerMessageQueue::getResponse( PeerResponse& resp )
 {
 	MutexClass::LockClass m(m_responseMutex, 0);

Core/GameEngine/Source/GameNetwork/GameSpy/Thread/PingThread.cpp

@@ -333,7 +333,7 @@ void PingThreadClass::Thread_Function()
 //-------------------------------------------------------------------------
 //-------------------------------------------------------------------------
 
-HANDLE WINAPI IcmpCreateFile(VOID); /* INVALID_HANDLE_VALUE on error */
+HANDLE WINAPI IcmpCreateFile(); /* INVALID_HANDLE_VALUE on error */
 BOOL WINAPI IcmpCloseHandle(HANDLE IcmpHandle); /* FALSE on error */
 
 /* Note 2: For the most part, you can refer to RFC 791 for detials
@@ -442,7 +442,7 @@ Int PingThreadClass::doPing(UnsignedInt IP, Int timeout)
    char achRepData[sizeof(ICMPECHO) + BUFSIZE];
 
 
-   HANDLE ( WINAPI *lpfnIcmpCreateFile )( VOID ) = nullptr;
+   HANDLE ( WINAPI *lpfnIcmpCreateFile )() = nullptr;
    BOOL ( WINAPI *lpfnIcmpCloseHandle )( HANDLE ) = nullptr;
    DWORD (WINAPI *lpfnIcmpSendEcho)(HANDLE, DWORD, LPVOID, WORD, LPVOID,
                                     LPVOID, DWORD, DWORD) = nullptr;

Core/GameEngine/Source/GameNetwork/WOLBrowser/WebBrowser.cpp

@@ -265,7 +265,7 @@ STDMETHODIMP WebBrowser::QueryInterface(REFIID iid, void** ppv) IUNKNOWN_NOEXCEP
 *
 ******************************************************************************/
 
-ULONG STDMETHODCALLTYPE WebBrowser::AddRef(void) IUNKNOWN_NOEXCEPT
+ULONG STDMETHODCALLTYPE WebBrowser::AddRef() IUNKNOWN_NOEXCEPT
 {
 	return ++mRefCount;
 }
@@ -285,7 +285,7 @@ ULONG STDMETHODCALLTYPE WebBrowser::AddRef(void) IUNKNOWN_NOEXCEPT
 *
 ******************************************************************************/
 
-ULONG STDMETHODCALLTYPE WebBrowser::Release(void) IUNKNOWN_NOEXCEPT
+ULONG STDMETHODCALLTYPE WebBrowser::Release() IUNKNOWN_NOEXCEPT
 {
 	DEBUG_ASSERTCRASH(mRefCount > 0, ("Negative reference count"));
 	--mRefCount;

Core/GameEngineDevice/Source/W3DDevice/GameClient/W3DView.cpp

@@ -2530,7 +2530,7 @@ void W3DView::initHeightForMap()
 
 //-------------------------------------------------------------------------------------------------
 //-------------------------------------------------------------------------------------------------
-void W3DView::resetPivotToGround( void )
+void W3DView::resetPivotToGround()
 {
 	m_groundLevel = getHeightAroundPos(m_pos.x, m_pos.y);
 	m_cameraAreaConstraintsValid = false; // possible ground level change invalidates camera constraints

Core/Libraries/Source/WWVegas/WWDownload/FTP.cpp

@@ -555,7 +555,7 @@
 
 /*$_Function****************************************************************
 * $_Name        :
-*	STDMETHOD Cftp::LogoffFromServer( void )
+*	STDMETHOD Cftp::LogoffFromServer()
 *
 * $_Description :
 *	Logs off from a server.  LoginToServer() must have succeeded before this
@@ -665,7 +665,7 @@
 	char ext[ 10 ];
 
 	if (m_findStart==0)
       m_findStart=time(nullptr);
 
 	if((time(nullptr)-m_findStart) > 30)  // try for 30 seconds
 	{
@@ -1069,7 +1069,7 @@
 
 /*$_Function****************************************************************
 * $_Name        :
-*	int Cftp::SendNewPort( void )
+*	int Cftp::SendNewPort()
 *
 * $_Description :
 *	Sends a PORT command to the server specifying a new ephemeral port to

Core/Tools/DebugWindow/DebugWindow.cpp

@@ -76,7 +76,7 @@ CDebugWindowApp::CDebugWindowApp()
 	m_DialogWindow = nullptr;
 }
 
-DebugWindowDialog* CDebugWindowApp::GetDialogWindow(void)
+DebugWindowDialog* CDebugWindowApp::GetDialogWindow()
 {
 	return m_DialogWindow;
 }
@@ -96,7 +96,7 @@ CDebugWindowApp::~CDebugWindowApp()
 
 CDebugWindowApp theApp;
 
-void __declspec(dllexport) CreateDebugDialog(void)
+void __declspec(dllexport) CreateDebugDialog()
 {
 	AFX_MANAGE_STATE(AfxGetStaticModuleState( ));
 
@@ -112,7 +112,7 @@ void __declspec(dllexport) CreateDebugDialog(void)
 	theApp.SetDialogWindow(tmpWnd);
 }
 
-void __declspec(dllexport) DestroyDebugDialog(void)
+void __declspec(dllexport) DestroyDebugDialog()
 {
 	AFX_MANAGE_STATE(AfxGetStaticModuleState( ));
 	DebugWindowDialog* tmpWnd = theApp.GetDialogWindow();
@@ -125,7 +125,7 @@ void __declspec(dllexport) DestroyDebugDialog(void)
 
 }
 
-bool __declspec(dllexport) CanAppContinue(void)
+bool __declspec(dllexport) CanAppContinue()
 {
 	AFX_MANAGE_STATE(AfxGetStaticModuleState( ));
 
@@ -138,7 +138,7 @@ bool __declspec(dllexport) CanAppContinue(void)
 	return pDbg->CanProceed();
 }
 
-void __declspec(dllexport) ForceAppContinue(void)
+void __declspec(dllexport) ForceAppContinue()
 {
 	AFX_MANAGE_STATE(AfxGetStaticModuleState( ));
 
@@ -151,7 +151,7 @@ void __declspec(dllexport) ForceAppContinue(void)
 	pDbg->ForceContinue();
 }
 
-bool __declspec(dllexport) RunAppFast(void)
+bool __declspec(dllexport) RunAppFast()
 {
 	AFX_MANAGE_STATE(AfxGetStaticModuleState( ));
 

Core/Tools/DebugWindow/DebugWindow.h

@@ -39,7 +39,7 @@ class CDebugWindowApp : public CWinApp
 	public:
 		CDebugWindowApp();
 		~CDebugWindowApp();
-		DebugWindowDialog* GetDialogWindow(void);
+		DebugWindowDialog* GetDialogWindow();
 		void SetDialogWindow(DebugWindowDialog* pWnd);
 
 	protected:

Core/Tools/DebugWindow/DebugWindowDialog.cpp

@@ -33,18 +33,18 @@ int DebugWindowDialog::OnCreate(LPCREATESTRUCT lpCreateStruct)
 	return CDialog::OnCreate(lpCreateStruct);
 }
 
-HWND DebugWindowDialog::GetMainWndHWND(void)
+HWND DebugWindowDialog::GetMainWndHWND()
 {
 	return mMainWndHWND;
 }
 
-void DebugWindowDialog::ForcePause(void)
+void DebugWindowDialog::ForcePause()
 {
 	mNumberOfStepsAllowed = 0;
 	_UpdatePauseButton();
 }
 
-void DebugWindowDialog::ForceContinue(void)
+void DebugWindowDialog::ForceContinue()
 {
 	mNumberOfStepsAllowed = -1;
 	_UpdatePauseButton();
@@ -88,7 +88,7 @@ void DebugWindowDialog::OnClearWindows()
 	_RebuildVarsString();
 }
 
-bool DebugWindowDialog::CanProceed(void)
+bool DebugWindowDialog::CanProceed()
 {
 	if (mNumberOfStepsAllowed < 0) {
 		return true;
@@ -104,7 +104,7 @@ bool DebugWindowDialog::CanProceed(void)
 	return true;
 }
 
-bool DebugWindowDialog::RunAppFast(void)
+bool DebugWindowDialog::RunAppFast()
 {
 	return mRunFast;
 }
@@ -147,7 +147,7 @@ void DebugWindowDialog::SetFrameNumber(int frameNumber)
 	}
 }
 
-void DebugWindowDialog::_RebuildVarsString(void)
+void DebugWindowDialog::_RebuildVarsString()
 {
 	int cursorPosBeg, cursorPosEnd;
 	((CEdit*)GetDlgItem(IDC_Variables))->GetSel(cursorPosBeg, cursorPosEnd);
@@ -168,7 +168,7 @@ void DebugWindowDialog::_RebuildVarsString(void)
 	((CEdit*)GetDlgItem(IDC_Variables))->GetSel(cursorPosBeg, cursorPosEnd);
 }
 
-void DebugWindowDialog::_RebuildMesgString(void)
+void DebugWindowDialog::_RebuildMesgString()
 {
 	mMessagesString = "";
 
@@ -183,7 +183,7 @@ void DebugWindowDialog::_RebuildMesgString(void)
 	((CEdit*)GetDlgItem(IDC_Messages))->SetSel(mMessagesString.length(), mMessagesString.length(), false);
 }
 
-void DebugWindowDialog::_UpdatePauseButton(void)
+void DebugWindowDialog::_UpdatePauseButton()
 {
 	// huh huh huhuh he said pButt
 	CButton* pButt = (CButton*) GetDlgItem(IDC_Pause);

Core/Tools/DebugWindow/DebugWindowDialog.h

@@ -36,14 +36,14 @@ class DebugWindowDialog : public CDialog
 		enum {IDD = IDD_DebugWindow};
 		DebugWindowDialog(UINT nIDTemplate = DebugWindowDialog::IDD, CWnd* pParentWnd = nullptr);
 
-		bool CanProceed(void);
-		bool RunAppFast(void);
+		bool CanProceed();
+		bool RunAppFast();
 		void AppendMessage(const std::string& messageToAppend);
 		void AdjustVariable(const std::string& varName, const std::string& varValue);
 		void SetFrameNumber(int frameNumber);
-		HWND GetMainWndHWND(void);
-		void ForcePause(void);
-		void ForceContinue(void);
+		HWND GetMainWndHWND();
+		void ForcePause();
+		void ForceContinue();
 
 	// This var shouldn't be here, but honsestly...
 	protected:
@@ -62,9 +62,9 @@ class DebugWindowDialog : public CDialog
 		VecPairString	mVariables;
 		VecString		mMessages;
 
-		void _RebuildVarsString(void);
-		void _RebuildMesgString(void);
-		void _UpdatePauseButton(void);
+		void _RebuildVarsString();
+		void _RebuildMesgString();
+		void _UpdatePauseButton();
 
 	protected:
 		afx_msg int OnCreate(LPCREATESTRUCT lpCreateStruct);

Core/Tools/DebugWindow/DebugWindowExport.h

@@ -21,19 +21,19 @@
 // Declared extern C to prevent name mangling, which makes life very unhappy
 extern "C" {
 	// Called to create the dialog
-	void __declspec(dllexport) CreateDebugDialog(void);
+	void __declspec(dllexport) CreateDebugDialog();
 
 	// Called to (not surprisingly) destroy the dialog (and free the resources)
-	void __declspec(dllexport) DestroyDebugDialog(void);
+	void __declspec(dllexport) DestroyDebugDialog();
 
 	// Call this each frame to determine whether to continue or not
-	bool __declspec(dllexport) CanAppContinue(void);
+	bool __declspec(dllexport) CanAppContinue();
 
 	// Call this to force the app to continue. (Unpause if necessary.)
-	void __declspec(dllexport) ForceAppContinue(void);
+	void __declspec(dllexport) ForceAppContinue();
 
 	// Call this to tell the app to run really fast
-	bool __declspec(dllexport) RunAppFast(void);
+	bool __declspec(dllexport) RunAppFast();
 
 	// Call this to add a message to the script window
 	void __declspec(dllexport) AppendMessage(const char* messageToPass);

Core/Tools/ImagePacker/Include/ImageDirectory.h

@@ -69,8 +69,8 @@ class ImageDirectory
 ///////////////////////////////////////////////////////////////////////////////
 // INLINING ///////////////////////////////////////////////////////////////////
 ///////////////////////////////////////////////////////////////////////////////
-inline ImageDirectory::~ImageDirectory( void ) { delete m_path; }
-inline ImageDirectory::ImageDirectory( void )
+inline ImageDirectory::~ImageDirectory() { delete m_path; }
+inline ImageDirectory::ImageDirectory()
 {
 	m_path = nullptr;
 	m_next = nullptr;