Re: [PATCH] user32/tests: make the MDI client take up the whole client area

[Dmitry Timoshkov]

Reece Dunn <msclrhd(a)googlemail.com> wrote:
> The MDI tests in user32/tests/msg.c were using an MDI client of zero
> size (0,0 - 0,0). This means that the tests are not realistic for
> real-world applications that use MDI, as their frame windows size the
> client window to fit the frame's client area.

It would be better IMO to create MDI client with the correct size
at the first place instead of resizing it later. That would prevent
any confusion, and perhaps even more problems once somebody adds tests
before the MoveWindow() call.

-- 
Dmitry.

[PATCH] user32/tests: make the MDI client take up the whole client area

[Reece Dunn]

On 8 February 2010 08:41, Dmitry Timoshkov <dmitry(a)codeweavers.com> wrote:
> Reece Dunn <msclrhd(a)googlemail.com> wrote:
> > The MDI tests in user32/tests/msg.c were using an MDI client of zero
> > size (0,0 - 0,0). This means that the tests are not realistic for
> > real-world applications that use MDI, as their frame windows size the
> > client window to fit the frame's client area.
>
> It would be better IMO to create MDI client with the correct size
> at the first place instead of resizing it later. That would prevent
> any confusion, and perhaps even more problems once somebody adds tests
> before the MoveWindow() call.

I have sent an updated version.

Thanks,
- Reece