Shawn O. Pearce wrote:
Junio C Hamano <junkio@xxxxxxx> wrote:
After seeing what this patch has to do, I feel dirty, but
that is not Mark's fault -- rather it is Tk's.
I am tempted to suggest adding an explicit "Save window
configuration" action on the menubar and forget about
resurrecting the window configuration immediately before the
end of the last session.
Maybe take a look at what git-gui does here, because its slightly
saner and still saves the geometry on exit:
set is_quitting 0
proc do_quit {} {
global is_quitting
if {$is_quitting} return
set is_quitting 1
# save wm geometry
}
bind . <Destroy> do_quit
OK, its not that much saner. But it does bypass needing to setup
some ugly bindings on every object in the UI. Though I recently
took a slightly different approach in a dialog:
proc do_quit {} {
bind . <Destroy> {}
# do cleanup
}
bind . <Destroy> do_quit
Yes the binding is firing in both cases for some arbitrary child
widget in the window, but it doesn't matter. In the latter version
setting the binding to the empty string removes the do_quit binding,
allowing the other widgets to destroy without reinvoking do_quit
and whacking whatever geometry data you may have saved before the
widgets started to get deleted.
The only problem I seem to have in git-gui is the window position
opens about 10 pixels lower and 2 pixels to the right than the last
time it opened. I think there's a bug in Tk on Windows where the
window position on the desktop doesn't include the titlebar when I
get it, but expects to include it when I attempt to set it on the
next start.
I should note I also see the same behavior with my day-time-job's
Java apps on Windows however, so I don't think its a specific Tk
or git-gui issue.
What you suggest is essentially what gitk does before my patch. In
git-gui, routine do_quit is invoked when Tk is ready to destroy the top
level window: this is _after_ Tk has destroyed everything in the window,
and thus the problem. I still haven't isolated which object(s) being
destroyed from gitk cause the geometry reported from wm info to change,
clearly my patch is binding widgets that do not need to be bound, but I
haven't found which specific one needs to be bound.
With my patch, the correct position is saved and restored on both Linux
and Cygwin. I think that should be the goal. A more elegant solution
accomplishing that end is of course desirable, but there remains the
issue of finding it. I think the objections to my patch are more
theoretical than practical: I seriously doubt you can find any
practically observable side effect of binding all the widgets other than
that the window geometry _is_ correctly saved and restored.
As to being specific to Tk, I have many windows applications that
successfully save and restore window state, this geometry issue is not
endemic to Windows and I have never encountered it except with Tk.
Mark
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html