Add a comment to clarify that we're handling the empty file case

tomfaulhaber · tomfaulhaber · commit 5a339690f8eb · 2020-07-03T20:26:16.000-07:00
Signed-off-by: Tom Faulhaber &lt;tffaulha@amazon.com&gt;
diff --git a/fs/path.go b/fs/path.go
@@ -123,7 +123,7 @@ func sameFile(f1, f2 *currentPath) (bool, error) {
 			} else if f1.f.Size() > 0 {
 				eq, err = compareFileContent(f1.fullPath, f2.fullPath)
 			} else {
-				eq, err = true, nil
+				eq, err = true, nil // if file sizes are zero length, the files are the same by definition
 			}
 			if err != nil || !eq {
 				return eq, err

Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ func sameFile(f1, f2 *currentPath) (bool, error) {`
`123`	`123`	`} else if f1.f.Size() > 0 {`
`124`	`124`	`eq, err = compareFileContent(f1.fullPath, f2.fullPath)`
`125`	`125`	`} else {`
`126`		`- eq, err = true, nil`
	`126`	`+ eq, err = true, nil // if file sizes are zero length, the files are the same by definition`
`127`	`127`	`}`
`128`	`128`	`if err != nil \|\| !eq {`
`129`	`129`	`return eq, err`