[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.