Re: [PATCH] comctl32: Update Thumb Rect when position changes

[Nikolay Sivov]

On 08.01.2016 10:33, Alistair Leslie-Hughes wrote:
> dlls/comctl32/trackbar.c | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/dlls/comctl32/trackbar.c b/dlls/comctl32/trackbar.c
> index 6d092a3..3a92794 100644
> --- a/dlls/comctl32/trackbar.c
> +++ b/dlls/comctl32/trackbar.c
> @@ -1225,13 +1225,17 @@ TRACKBAR_SetPos (TRACKBAR_INFO *infoPtr, BOOL fPosition, LONG lPosition)
>      infoPtr->lPos = lPosition;
>  if (infoPtr->lPos < infoPtr->lRangeMin)
> - infoPtr->lPos = infoPtr->lRangeMin;
> -
>    infoPtr->lPos = infoPtr->lRangeMin;
>
> if (infoPtr->lPos > infoPtr->lRangeMax)
> - infoPtr->lPos = infoPtr->lRangeMax;
> -
>    infoPtr->lPos = infoPtr->lRangeMax;
> infoPtr->flags |= TB_THUMBPOSCHANGED;
> - if (fPosition && oldPos != lPosition) TRACKBAR_InvalidateThumbMove(infoPtr, oldPos, lPosition);
> -
>
> if (fPosition && oldPos != lPosition)
> -
>
> {
> -
>    TRACKBAR_UpdateThumb(infoPtr);
> -
>    TRACKBAR_InvalidateThumbMove(infoPtr, oldPos, infoPtr->lPos);
> -
>
> }
>
> return 0;
>
> }

It's updated indirectly later I think. What does this fix? Visually
SETPOS works as it is for me.