Alexandre Julliard : user: Remove some redundant NULL pointer checks.

Alexandre Julliard julliard at wine.codeweavers.com
Tue May 9 14:15:25 CDT 2006


Module: wine
Branch: refs/heads/master
Commit: 9d4bf4914fd7580d4a6d3b962e7bacb7c4da8e46
URL:    http://source.winehq.org/git/?p=wine.git;a=commit;h=9d4bf4914fd7580d4a6d3b962e7bacb7c4da8e46

Author: Alexandre Julliard <julliard at winehq.org>
Date:   Tue May  9 15:16:47 2006 +0200

user: Remove some redundant NULL pointer checks.

---

 dlls/user/winproc.c |   32 ++++++++------------------------
 1 files changed, 8 insertions(+), 24 deletions(-)

diff --git a/dlls/user/winproc.c b/dlls/user/winproc.c
index 4e12c7c..e885784 100644
--- a/dlls/user/winproc.c
+++ b/dlls/user/winproc.c
@@ -3248,14 +3248,10 @@ LRESULT WINAPI CallWindowProc16( WNDPROC
     switch(proc->type)
     {
     case WIN_PROC_16:
-        if (!proc->thunk.t_from32.proc) return 0;
-        return WINPROC_CallWndProc16( proc->thunk.t_from32.proc,
-                                      hwnd, msg, wParam, lParam );
+        return WINPROC_CallWndProc16( proc->thunk.t_from32.proc, hwnd, msg, wParam, lParam );
     case WIN_PROC_32A:
-        if (!proc->thunk.t_from16.proc) return 0;
         return __wine_call_wndproc_32A( hwnd, msg, wParam, lParam, proc->thunk.t_from16.proc );
     case WIN_PROC_32W:
-        if (!proc->thunk.t_from16.proc) return 0;
         return __wine_call_wndproc_32W( hwnd, msg, wParam, lParam, proc->thunk.t_from16.proc );
     default:
         WARN_(relay)("Invalid proc %p\n", proc );
@@ -3305,17 +3301,11 @@ LRESULT WINAPI CallWindowProcA(
     switch(proc->type)
     {
     case WIN_PROC_16:
-        if (!proc->thunk.t_from32.proc) return 0;
-        return WINPROC_CallProc32ATo16( proc->thunk.t_from32.proc,
-                                        hwnd, msg, wParam, lParam );
+        return WINPROC_CallProc32ATo16( proc->thunk.t_from32.proc, hwnd, msg, wParam, lParam );
     case WIN_PROC_32A:
-        if (!proc->thunk.t_from16.proc) return 0;
-        return WINPROC_CallWndProc( proc->thunk.t_from16.proc,
-                                      hwnd, msg, wParam, lParam );
+        return WINPROC_CallWndProc( proc->thunk.t_from16.proc, hwnd, msg, wParam, lParam );
     case WIN_PROC_32W:
-        if (!proc->thunk.t_from16.proc) return 0;
-        return WINPROC_CallProc32ATo32W( proc->thunk.t_from16.proc,
-                                         hwnd, msg, wParam, lParam );
+        return WINPROC_CallProc32ATo32W( proc->thunk.t_from16.proc, hwnd, msg, wParam, lParam );
     default:
         WARN_(relay)("Invalid proc %p\n", proc );
         return 0;
@@ -3335,23 +3325,17 @@ LRESULT WINAPI CallWindowProcW( WNDPROC 
 
     if (!func) return 0;
 
-    if (!(proc = WINPROC_GetPtr( (WNDPROC)func )))
+    if (!(proc = WINPROC_GetPtr( func )))
         return WINPROC_CallWndProc( func, hwnd, msg, wParam, lParam );
 
     switch(proc->type)
     {
     case WIN_PROC_16:
-        if (!proc->thunk.t_from32.proc) return 0;
-        return WINPROC_CallProc32WTo16( proc->thunk.t_from32.proc,
-                                        hwnd, msg, wParam, lParam );
+        return WINPROC_CallProc32WTo16( proc->thunk.t_from32.proc, hwnd, msg, wParam, lParam );
     case WIN_PROC_32A:
-        if (!proc->thunk.t_from16.proc) return 0;
-        return WINPROC_CallProc32WTo32A( proc->thunk.t_from16.proc,
-                                         hwnd, msg, wParam, lParam );
+        return WINPROC_CallProc32WTo32A( proc->thunk.t_from16.proc, hwnd, msg, wParam, lParam );
     case WIN_PROC_32W:
-        if (!proc->thunk.t_from16.proc) return 0;
-        return WINPROC_CallWndProc( proc->thunk.t_from16.proc,
-                                      hwnd, msg, wParam, lParam );
+        return WINPROC_CallWndProc( proc->thunk.t_from16.proc, hwnd, msg, wParam, lParam );
     default:
         WARN_(relay)("Invalid proc %p\n", proc );
         return 0;




More information about the wine-cvs mailing list