[Common/Core/Windows] Removed excess check pointer before delete or free()

Common/Data/Collections/FastVec.h

@@ -20,7 +20,7 @@ public:
 		data_ = (T *)malloc(initialCapacity * sizeof(T));
 		_assert_(data_ != nullptr);
 	}
-	~FastVec() { if (data_) free(data_); }
+	~FastVec() { free(data_); }
 
 	T &push_uninitialized() {
 		if (size_ < capacity_) {

Common/Data/Format/JSONReader.h

@@ -61,7 +61,6 @@ public:
 	}
 
 	~JsonReader() {
-		if (buffer_)
 		free(buffer_);
 	}
 

Common/GPU/OpenGL/GLRenderManager.h

@@ -122,10 +122,8 @@ public:
 		if (program) {
 			glDeleteProgram(program);
 		}
-		if (locData_) {
 		delete locData_;
 	}
-	}
 	struct Semantic {
 		int location;
 		const char *attrib;

Common/GPU/OpenGL/GLSLProgram.cpp

@@ -63,9 +63,7 @@ struct AutoCharArrayBuf {
 		buf_ = nullptr;
 	}
 	void reset(char *buf) {
-		if (buf_) {
 		delete[] buf_;
-		}
 		buf_ = buf;
 	}
 	operator char *() {

Common/Log/LogManager.cpp

@@ -189,7 +189,6 @@ LogManager::~LogManager() {
 	// Make sure we don't shutdown while logging.  RemoveListener locks too, but there are gaps.
 	std::lock_guard<std::mutex> listeners_lock(listeners_lock_);
 
-	if (fileLog_)
 	delete fileLog_;
 #if !defined(MOBILE_DEVICE) || defined(_DEBUG)
 #if PPSSPP_PLATFORM(WINDOWS) && !PPSSPP_PLATFORM(UWP)

Common/Serialize/SerializeFuncs.h

@@ -64,7 +64,6 @@ void DoClass(PointerWrap &p, T &x) {
 template<class T>
 void DoClass(PointerWrap &p, T *&x) {
 	if (p.mode == PointerWrap::MODE_READ) {
-		if (x != nullptr)
 		delete x;
 		x = new T();
 	}

Common/Serialize/SerializeMap.h

@@ -63,7 +63,6 @@ template<class K, class T>
 void Do(PointerWrap &p, std::map<K, T *> &x) {
 	if (p.mode == PointerWrap::MODE_READ) {
 		for (auto it = x.begin(), end = x.end(); it != end; ++it) {
-			if (it->second != nullptr)
 			delete it->second;
 		}
 	}
@@ -81,7 +80,6 @@ template<class K, class T>
 void Do(PointerWrap &p, std::unordered_map<K, T *> &x) {
 	if (p.mode == PointerWrap::MODE_READ) {
 		for (auto it = x.begin(), end = x.end(); it != end; ++it) {
-			if (it->second != nullptr)
 			delete it->second;
 		}
 	}
@@ -135,7 +133,6 @@ template<class K, class T>
 void Do(PointerWrap &p, std::multimap<K, T *> &x) {
 	if (p.mode == PointerWrap::MODE_READ) {
 		for (auto it = x.begin(), end = x.end(); it != end; ++it) {
-			if (it->second != nullptr)
 			delete it->second;
 		}
 	}
@@ -153,7 +150,6 @@ template<class K, class T>
 void Do(PointerWrap &p, std::unordered_multimap<K, T *> &x) {
 	if (p.mode == PointerWrap::MODE_READ) {
 		for (auto it = x.begin(), end = x.end(); it != end; ++it) {
-			if (it->second != nullptr)
 			delete it->second;
 		}
 	}

Common/Serialize/SerializeSet.h

@@ -55,7 +55,6 @@ template <class T>
 void Do(PointerWrap &p, std::set<T *> &x) {
 	if (p.mode == PointerWrap::MODE_READ) {
 		for (auto it = x.begin(), end = x.end(); it != end; ++it) {
-			if (*it != nullptr)
 			delete *it;
 		}
 	}

Common/UI/Screen.cpp

@@ -96,9 +96,7 @@ void ScreenManager::switchToNext() {
 	}
 	stack_.push_back(nextStack_.front());
 	nextStack_.front().screen->focusChanged(ScreenFocusChange::FOCUS_BECAME_TOP);
-	if (temp.screen) {
 	delete temp.screen;
-	}
 	UI::SetFocusedView(nullptr);
 
 	// When will this ever happen? Should handle focus here too?
@@ -403,15 +401,11 @@ void ScreenManager::processFinishDialog() {
 }
 
 void ScreenManager::SetBackgroundOverlayScreens(Screen *backgroundScreen, Screen *overlayScreen) {
-	if (backgroundScreen_) {
 	delete backgroundScreen_;
-	}
 	backgroundScreen_ = backgroundScreen;
 	backgroundScreen_->setScreenManager(this);
 
-	if (overlayScreen_) {
 	delete overlayScreen_;
-	}
 	overlayScreen_ = overlayScreen;
 	overlayScreen_->setScreenManager(this);
 }

Core/Dialog/SavedataParam.cpp

@@ -1494,16 +1494,15 @@ void SavedataParam::Clear()
 		}
 
 		delete [] saveDataList;
-		saveDataList = 0;
+		saveDataList = NULL;
 		saveDataListCount = 0;
 	}
 	if (noSaveIcon)
 	{
-		if (noSaveIcon->texture != NULL)
 		delete noSaveIcon->texture;
 		noSaveIcon->texture = NULL;
 		delete noSaveIcon;
-		noSaveIcon = 0;
+		noSaveIcon = NULL;
 	}
 }
 
@@ -1924,7 +1923,6 @@ void SavedataParam::DoState(PointerWrap &p) {
 	Do(p, saveDataListCount);
 	Do(p, saveNameListDataCount);
 	if (p.mode == p.MODE_READ) {
-		if (saveDataList)
 		delete [] saveDataList;
 		if (saveDataListCount != 0) {
 			saveDataList = new SaveFileInfo[saveDataListCount];

Core/ELF/ParamSFO.h

@@ -83,7 +83,6 @@ private:
 		void SetData(const u8* data, int size);
 
 		~ValueData() {
-			if (u_value)
 			delete[] u_value;
 		}
 	};

Core/Font/PGF.cpp

@@ -82,9 +82,7 @@ PGF::PGF()
 }
 
 PGF::~PGF() {
-	if (fontData) {
 	delete [] fontData;
-	}
 }
 
 struct GlyphFromPGF1State {
@@ -139,9 +137,7 @@ void PGF::DoState(PointerWrap &p) {
 	Do(p, fontDataSizeTemp);
 	fontDataSize = (size_t)fontDataSizeTemp;
 	if (p.mode == p.MODE_READ) {
-		if (fontData) {
 		delete [] fontData;
-		}
 		if (fontDataSize) {
 			fontData = new u8[fontDataSize];
 			DoArray(p, fontData, (int)fontDataSize);

Core/HLE/ThreadQueueList.h

@@ -160,7 +160,6 @@ struct ThreadQueueList {
 
 	inline void clear() {
 		for (int i = 0; i < NUM_QUEUES; ++i) {
-			if (queues[i].data != nullptr)
 			free(queues[i].data);
 		}
 		memset(queues, 0, sizeof(queues));

Core/HLE/sceKernelInterrupt.cpp

@@ -433,7 +433,6 @@ void __KernelReturnFromInterrupt()
 
 void __RegisterIntrHandler(u32 intrNumber, IntrHandler* handler)
 {
-	if(intrHandlers[intrNumber])
 	delete intrHandlers[intrNumber];
 	intrHandlers[intrNumber] = handler;
 }

Core/HLE/sceKernelMemory.cpp

@@ -95,10 +95,8 @@ struct FPL : public KernelObject
 {
 	FPL() : blocks(NULL), nextBlock(0) {}
 	~FPL() {
-		if (blocks != NULL) {
 		delete [] blocks;
 	}
-	}
 	const char *GetName() override { return nf.name; }
 	const char *GetTypeName() override { return GetStaticTypeName(); }
 	static const char *GetStaticTypeName() { return "FPL"; }

Core/HLE/sceKernelModule.cpp

@@ -1387,7 +1387,6 @@ static PSPModule *__KernelLoadELFFromPtr(const u8 *ptr, size_t elfSize, u32 load
 	if (*magicPtr != 0x464c457f) {
 		ERROR_LOG(Log::sceModule, "Wrong magic number %08x", *magicPtr);
 		*error_string = "File corrupt";
-		if (newptr)
 		delete [] newptr;
 		module->Cleanup();
 		kernelObjects.Destroy<PSPModule>(module->GetUID());
@@ -1401,7 +1400,6 @@ static PSPModule *__KernelLoadELFFromPtr(const u8 *ptr, size_t elfSize, u32 load
 	int result = reader.LoadInto(loadAddress, fromTop);
 	if (result != SCE_KERNEL_ERROR_OK) {
 		ERROR_LOG(Log::sceModule, "LoadInto failed with error %08x",result);
-		if (newptr)
 		delete [] newptr;
 		module->Cleanup();
 		kernelObjects.Destroy<PSPModule>(module->GetUID());
@@ -1426,7 +1424,6 @@ static PSPModule *__KernelLoadELFFromPtr(const u8 *ptr, size_t elfSize, u32 load
 	if (!Memory::IsValidAddress(modinfoaddr)) {
 		*error_string = StringFromFormat("Bad module info address %08x", modinfoaddr);
 		ERROR_LOG(Log::sceModule, "Bad module info address %08x", modinfoaddr);
-		if (newptr)
 		delete[] newptr;
 		module->Cleanup();
 		kernelObjects.Destroy<PSPModule>(module->GetUID());
@@ -1729,7 +1726,6 @@ static PSPModule *__KernelLoadELFFromPtr(const u8 *ptr, size_t elfSize, u32 load
 		module->nm.entry_addr = -1;
 	}
 
-	if (newptr)
 	delete [] newptr;
 
 	if (!reportedModule && IsHLEVersionedModule(modinfo->name)) {
@@ -1817,9 +1813,7 @@ static PSPModule *__KernelLoadModule(u8 *fileptr, size_t fileSize, SceKernelLMOp
 		u32 error;
 		module = __KernelLoadELFFromPtr(temp ? temp : fileptr + offsets[5], elfSize, PSP_GetDefaultLoadAddress(), false, error_string, &magic, error);
 
-		if (temp) {
 		delete [] temp;
-		}
 	} else if (fileSize > sizeof(PSP_Header)) {
 		u32 error;
 		u32 magic = 0;

Core/HLE/sceNetAdhoc.cpp

@@ -5008,7 +5008,7 @@ int NetAdhocMatching_Start(int matchingId, int evthPri, int evthPartitionId, int
 		//sceNetAdhocMatchingSetHelloOpt(matchingId, optLen, optDataAddr); //SetHelloOpt only works when context is running
 		if ((optLen > 0) && Memory::IsValidAddress(optDataAddr)) {
 			// Allocate the memory and copy the content
-			if (item->hello != NULL) free(item->hello);
+			free(item->hello);
 			item->hello = (uint8_t*)malloc(optLen);
 			if (item->hello != NULL) {
 				Memory::Memcpy(item->hello, optDataAddr, optLen);

Core/HLE/scePsmf.cpp

@@ -219,7 +219,6 @@ public:
 	PsmfPlayer(const PsmfPlayerCreateData *data);
 	~PsmfPlayer() {
 		AbortFinish();
-		if (mediaengine) 
 		delete mediaengine;
 		pspFileSystem.CloseFile(filehandle);
 	}

Core/HW/BufferQueue.h

@@ -29,13 +29,11 @@ struct BufferQueue {
 	}
 
 	~BufferQueue() {
-		if (bufQueue)
 		delete [] bufQueue;
 	}
 
 	bool alloc(int size) {
 		_assert_(size > 0);
-		if (bufQueue)
 		delete [] bufQueue;
 		bufQueue = new unsigned char[size];
 		bufQueueSize = size;

Core/HW/MediaEngine.cpp

@@ -147,9 +147,7 @@ MediaEngine::~MediaEngine() {
 
 void MediaEngine::closeMedia() {
 	closeContext();
-	if (m_pdata)
 	delete m_pdata;
-	if (m_demux)
 	delete m_demux;
 	m_pdata = nullptr;
 	m_demux = nullptr;

Core/HW/SasAudio.h

@@ -138,7 +138,7 @@ private:
 class SasAtrac3 {
 public:
 	SasAtrac3() : contextAddr_(0), atracID_(-1), sampleQueue_(0), end_(false) {}
-	~SasAtrac3() { if (sampleQueue_) delete sampleQueue_; }
+	~SasAtrac3() { delete sampleQueue_; }
 	int setContext(u32 context);
 	void getNextSamples(s16 *outbuf, int wantedSamples);
 	int addStreamData(u32 bufPtr, u32 addbytes);

Core/MIPS/MIPSVFPUUtils.cpp

@@ -1000,10 +1000,9 @@ static inline bool load_vfpu_table(T *&ptr, const char *filename, size_t expecte
 	size_t size = 0u;
 	INFO_LOG(Log::CPU, "Loading '%s'...", filename);
 	ptr = reinterpret_cast<decltype(&*ptr)>(g_VFS.ReadFile(filename, &size));
-	if(!ptr || size != expected_size)
+	if (!ptr || size != expected_size) {
-	{
 		ERROR_LOG(Log::CPU, "Error loading '%s' (size=%u, expected: %u)", filename, (unsigned)size, (unsigned)expected_size);
-		if(ptr) delete[] ptr;
+		delete[] ptr;
 		ptr = nullptr;
 		return false;
 	}

GPU/Vulkan/ShaderManagerVulkan.cpp

@@ -91,7 +91,6 @@ static Promise<VkShaderModule> *CompileShaderModuleAsync(VulkanContext *vulkan,
 #ifdef SHADERLOG
 			OutputDebugStringA("OK");
 #endif
-			if (tag)
 			delete tag;
 		}
 		return shaderModule;

Windows/Debugger/CtrlRegisterList.cpp

@@ -142,9 +142,7 @@ CtrlRegisterList::CtrlRegisterList(HWND _wnd)
 CtrlRegisterList::~CtrlRegisterList()
 {
 	DeleteObject(font);
-	if (lastCat0Values != NULL)
 	delete [] lastCat0Values;
-	if (changedCat0Regs != NULL)
 	delete [] changedCat0Regs;
 }
 

Windows/MainWindow.cpp

@@ -583,24 +583,20 @@ namespace MainWindow
 
 	void DestroyDebugWindows() {
 		DialogManager::RemoveDlg(disasmWindow);
-		if (disasmWindow)
 		delete disasmWindow;
 		disasmWindow = nullptr;
 
 #if PPSSPP_API(ANY_GL)
 		DialogManager::RemoveDlg(geDebuggerWindow);
-		if (geDebuggerWindow)
 		delete geDebuggerWindow;
 		geDebuggerWindow = nullptr;
 #endif
 
 		DialogManager::RemoveDlg(memoryWindow);
-		if (memoryWindow)
 		delete memoryWindow;
 		memoryWindow = nullptr;
 
 		DialogManager::RemoveDlg(vfpudlg);
-		if (vfpudlg)
 		delete vfpudlg;
 		vfpudlg = nullptr;
 	}

Windows/RawInput.cpp

@@ -443,9 +443,7 @@ namespace WindowsRawInput {
 	}
 
 	void Shutdown() {
-		if (rawInputBuffer) {
 		free(rawInputBuffer);
-		}
 		rawInputBuffer = 0;
 	}
 };