git-gui: Always disable the Tcl EOF character when reading
authorShawn O. Pearce <spearce@spearce.org>
Tue, 17 Jul 2007 05:50:10 +0000 (01:50 -0400)
committerShawn O. Pearce <spearce@spearce.org>
Tue, 17 Jul 2007 05:50:10 +0000 (01:50 -0400)
On Windows (which includes Cygwin) Tcl defaults to leaving the EOF
character of input file streams set to the ASCII EOF character, but
if that character were to appear in the data stream then Tcl will
close the channel early.  So we have to disable eofchar on Windows.
Since the default is disabled on all platforms except Windows, we
can just disable it everywhere to prevent any sort of read problem.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
git-gui.sh
lib/blame.tcl
lib/commit.tcl
lib/diff.tcl

index 3808451..d0b75c0 100755 (executable)
@@ -414,6 +414,7 @@ proc _open_stdout_stderr {cmd} {
                        error $err
                }
        }
+       fconfigure $fd -eofchar {}
        return $fd
 }
 
@@ -828,6 +829,7 @@ proc load_message {file} {
                if {[catch {set fd [open $f r]}]} {
                        return 0
                }
+               fconfigure $fd -eofchar {}
                set content [string trim [read $fd]]
                close $fd
                regsub -all -line {[ \r\t]+$} $content {} content
index 4bdb9a2..1bcb4b4 100644 (file)
@@ -370,6 +370,7 @@ method _load {jump} {
        $w_path conf -text [escape_path $path]
        if {$commit eq {}} {
                set fd [open $path r]
+               fconfigure $fd -eofchar {}
        } else {
                set fd [git_read cat-file blob "$commit:$path"]
        }
index 46a78c1..0f3b16d 100644 (file)
@@ -209,7 +209,7 @@ A good commit message has the following format:
        ui_status {Calling pre-commit hook...}
        set pch_error {}
        set fd_ph [open "| $pchook" r]
-       fconfigure $fd_ph -blocking 0 -translation binary
+       fconfigure $fd_ph -blocking 0 -translation binary -eofchar {}
        fileevent $fd_ph readable \
                [list commit_prehook_wait $fd_ph $curHEAD $msg]
 }
index 9cb9d06..e09e125 100644 (file)
@@ -86,6 +86,7 @@ proc show_diff {path w {lno {}}} {
                set max_sz [expr {128 * 1024}]
                if {[catch {
                                set fd [open $path r]
+                               fconfigure $fd -eofchar {}
                                set content [read $fd $max_sz]
                                close $fd
                                set sz [file size $path]