Re: [PATCH v5] user32: Don't save maximized position placement for toplevel windows covering the entire work area.

[Huw Davies]

On Thu, Aug 26, 2021 at 05:59:45PM +0300, Gabriel Ivăncescu wrote:
> Thanks, I didn't really like the IsIconic thing, it seemed fragile, so I
> went with a slightly different approach, which also makes
> update_maximized_pos more typical like the other helpers, in that it takes a
> HWND rather than a WND, and does all the implementation details within. In
> one call site I just moved the call outside the lock, but for the other I
> had to release/re-acquire the lock, I hope it's not too big of a deal. It's
> still less code than special casing the work rect it in both callers.

Dropping and reacquiring the lock isn't very pretty (and needs care to
keep the window state consistent).

Huw.