Re: [PATCH 0/5] MR9409: comctl32/progress: Add a helper to get the draw procedures.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/comctl32/progress.c:
>  return GetWindowLongW(hwnd, GWL_STYLE) & PBS_SMOOTH;
>
> }
>
> +static const ProgressDrawProc *PROGRESS_GetDrawProcs(HWND hwnd, BOOL smooth, DWORD style)
> +{
> - int proc_idx = (smooth ? 0 : 4) + ((style & PBS_VERTICAL) ? 2 : 0);

It's en existing thing, but I still want to mention it. This is not going to use all 8 drawing functions, any idea why we have 8 of them? If all we need is 4 states, this should be made more obvious, for example
> drawProcThemed[!!smooth][!!(style & PBS_VERTICAL)]