Simplify dlls/comctl32/rebar.c

Francois Gouget fgouget at free.fr
Mon Feb 2 05:46:49 CST 2009


On Fri, 30 Jan 2009, Gerald Pfeifer wrote:

> (It's not clear which parameters we'd really need if it is ever fully
> implemented.)
> 
> ChangeLog:
> Simplify.
[...]
> 
> Index: dlls/comctl32/rebar.c
[...]
> @@ -3618,7 +3618,6 @@ REBAR_WindowProc (HWND hwnd, UINT uMsg, 
>  
>  /*	case RB_SETCOLORSCHEME: */
>  /*	case RB_SETPALETTE: */
> -/*	    return REBAR_GetPalette (infoPtr, wParam, lParam); */
>  
>  	case RB_SETPARENT:

Once the commented out return has been removed it looks like 
RB_SETCOLORSCHEME and RB_SETPALETTE are meant to fall through to 
RB_SETPARENT, which does not seem right...

So I'd leave a comment there, maybe /* return ...; */.


-- 
Francois Gouget <fgouget at free.fr>              http://fgouget.free.fr/
             Research is the transformation of money to knowledge.
            Innovation is the transformation of knowledge to money.
                              -- Dr. Hans Meixner



More information about the wine-patches mailing list