Re: [PATCH 1/2] riched20: Don't call UpdateWindow after setting the background color.

[Bruno Jesus]

On Fri, May 6, 2016 at 11:04 AM, Alex Henrie  wrote:
> +static void test_background(void)
> +{
> - HWND hwndRichEdit = new_richedit(NULL);
> - RECT updateRect;
> -
> - /* set the background color to black */
> - ValidateRect(hwndRichEdit, NULL);
> - SendMessageA(hwndRichEdit, EM_SETBKGNDCOLOR, FALSE, 0);
> - GetUpdateRect(hwndRichEdit, &updateRect, FALSE);
> - ok(!IsRectEmpty(&updateRect), "The update rect should not be empty\n");
> -
> - DestroyWindow(hwndRichEdit);
>
> +}

I believe you should clear updateRect before calling GetUpdateRect,
also according to msdn the return value of GetUpdateRect should also
reflect if the rect was changed or not so it could be a double test.

Best wishes,
Bruno

[PATCH 1/2] riched20: Don't call UpdateWindow after setting the background color.

[Alex Henrie]

2016-05-06 0:32 GMT-06:00 Bruno Jesus :
> On Fri, May 6, 2016 at 11:04 AM, Alex Henrie  wrote:
> > +static void test_background(void)
> > +{
> > - HWND hwndRichEdit = new_richedit(NULL);
> > - RECT updateRect;
> > -
> > - /* set the background color to black */
> > - ValidateRect(hwndRichEdit, NULL);
> > - SendMessageA(hwndRichEdit, EM_SETBKGNDCOLOR, FALSE, 0);
> > - GetUpdateRect(hwndRichEdit, &updateRect, FALSE);
> > - ok(!IsRectEmpty(&updateRect), "The update rect should not be empty\n");
> > -
> > - DestroyWindow(hwndRichEdit);
> >
> > +}
>
> I believe you should clear updateRect before calling GetUpdateRect,
> also according to msdn the return value of GetUpdateRect should also
> reflect if the rect was changed or not so it could be a double test.

The tests in dlls/user32/tests/win.c show that GetUpdateRect always
initializes the rect parameter (unless it is NULL). However, you're
right that checking the return value would be cleaner. I will submit a
simplified version 2 momentarily.

-Alex