Make interfaces consistent

elishacloud · Dec 2, 2024 · d3a3878 · d3a3878
1 parent 986654f
commit d3a3878
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 20 deletions.
diff --git a/BuildNo.rc b/BuildNo.rc
@@ -1 +1 @@
-#define BUILD_NUMBER 81 
+#define BUILD_NUMBER 82 
diff --git a/IDirectInputDeviceX.cpp b/IDirectInputDeviceX.cpp
@@ -936,7 +936,7 @@ HRESULT m_IDirectInputDeviceX::WriteEffectToFileX(V lpszFileName, DWORD dwEntrie
 	return GetProxyInterface<T>()->WriteEffectToFile(lpszFileName, dwEntries, rgDiFileEft, dwFlags);
 }
 
-void m_IDirectInputDeviceX::ReleaseDirectInput()
+void m_IDirectInputDeviceX::ReleaseInterface()
 {
 	// Don't delete wrapper interface
 	if (StringType == ANSI_CHARSET)

diff --git a/IDirectInputDeviceX.h b/IDirectInputDeviceX.h
@@ -112,7 +112,7 @@ class m_IDirectInputDeviceX : public AddressLookupTableDinputObject
 	template <class T, class V>
 	inline HRESULT WriteEffectToFileX(V lpszFileName, DWORD dwEntries, LPDIFILEEFFECT rgDiFileEft, DWORD dwFlags);
 
-	void ReleaseDirectInput();
+	void ReleaseInterface();
 
 public:
 	m_IDirectInputDeviceX(IDirectInputDevice8W *aOriginal, REFIID riid) : ProxyInterface(aOriginal), WrapperID(riid), StringType(GetStringType(riid))
@@ -131,7 +131,7 @@ class m_IDirectInputDeviceX : public AddressLookupTableDinputObject
 		// Delete Critical Section
 		DeleteCriticalSection(&dics);
 
-		ReleaseDirectInput();
+		ReleaseInterface();
 	}
 
 	/*** IUnknown methods ***/

diff --git a/IDirectInputEffect.cpp b/IDirectInputEffect.cpp
@@ -51,21 +51,11 @@ HRESULT m_IDirectInputEffect::QueryInterface(REFIID riid, LPVOID * ppvObj)
 
 	if (!ProxyInterface)
 	{
-		if (!ppvObj)
+		if (ppvObj)
 		{
-			return E_POINTER;
+			*ppvObj = nullptr;
 		}
-		*ppvObj = nullptr;
-
-		if (riid == WrapperID || riid == IID_IUnknown)
-		{
-			*ppvObj = this;
-
-			AddRef();
-
-			return DI_OK;
-		}
-		return E_POINTER;
+		return E_NOINTERFACE;
 	}
 
 	return ProxyQueryInterface(ProxyInterface, riid, ppvObj, WrapperID, WrapperInterface);

diff --git a/IDirectInputX.cpp b/IDirectInputX.cpp
@@ -301,7 +301,7 @@ HRESULT m_IDirectInputX::CreateDeviceExX(REFGUID rguid, REFIID riid, V *ppvObj,
 	return hr;
 }
 
-void m_IDirectInputX::ReleaseDirectInput()
+void m_IDirectInputX::ReleaseInterface()
 {
 	// Don't delete wrapper interface
 	if (StringType == ANSI_CHARSET)

diff --git a/IDirectInputX.h b/IDirectInputX.h
@@ -48,7 +48,7 @@ class m_IDirectInputX : public AddressLookupTableDinputObject
 	template <class T, class V>
 	inline HRESULT CreateDeviceExX(REFGUID rguid, REFIID riid, V *ppvObj, LPUNKNOWN pUnkOuter);
 
-	void ReleaseDirectInput();
+	void ReleaseInterface();
 
 public:
 	m_IDirectInputX(IDirectInput8W *aOriginal, REFIID riid) : ProxyInterface(aOriginal), WrapperID(riid), StringType(GetStringType(riid))
@@ -59,7 +59,7 @@ class m_IDirectInputX : public AddressLookupTableDinputObject
 	{
 		LOG_LIMIT(3, __FUNCTION__ << " (" << this << ")" << " deleting interface!");
 
-		ReleaseDirectInput();
+		ReleaseInterface();
 	}
 
 	/*** IUnknown methods ***/