login
Header Space

 
 

Re: exception in git-gui when saving options

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Martin Waitz <tali@...>
Cc: <git@...>
Date: Sunday, February 18, 2007 - 3:14 am

Martin Waitz <tali@admingilde.org> wrote:

Whoops.  :-)  Its fixed with the following commit in git-gui:

commit 372ef954a15c321a1065bf7e764449ce7767277f
Author: Shawn O. Pearce <spearce@spearce.org>
Date:   Sun Feb 18 02:12:32 2007 -0500

    git-gui: Correct crash when saving options in blame mode.
    
    Martin Waitz noticed that git-gui crashed while saving the user's
    options out if the application was started in blame mode.  This
    was caused by the do_save_config procedure invoking reshow_diff
    incase the number of context lines was modified by the user.
    Because we bypassed main window UI setup to enter blame mode we
    did not set many of the globals which were accessed by reshow_diff,
    and reading unset variables is an error in Tcl.
    
    Aside from moving the globals to be set earlier, I also modified
    reshow_diff to not invoke clear_diff if there is no path currently
    in the diff viewer.  This way reshow_diff does not crash when in
    blame mode due to the $ui_diff command not being defined.
    
    Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

-- 
Shawn.
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
exception in git-gui when saving options, Martin Waitz, (Sat Feb 17, 3:40 pm)
Re: exception in git-gui when saving options, Shawn O. Pearce, (Sun Feb 18, 3:14 am)
speck-geostationary