~sircmpwn/annotatego

bf211688254ff94c3221640925c4608e4738c9d1 — Chris Waldon 2 years ago 02e9b7f master
bugfix: ensure we don't Wait() for subprocess twice

My previous patch introduced this problem. Run() is a combination
of Start() and Wait(), and I meant to switch the first Run() call into
Start() in my last patch (I even claim to have done so in the commit
message). I think this got messed up by a bad interactive rebase on my
part. Either way, this commit should prevent the program from panicking
due to a double-Wait().
1 files changed, 1 insertions(+), 1 deletions(-)

M main.go
M main.go => main.go +1 -1
@@ 67,7 67,7 @@ func crawlGitTree() map[string]string {
	}
	defer pipe.Close()
	scanner := bufio.NewScanner(pipe)
	if err := cmd.Run(); err != nil {
	if err := cmd.Start(); err != nil {
		panic(err)
	}
	scanner.Split(scanNulls)