git-gui: Display the "Loading annotation..." message in italic
[gitweb.git] / lib / blame.tcl
index 1c3f5e985e0ecdbd6c7b297c870b1445fc5e2a6f..0641ea6d373d1e47ebf029818b5772629f68a238 100644 (file)
@@ -5,52 +5,59 @@ class blame {
 
 image create photo ::blame::img_back_arrow -data {R0lGODlhGAAYAIUAAPwCBEzKXFTSZIz+nGzmhGzqfGTidIT+nEzGXHTqhGzmfGzifFzadETCVES+VARWDFzWbHzyjAReDGTadFTOZDSyRDyyTCymPARaFGTedFzSbDy2TCyqRCyqPARaDAyCHES6VDy6VCyiPAR6HCSeNByWLARyFARiDARqFGTifARiFAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAACH5BAEAAAAALAAAAAAYABgAAAajQIBwSCwaj8ikcsk0BppJwRPqHEypQwHBis0WDAdEFyBIKBaMAKLBdjQeSkFBYTBAIvgEoS6JmhUTEwIUDQ4VFhcMGEhyCgoZExoUaxsWHB0THkgfAXUGAhoBDSAVFR0XBnCbDRmgog0hpSIiDJpJIyEQhBUcJCIlwA22SSYVogknEg8eD82qSigdDSknY0IqJQXPYxIl1dZCGNvWw+Dm510GQQAh/mhDcmVhdGVkIGJ5IEJNUFRvR0lGIFBybyB2ZXJzaW9uIDIuNQ0KqSBEZXZlbENvciAxOTk3LDE5OTguIEFsbCByaWdodHMgcmVzZXJ2ZWQuDQpodHRwOi8vd3d3LmRldmVsY29yLmNvbQA7}
 
-field commit    ; # input commit to blame
-field path      ; # input filename to view in $commit
+# Persistant data (survives loads)
+#
 field history {}; # viewer history: {commit path}
+field header    ; # array commit,key -> header field
 
+# Tk UI control paths
+#
 field w          ; # top window in this viewer
 field w_back     ; # our back button
 field w_path     ; # label showing the current file path
+field w_columns  ; # list of all column widgets in the viewer
 field w_line     ; # text column: all line numbers
-field w_cgrp     ; # text column: abbreviated commit SHA-1s
+field w_amov     ; # text column: annotations + move tracking
 field w_file     ; # text column: actual file data
-field w_cmit     ; # pane showing commit message
+field w_cviewer  ; # pane showing commit message
 field status     ; # text variable bound to status bar
 field old_height ; # last known height of $w.file_pane
 
-field current_fd       {} ; # background process running
-field highlight_line   -1 ; # current line selected
-field highlight_commit {} ; # sha1 of commit selected
-
-field total_lines       0  ; # total length of file
-field blame_lines       0  ; # number of lines computed
-field commit_count      0  ; # number of commits in $commit_list
-field commit_list      {}  ; # list of commit sha1 in receipt order
-field order                ; # array commit -> receipt order
-field header               ; # array commit,key -> header field
-field line_commit          ; # array line -> sha1 commit
-field line_file            ; # array line -> file name
-
-field r_commit      ; # commit currently being parsed
-field r_orig_line   ; # original line number
-field r_final_line  ; # final line number
-field r_line_count  ; # lines in this region
-
-field tooltip_wm     {} ; # Current tooltip toplevel, if open
-field tooltip_timer  {} ; # Current timer event for our tooltip
-field tooltip_commit {} ; # Commit in tooltip
-field tooltip_text   {} ; # Text in current tooltip
-
-variable active_color #c0edc5
-variable group_colors {
+# Tk UI colors
+#
+field active_color #c0edc5
+field group_colors {
        #d6d6d6
        #e1e1e1
        #ececec
 }
 
+# Current blame data; cleared/reset on each load
+#
+field commit               ; # input commit to blame
+field path                 ; # input filename to view in $commit
+
+field current_fd        {} ; # background process running
+field highlight_line    -1 ; # current line selected
+field highlight_commit  {} ; # sha1 of commit selected
+field old_bgcolor       {} ; # background of current selection
+
+field total_lines       0  ; # total length of file
+field blame_lines       0  ; # number of lines computed
+field have_commit          ; # array commit -> 1
+field amov_data            ; # list of {commit origfile origline}
+
+field r_commit             ; # commit currently being parsed
+field r_orig_line          ; # original line number
+field r_final_line         ; # final line number
+field r_line_count         ; # lines in this region
+
+field tooltip_wm        {} ; # Current tooltip toplevel, if open
+field tooltip_timer     {} ; # Current timer event for our tooltip
+field tooltip_commit    {} ; # Commit in tooltip
+field tooltip_text      {} ; # Text in current tooltip
+
 constructor new {i_commit i_path} {
-       variable active_color
        global cursor_ptr
 
        set commit $i_commit
@@ -122,12 +129,12 @@ constructor new {i_commit i_path} {
                -state disabled \
                -wrap none \
                -height 40 \
-               -width 5 \
+               -width 6 \
                -font font_diff
        $w_line tag conf linenumber -justify right -rmargin 5
 
-       set w_cgrp $w.file_pane.out.commit_t
-       text $w_cgrp \
+       set w_amov $w.file_pane.out.amove_t
+       text $w_amov \
                -takefocus 0 \
                -highlightthickness 0 \
                -padx 0 -pady 0 \
@@ -137,11 +144,11 @@ constructor new {i_commit i_path} {
                -height 40 \
                -width 4 \
                -font font_diff
-       $w_cgrp tag conf curr_commit
-       $w_cgrp tag conf prior_commit \
+       $w_amov tag conf curr_commit
+       $w_amov tag conf prior_commit \
                -foreground blue \
                -underline 1
-       $w_cgrp tag bind prior_commit \
+       $w_amov tag bind prior_commit \
                <Button-1> \
                "[cb _load_commit @%x,%y];break"
 
@@ -158,28 +165,27 @@ constructor new {i_commit i_path} {
                -xscrollcommand [list $w.file_pane.out.sbx set] \
                -font font_diff
 
+       set w_columns [list $w_amov $w_line $w_file]
+
        scrollbar $w.file_pane.out.sbx \
                -orient h \
                -command [list $w_file xview]
        scrollbar $w.file_pane.out.sby \
                -orient v \
-               -command [list scrollbar2many [list \
-               $w_line \
-               $w_cgrp \
-               $w_file \
-               ] yview]
-       grid \
-               $w_cgrp \
-               $w_line \
-               $w_file \
-               $w.file_pane.out.sby \
-               -sticky nsew
-       grid conf $w.file_pane.out.sbx -column 2 -sticky we
-       grid columnconfigure $w.file_pane.out 2 -weight 1
+               -command [list scrollbar2many $w_columns yview]
+       eval grid $w_columns $w.file_pane.out.sby -sticky nsew
+       grid conf \
+               $w.file_pane.out.sbx \
+               -column [expr {[llength $w_columns] - 1}] \
+               -sticky we
+       grid columnconfigure \
+               $w.file_pane.out \
+               [expr {[llength $w_columns] - 1}] \
+               -weight 1
        grid rowconfigure $w.file_pane.out 0 -weight 1
 
-       set w_cmit $w.file_pane.cm.t
-       text $w_cmit \
+       set w_cviewer $w.file_pane.cm.t
+       text $w_cviewer \
                -background white -borderwidth 0 \
                -state disabled \
                -wrap none \
@@ -188,23 +194,26 @@ constructor new {i_commit i_path} {
                -xscrollcommand [list $w.file_pane.cm.sbx set] \
                -yscrollcommand [list $w.file_pane.cm.sby set] \
                -font font_diff
-       $w_cmit tag conf header_key \
+       $w_cviewer tag conf still_loading \
+               -font font_uiitalic \
+               -justify center
+       $w_cviewer tag conf header_key \
                -tabs {3c} \
                -background $active_color \
                -font font_uibold
-       $w_cmit tag conf header_val \
+       $w_cviewer tag conf header_val \
                -background $active_color \
                -font font_ui
-       $w_cmit tag raise sel
+       $w_cviewer tag raise sel
        scrollbar $w.file_pane.cm.sbx \
                -orient h \
-               -command [list $w_cmit xview]
+               -command [list $w_cviewer xview]
        scrollbar $w.file_pane.cm.sby \
                -orient v \
-               -command [list $w_cmit yview]
+               -command [list $w_cviewer yview]
        pack $w.file_pane.cm.sby -side right -fill y
        pack $w.file_pane.cm.sbx -side bottom -fill x
-       pack $w_cmit -expand 1 -fill both
+       pack $w_cviewer -expand 1 -fill both
 
        frame $w.status \
                -borderwidth 1 \
@@ -220,17 +229,10 @@ constructor new {i_commit i_path} {
                -label "Copy Commit" \
                -command [cb _copycommit]
 
-       foreach i [list \
-               $w_cgrp \
-               $w_line \
-               $w_file] {
+       foreach i $w_columns {
                $i conf -cursor $cursor_ptr
-               $i conf -yscrollcommand \
-                       [list many2scrollbar [list \
-                       $w_cgrp \
-                       $w_line \
-                       $w_file \
-                       ] yview $w.file_pane.out.sby]
+               $i conf -yscrollcommand [list many2scrollbar \
+                       $w_columns yview $w.file_pane.out.sby]
                bind $i <Button-1> "
                        [cb _hide_tooltip]
                        [cb _click $i @%x,%y]
@@ -248,11 +250,7 @@ constructor new {i_commit i_path} {
                "
        }
 
-       foreach i [list \
-               $w_cgrp \
-               $w_line \
-               $w_file \
-               $w_cmit] {
+       foreach i [concat $w_columns $w_cviewer] {
                bind $i <Key-Up>        {catch {%W yview scroll -1 units};break}
                bind $i <Key-Down>      {catch {%W yview scroll  1 units};break}
                bind $i <Key-Left>      {catch {%W xview scroll -1 units};break}
@@ -265,7 +263,7 @@ constructor new {i_commit i_path} {
                bind $i <Control-Key-f> {catch {%W yview scroll  1 pages};break}
        }
 
-       bind $w_cmit <Button-1> [list focus $w_cmit]
+       bind $w_cviewer <Button-1> [list focus $w_cviewer]
        bind $top <Visibility> [list focus $top]
        bind $w_file <Destroy> [list delete_this $this]
 
@@ -304,27 +302,20 @@ method _load {} {
                        set current_fd {}
                }
 
+               foreach i $w_columns {
+                       $i conf -state normal
+                       $i delete 0.0 end
+                       foreach cmit [array names have_commit] {
+                               $i tag delete g$cmit
+                       }
+                       $i conf -state disabled
+               }
+
                set highlight_line -1
                set highlight_commit {}
                set total_lines 0
                set blame_lines 0
-               set commit_count 0
-               set commit_list {}
-               array unset order
-               array unset line_commit
-               array unset line_file
-
-               $w_cgrp conf -state normal
-               $w_line conf -state normal
-               $w_file conf -state normal
-
-               $w_cgrp delete 0.0 end
-               $w_line delete 0.0 end
-               $w_file delete 0.0 end
-
-               $w_cgrp conf -state disabled
-               $w_line conf -state disabled
-               $w_file conf -state disabled
+               array unset have_commit
        }
 
        if {[winfo exists $w.status.c]} {
@@ -347,6 +338,12 @@ method _load {} {
        }
        lappend history [list $commit $path]
 
+       # Index 0 is always empty.  There is never line 0 as
+       # we use only 1 based lines, as that matches both with
+       # git-blame output and with Tk's text widget.
+       #
+       set amov_data [list [list]]
+
        set status "Loading $commit:[escape_path $path]..."
        $w_path conf -text [escape_path $path]
        if {$commit eq {}} {
@@ -376,6 +373,8 @@ method _history_menu {} {
 
                if {[regexp {^[0-9a-f]{40}$} $c]} {
                        set t [string range $c 0 8]...
+               } elseif {$c eq {}} {
+                       set t {Working Directory}
                } else {
                        set t $c
                }
@@ -406,28 +405,30 @@ method _read_file {fd} {
                return
        }
 
-       $w_cgrp conf -state normal
-       $w_line conf -state normal
-       $w_file conf -state normal
+       foreach i $w_columns {$i conf -state normal}
        while {[gets $fd line] >= 0} {
                regsub "\r\$" $line {} line
                incr total_lines
+               lappend amov_data {}
 
                if {$total_lines > 1} {
-                       $w_cgrp insert end "\n"
-                       $w_line insert end "\n"
-                       $w_file insert end "\n"
+                       foreach i $w_columns {$i insert end "\n"}
                }
 
                $w_line insert end "$total_lines" linenumber
                $w_file insert end "$line"
        }
-       $w_cgrp conf -state disabled
-       $w_line conf -state disabled
-       $w_file conf -state disabled
+
+       set ln_wc [expr {[string length $total_lines] + 2}]
+       if {[$w_line cget -width] < $ln_wc} {
+               $w_line conf -width $ln_wc
+       }
+
+       foreach i $w_columns {$i conf -state disabled}
 
        if {[eof $fd]} {
                close $fd
+
                _status $this
                set cmd {nice git blame -M -C --incremental}
                if {$commit eq {}} {
@@ -444,14 +445,12 @@ method _read_file {fd} {
 } ifdeleted { catch {close $fd} }
 
 method _read_blame {fd} {
-       variable group_colors
-
        if {$fd ne $current_fd} {
                catch {close $fd}
                return
        }
 
-       $w_cgrp conf -state normal
+       $w_amov conf -state normal
        while {[gets $fd line] >= 0} {
                if {[regexp {^([a-z0-9]{40}) (\d+) (\d+) (\d+)$} $line line \
                        cmit original_line final_line line_count]} {
@@ -460,18 +459,14 @@ method _read_blame {fd} {
                        set r_final_line $final_line
                        set r_line_count $line_count
 
-                       if {[catch {set g $order($cmit)}]} {
+                       if {[catch {set g $have_commit($cmit)}]} {
                                set bg [lindex $group_colors 0]
                                set group_colors [lrange $group_colors 1 end]
                                lappend group_colors $bg
-
-                               $w_cgrp tag conf g$cmit -background $bg
-                               $w_line tag conf g$cmit -background $bg
-                               $w_file tag conf g$cmit -background $bg
-
-                               set order($cmit) $commit_count
-                               incr commit_count
-                               lappend commit_list $cmit
+                               foreach i $w_columns {
+                                       $i tag conf g$cmit -background $bg
+                               }
+                               set have_commit($cmit) 1
                        }
                } elseif {[string match {filename *} $line]} {
                        set file [string range $line 9 end]
@@ -513,45 +508,35 @@ method _read_blame {fd} {
 
                        set first_lno $lno
                        while {
-                          ![catch {set ncmit $line_commit([expr {$first_lno - 1}])}]
-                       && ![catch {set nfile $line_file([expr {$first_lno - 1}])}]
-                       && $ncmit eq $cmit
-                       && $nfile eq $file
+                          $first_lno > 1
+                       && $cmit eq [lindex $amov_data [expr {$first_lno - 1}] 0]
+                       && $file eq [lindex $amov_data [expr {$first_lno - 1}] 1]
                        } {
                                incr first_lno -1
                        }
 
                        while {$n > 0} {
                                set lno_e "$lno.0 lineend + 1c"
-                               if {![catch {set g g$line_commit($lno)}]} {
-                                       $w_cgrp tag remove g$g $lno.0 $lno_e
-                                       $w_line tag remove g$g $lno.0 $lno_e
-                                       $w_file tag remove g$g $lno.0 $lno_e
-
-                                       $w_cgrp tag remove a$g $lno.0 $lno_e
-                                       $w_line tag remove a$g $lno.0 $lno_e
-                                       $w_file tag remove a$g $lno.0 $lno_e
+                               if {[lindex $amov_data $lno] ne {}} {
+                                       set g [lindex $amov_data $lno 0]
+                                       foreach i $w_columns {
+                                               $i tag remove g$g $lno.0 $lno_e
+                                       }
                                }
+                               lset amov_data $lno [list $cmit $file]
 
-                               set line_commit($lno) $cmit
-                               set line_file($lno)   $file
-
-                               $w_cgrp delete $lno.0 "$lno.0 lineend"
+                               $w_amov delete $lno.0 "$lno.0 lineend"
                                if {$lno == $first_lno} {
-                                       $w_cgrp insert $lno.0 $commit_abbr $commit_type
+                                       $w_amov insert $lno.0 $commit_abbr $commit_type
                                } elseif {$lno == [expr {$first_lno + 1}]} {
-                                       $w_cgrp insert $lno.0 $author_abbr
+                                       $w_amov insert $lno.0 $author_abbr
                                } else {
-                                       $w_cgrp insert $lno.0 { |}
+                                       $w_amov insert $lno.0 { |}
                                }
 
-                               $w_cgrp tag add g$cmit $lno.0 $lno_e
-                               $w_line tag add g$cmit $lno.0 $lno_e
-                               $w_file tag add g$cmit $lno.0 $lno_e
-
-                               $w_cgrp tag add a$cmit $lno.0 $lno_e
-                               $w_line tag add a$cmit $lno.0 $lno_e
-                               $w_file tag add a$cmit $lno.0 $lno_e
+                               foreach i $w_columns {
+                                       $i tag add g$cmit $lno.0 $lno_e
+                               }
 
                                if {$highlight_line == -1} {
                                        if {[lindex [$w_file yview] 0] == 0} {
@@ -568,19 +553,17 @@ method _read_blame {fd} {
                        }
 
                        while {
-                          ![catch {set ncmit $line_commit($lno)}]
-                       && ![catch {set nfile $line_file($lno)}]
-                       && $ncmit eq $cmit
-                       && $nfile eq $file
+                          $cmit eq [lindex $amov_data $lno 0]
+                       && $file eq [lindex $amov_data $lno 1]
                        } {
-                               $w_cgrp delete $lno.0 "$lno.0 lineend"
+                               $w_amov delete $lno.0 "$lno.0 lineend"
 
                                if {$lno == $first_lno} {
-                                       $w_cgrp insert $lno.0 $commit_abbr $commit_type
+                                       $w_amov insert $lno.0 $commit_abbr $commit_type
                                } elseif {$lno == [expr {$first_lno + 1}]} {
-                                       $w_cgrp insert $lno.0 $author_abbr
+                                       $w_amov insert $lno.0 $author_abbr
                                } else {
-                                       $w_cgrp insert $lno.0 { |}
+                                       $w_amov insert $lno.0 { |}
                                }
                                incr lno
                        }
@@ -589,7 +572,7 @@ method _read_blame {fd} {
                        set header($r_commit,$key) $data
                }
        }
-       $w_cgrp conf -state disabled
+       $w_amov conf -state disabled
 
        if {[eof $fd]} {
                close $fd
@@ -620,35 +603,39 @@ method _click {cur_w pos} {
 }
 
 method _load_commit {pos} {
-       set lno [lindex [split [$w_cgrp index $pos] .] 0]
-       if {[catch {set cmit $line_commit($lno)}]} return
-       if {[catch {set file $line_file($lno)  }]} return
-
-       set commit $cmit
-       set path $file
-       _load $this
+       set lno [lindex [split [$w_amov index $pos] .] 0]
+       set dat [lindex $amov_data $lno]
+       if {$dat ne {}} {
+               set commit [lindex $dat 0]
+               set path   [lindex $dat 1]
+               _load $this
+       }
 }
 
 method _showcommit {lno} {
        global repo_config
-       variable active_color
 
        if {$highlight_commit ne {}} {
-               set cmit $highlight_commit
-               $w_cgrp tag conf a$cmit -background {}
-               $w_line tag conf a$cmit -background {}
-               $w_file tag conf a$cmit -background {}
+               foreach i $w_columns {
+                       $i tag conf g$highlight_commit -background $old_bgcolor
+               }
        }
 
-       $w_cmit conf -state normal
-       $w_cmit delete 0.0 end
-       if {[catch {set cmit $line_commit($lno)}]} {
+       $w_cviewer conf -state normal
+       $w_cviewer delete 0.0 end
+
+       set dat [lindex $amov_data $lno]
+       if {$dat eq {}} {
                set cmit {}
-               $w_cmit insert end "Loading annotation..."
+               $w_cviewer insert end "Loading annotation..." still_loading
        } else {
-               $w_cgrp tag conf a$cmit -background $active_color
-               $w_line tag conf a$cmit -background $active_color
-               $w_file tag conf a$cmit -background $active_color
+               set cmit [lindex $dat 0]
+               set file [lindex $dat 1]
+
+               set old_bgcolor [$w_file tag cget g$cmit -background]
+               foreach i $w_columns {
+                       $i tag conf g$cmit -background $active_color
+               }
 
                set author_name {}
                set author_email {}
@@ -696,23 +683,23 @@ method _showcommit {lno} {
                        set header($cmit,message) $msg
                }
 
-               $w_cmit insert end "commit $cmit\n" header_key
-               $w_cmit insert end "Author:\t" header_key
-               $w_cmit insert end "$author_name $author_email" header_val
-               $w_cmit insert end "$author_time\n" header_val
+               $w_cviewer insert end "commit $cmit\n" header_key
+               $w_cviewer insert end "Author:\t" header_key
+               $w_cviewer insert end "$author_name $author_email" header_val
+               $w_cviewer insert end "  $author_time\n" header_val
 
-               $w_cmit insert end "Committer:\t" header_key
-               $w_cmit insert end "$committer_name $committer_email" header_val
-               $w_cmit insert end "$committer_time\n" header_val
+               $w_cviewer insert end "Committer:\t" header_key
+               $w_cviewer insert end "$committer_name $committer_email" header_val
+               $w_cviewer insert end "  $committer_time\n" header_val
 
-               if {$line_file($lno) ne $path} {
-                       $w_cmit insert end "Original File:\t" header_key
-                       $w_cmit insert end "[escape_path $line_file($lno)]\n" header_val
+               if {$file ne $path} {
+                       $w_cviewer insert end "Original File:\t" header_key
+                       $w_cviewer insert end "[escape_path $file]\n" header_val
                }
 
-               $w_cmit insert end "\n$msg"
+               $w_cviewer insert end "\n$msg"
        }
-       $w_cmit conf -state disabled
+       $w_cviewer conf -state disabled
 
        set highlight_line $lno
        set highlight_commit $cmit
@@ -725,21 +712,24 @@ method _showcommit {lno} {
 method _copycommit {} {
        set pos @$::cursorX,$::cursorY
        set lno [lindex [split [$::cursorW index $pos] .] 0]
-       if {![catch {set commit $line_commit($lno)}]} {
+       set dat [lindex $amov_data $lno]
+       if {$dat ne {}} {
                clipboard clear
                clipboard append \
                        -format STRING \
                        -type STRING \
-                       -- $commit
+                       -- [lindex $dat 0]
        }
 }
 
 method _show_tooltip {cur_w pos} {
        set lno [lindex [split [$cur_w index $pos] .] 0]
-       if {[catch {set cmit $line_commit($lno)}]} {
+       set dat [lindex $amov_data $lno]
+       if {$dat eq {}} {
                _hide_tooltip $this
                return
        }
+       set cmit [lindex $dat 0]
 
        if {$cmit eq $highlight_commit} {
                _hide_tooltip $this
@@ -768,7 +758,9 @@ method _open_tooltip {cur_w} {
                [expr {$pos_x - [winfo rootx $cur_w]}] \
                [expr {$pos_y - [winfo rooty $cur_w]}]] ,]
        set lno [lindex [split [$cur_w index $pos] .] 0]
-       set cmit $line_commit($lno)
+       set dat [lindex $amov_data $lno]
+       set cmit [lindex $dat 0]
+       set file [lindex $dat 1]
 
        set author_name {}
        set author_email {}
@@ -798,7 +790,6 @@ method _open_tooltip {cur_w} {
 $author_name $author_email  $author_time
 $summary"
 
-       set file $line_file($lno)
        if {$file ne $path} {
                append tooltip_text "