Merge pull request #17 from liny01-nbsa/patch-1

Update repeater.go, fix for #16

Author: mame

repeater.go

@@ -60,9 +60,19 @@ func newRepeater(ctx context.Context, powershell string, pipename string) (*repe
 		// wait for the process start up
 		go func() {
 			// the process should output "\xff" if it starts successfully
+			// ignore any output until we got "\xff"
 			buf := make([]byte, 1)
-			n, err := out.Read(buf)
-			done <- err == nil && n == 1 && buf[0] == 0xff
+			for {
+				n, err := out.Read(buf)
+				if err != nil {
+					done <- false
+					return
+				}
+				if n == 1 && buf[0] == 0xff {
+					done <- true
+					return
+				}
+			}
 		}()
 
 		select {

repeater_test.go

@@ -16,6 +16,7 @@ $stdin.read
 
 const dummyEchoPowerShell = `#!/usr/bin/ruby
 $stdout.sync = true
+$stdout << "Warning: Some dummy ramdom warming messages ... Dummy ...\n"
 $stdout << "\xff"
 loop do
   $stdout << $stdin.getc