Simplify dlls/comctl32/rebar.c (RESEND)

Gerald Pfeifer gerald at pfeifer.com
Sun Aug 16 18:02:39 CDT 2009


On Mon, 11 May 2009, Dmitry Timoshkov wrote:
> "Gerald Pfeifer" <gerald at pfeifer.com> wrote:
> 
>> static LRESULT
>> -REBAR_GetPalette (const REBAR_INFO *infoPtr, WPARAM wParam, LPARAM lParam)
>> +REBAR_GetPalette ()
>> {
> (void) is mandatory, please don't omit it.

Sorry, first I misunderstood this comment, but know I think I got it. :-)

Updated patch below.

Gerald


ChangeLog:
Simplify REBAR_GetPalette().

diff --git a/dlls/comctl32/rebar.c b/dlls/comctl32/rebar.c
index e11d974..af7930b 100644
--- a/dlls/comctl32/rebar.c
+++ b/dlls/comctl32/rebar.c
@@ -2255,7 +2255,7 @@ REBAR_GetBkColor (const REBAR_INFO *infoPtr)
 
 
 static LRESULT
-REBAR_GetPalette (const REBAR_INFO *infoPtr, WPARAM wParam, LPARAM lParam)
+REBAR_GetPalette (void)
 {
     FIXME("empty stub!\n");
 
@@ -3557,7 +3557,7 @@ REBAR_WindowProc (HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
 /*	case RB_GETDROPTARGET: */
 
 	case RB_GETPALETTE:
-	    return REBAR_GetPalette (infoPtr, wParam, lParam);
+	    return REBAR_GetPalette ();
 
 	case RB_GETRECT:
 	    return REBAR_GetRect (infoPtr, wParam, lParam);
@@ -3618,7 +3618,7 @@ REBAR_WindowProc (HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
 
 /*	case RB_SETCOLORSCHEME: */
 /*	case RB_SETPALETTE: */
-/*	    return REBAR_GetPalette (infoPtr, wParam, lParam); */
+/*	    return REBAR_GetPalette (); */
 
 	case RB_SETPARENT:
 	    return REBAR_SetParent (infoPtr, wParam);



More information about the wine-patches mailing list