Christian Neumair chris@gnome-de.org 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.