[PATCH] winecfg => control panel applets patch

[Vitaliy Margolen]

pure_evil at mail.bg wrote:
> Allows access to different winecfg tabs via control-panel applets (without 
> showing all of the cfg's tabs when accessed thusly)
>    
> +LONG APIENTRY EXECUTE_APPLET(HWND hwndCpl, UINT message, LPARAM lParam1, 
> LPARAM lParam2)
> +{
> +
> + int NumOfApplet = (int)lParam1;
> +
> + switch (NumOfApplet)
> + {
> +
> +  case 0:

You patch is wrapped. Also please for new files follow standard Wine codding 
style - 4 spaces indentation, no tabs.

> +    /*It would get messier with an or statement everywhere, so...*/
> +    BOOL Do_App = FALSE;
> +    BOOL Do_Lib = FALSE;
> +    BOOL Do_Graph = FALSE;
> +    BOOL Do_Desktop = FALSE;
> +    BOOL Do_Drives = FALSE;
> +    BOOL Do_Audio = FALSE;

> +    if (display_what_tabs == 2) { Do_Lib = TRUE; }
> +    if (display_what_tabs == 3) { Do_Graph = TRUE; }
> +    if (display_what_tabs == 4) { Do_Desktop = TRUE; }
> +    if (display_what_tabs == 5) { Do_Drives = TRUE; }
> +    if (display_what_tabs == 6) { Do_Audio = TRUE; }

> +   if ( lstrcmpi(CommandLine,"applications") == 0 ) display_what_tabs = 1;
> +   if ( lstrcmpi(CommandLine,"libraries") == 0 ) display_what_tabs = 2;
> +   if ( lstrcmpi(CommandLine,"graphics") == 0 ) display_what_tabs = 3;
> +   if ( lstrcmpi(CommandLine,"desktop") == 0 ) display_what_tabs = 4;
> +   if ( lstrcmpi(CommandLine,"drives") == 0 ) display_what_tabs = 5;
> +   if ( lstrcmpi(CommandLine,"audio") == 0 ) display_what_tabs = 6;


This is ugly and redundant. You should make it a bit-mask instead. However 
with all this you removing really big functionality from the winecfg - 
ability to set this on per application basis.

Vitaliy.