Christian Neumair <[EMAIL PROTECTED]> writes:

> The version in HEAD actually doesn't unlock the window inside the
> function but just exits the if loop. The patch adds an unlock function
> (see msdn, when not passing hwnd, the active window should be unlocked)
> and moves the stub to the place where it belongs. Basically this makes
> the code easier to read. If you want we can remove the whole if (!hwnd)
> code because if hwnd is 0, lockedWnd = hwnd does the right thing, too.

Well, it already does the right thing today, so I don't see why you
need to change it. If it's just to make it more readable OK, but I
don't see much point given that this function is completely wrong
anyway. And I definitely don't see a reason for removing the fixme
message.

-- 
Alexandre Julliard
[EMAIL PROTECTED]

Reply via email to