Re: [PATCH] Re: [PATCH] some dc:LockWindowUpdate work

[Alexandre Julliard]

Christian Neumair  writes:
> I see a reason: FIXME denotes that something should be fixed, but in
> this place we don't need to fix anything. I prepared a really small
> patch, have a look at it and tell me if you still find the cleanup
> obsolete. And I think it's not a good idea to tell the users that
> something has gone wrong (stub) while the desired action was taken
> without any problems.

But no desired action has taken place at all!  All we do is store the
window in an internal variable, but we don't lock the window updates
at all, which is the whole point of the function. So a fixme is
definitely appropriate.