[ros-diffs] [mf] 15225: eliminate GCC4 warnings

eliminate GCC4 warnings
Modified: trunk/reactos/subsys/system/ibrowser/Makefile
Modified: trunk/reactos/subsys/system/ibrowser/Makefile.MinGW
Modified: trunk/reactos/subsys/system/ibrowser/Makefile.PCH
Modified: trunk/reactos/subsys/system/ibrowser/webchild.h
  _____  

Modified: trunk/reactos/subsys/system/ibrowser/Makefile
--- trunk/reactos/subsys/system/ibrowser/Makefile	2005-05-11
19:08:36 UTC (rev 15224)
+++ trunk/reactos/subsys/system/ibrowser/Makefile	2005-05-11
19:11:27 UTC (rev 15225)
@@ -17,7 +17,7 @@

 TARGET_CFLAGS := \
 	-D__USE_W32API -DWIN32 -D_ROS_ \
 	-D_WIN32_IE=0x0600 -D_WIN32_WINNT=0x0501 -DWINVER=0x0500 \
-	-DUNICODE -fexceptions -Wall \
+	-DUNICODE -fexceptions -Wall -Wno-unused-value \
 	-I../../../include/expat
 
 TARGET_CPPFLAGS := $(TARGET_CFLAGS)
  _____  

Modified: trunk/reactos/subsys/system/ibrowser/Makefile.MinGW
--- trunk/reactos/subsys/system/ibrowser/Makefile.MinGW	2005-05-11
19:08:36 UTC (rev 15224)
+++ trunk/reactos/subsys/system/ibrowser/Makefile.MinGW	2005-05-11
19:11:27 UTC (rev 15225)
@@ -8,7 +8,7 @@

 CXX = g++
 LINK = g++
 
-CFLAGS	= -DWIN32 -D_WIN32_IE=0x0600 -D_WIN32_WINNT=0x0501
-DWINVER=0x0500 -fexceptions -Wall -I. -I$(EXPAT_INC)
+CFLAGS	= -DWIN32 -D_WIN32_IE=0x0600 -D_WIN32_WINNT=0x0501
-DWINVER=0x0500 -fexceptions -Wall -Wno-unused-value -I. -I$(EXPAT_INC)
 RCFLAGS	= -DWIN32 -D__WINDRES__
 LFLAGS	= -Wl,--subsystem,windows
 
  _____  

Modified: trunk/reactos/subsys/system/ibrowser/Makefile.PCH
--- trunk/reactos/subsys/system/ibrowser/Makefile.PCH	2005-05-11
19:08:36 UTC (rev 15224)
+++ trunk/reactos/subsys/system/ibrowser/Makefile.PCH	2005-05-11
19:11:27 UTC (rev 15225)
@@ -10,7 +10,7 @@

 CXX = g++
 LINK = g++
 
-CFLAGS	= -DWIN32 -D_WIN32_IE=0x0600 -D_WIN32_WINNT=0x0501
-DWINVER=0x0500 -fexceptions -Wall -I. -I$(EXPAT_INC)
+CFLAGS	= -DWIN32 -D_WIN32_IE=0x0600 -D_WIN32_WINNT=0x0501
-DWINVER=0x0500 -fexceptions -Wall -Wno-unused-value -I. -I$(EXPAT_INC)
 RCFLAGS	= -DWIN32 -D__WINDRES__
 LFLAGS	= -Wl,--subsystem,windows
 
  _____  

Modified: trunk/reactos/subsys/system/ibrowser/webchild.h
--- trunk/reactos/subsys/system/ibrowser/webchild.h	2005-05-11
19:08:36 UTC (rev 15224)
+++ trunk/reactos/subsys/system/ibrowser/webchild.h	2005-05-11
19:11:27 UTC (rev 15225)
@@ -115,25 +115,38 @@

 	{
 	}
 
-	STDMETHODIMP QueryInterface(REFIID riid, LPVOID* ppv) {*ppv=0;
-	 if (IsEqualIID(riid, _uuid_base) ||
-		 IsEqualIID(riid, IID_IUnknown))
{*ppv=static_cast<BASE*>(this); this->AddRef(); return S_OK;}
-	 return E_NOINTERFACE;}
+	STDMETHODIMP QueryInterface(REFIID riid, LPVOID* ppv)
+	{
+		*ppv = NULL;
 
+		if (IsEqualIID(riid, _uuid_base) || IsEqualIID(riid,
IID_IUnknown))
+			{*ppv=static_cast<BASE*>(this); this->AddRef();
return S_OK;}
+
+		return E_NOINTERFACE;
+	}
+
 protected:
 	IComSrvQI() {}
+	virtual ~IComSrvQI() {}
 
 	REFIID	_uuid_base;
 };
 
 template<> struct IComSrvQI<IUnknown> : public IUnknown
 {
-	STDMETHODIMP QueryInterface(REFIID riid, LPVOID* ppv) {*ppv=0;
-	 if (IsEqualIID(riid, IID_IUnknown)) {*ppv=this; AddRef();
return S_OK;}
-	 return E_NOINTERFACE;}
+	STDMETHODIMP QueryInterface(REFIID riid, LPVOID* ppv)
+	{
+		*ppv = NULL;
 
+		if (IsEqualIID(riid, IID_IUnknown))
+			{*ppv=this; AddRef(); return S_OK;}
+
+		return E_NOINTERFACE;
+	}
+
 protected:
 	IComSrvQI<IUnknown>() {}
+	virtual ~IComSrvQI<IUnknown>() {}
 };
 
 
@@ -455,6 +468,8 @@
 
 struct DWebBrowserEvents2IF
 {
+	virtual ~DWebBrowserEvents2IF() {}
+
     virtual void StatusTextChange(const BStr& text)
 		{}
 
@@ -594,6 +609,9 @@
  // MinGW defines a wrong FixedDWebBrowserEvents2 interface with
virtual functions for DISPID calls, so we use our own, corrected
version:
 interface FixedDWebBrowserEvents2 : public IDispatch
 {
+#ifdef __GNUC__
+	virtual ~FixedDWebBrowserEvents2() {}
+#endif
 };
 
 struct ANSUNC DWebBrowserEvents2Impl : public SimpleComObject,