Also print fixed ... messages with -stdout (#1373)

* Also print `fixed ...` messages with `-stdout`

This makes it easier for users to implement a "check only" mode that doesn't modify any files, but is able to tell which ones would have been changed by buildozer.

* Use diff

Author: fmeum

buildozer/buildozer_test.sh

@@ -2393,4 +2393,34 @@ EOF
   diff -u MODULE.bazel.expected MODULE.bazel || fail "Output didn't match"
 }
 
+function test_stdout() {
+  cat > MODULE.bazel <<EOF
+module(
+  name = "foo", version = "0.27.0",
+)
+EOF
+
+  cat > MODULE.bazel.expected <<EOF
+module(
+  name = "foo", version = "0.27.0",
+)
+EOF
+
+  cat > MODULE.bazel.expected.stdout <<EOF
+module(
+    name = "foo",
+    version = "0.27.0",
+)
+EOF
+
+  cat > MODULE.bazel.expected.stderr <<EOF
+fixed $(pwd)/MODULE.bazel
+EOF
+
+  $buildozer -stdout 'format' //MODULE.bazel:all > stdout 2> stderr
+  diff -u MODULE.bazel.expected MODULE.bazel || fail "File was changed"
+  diff -u MODULE.bazel.expected.stdout stdout || fail "Output didn't match"
+  diff -u MODULE.bazel.expected.stderr stderr || fail "Error output didn't match"
+}
+
 run_suite "buildozer tests"

edit/buildozer.go

@@ -1260,7 +1260,7 @@ func rewrite(opts *Options, commandsForFile commandsForFile) *rewriteResult {
 
 	if opts.Stdout || name == stdinPackageName {
 		opts.OutWriter.Write(ndata)
-		return &rewriteResult{file: name, errs: errs, records: records}
+		return &rewriteResult{file: name, errs: errs, modified: !bytes.Equal(data, ndata), records: records}
 	}
 
 	if bytes.Equal(data, ndata) {