interfaces: do not sync on output writers

Fixes: sync /dev/stdout: inappropriate ioctl for device When the writer is os.Stdout. (*os.File).Close() does flush the file to disk either way, so calling Sync() directly is redundant.
rjeczalik · May 1, 2019 · 073b0dc · 073b0dc
1 parent 4dd7b3e
commit 073b0dc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cmd/interfacer/main.go b/cmd/interfacer/main.go
@@ -92,7 +92,7 @@ func main() {
 			die(err)
 		}
 	}
-	err = nonil(tmpl.Execute(f, v), f.Sync(), f.Close())
+	err = nonil(tmpl.Execute(f, v), f.Close())
 	if err != nil {
 		die(err)
 	}

diff --git a/cmd/structer/main.go b/cmd/structer/main.go
@@ -171,7 +171,7 @@ func run() (err error) {
 		}
 	}
 
-	return nonil(tmpl.Execute(w, &v), appendTemplate(*typ, &v, w), w.Sync(), w.Close())
+	return nonil(tmpl.Execute(w, &v), appendTemplate(*typ, &v, w), w.Close())
 }
 
 var tmplFuncs = template.FuncMap{