Issue 17 (#18)

* update version

* add defer close for dst file descriptor for non-atomic copy

Author: swtch1

VERSION

@@ -1 +1 @@
-v0.1.4
+v0.1.5

copy.go

@@ -30,9 +30,7 @@ type File struct {
 
 // NewFile creates a new File.
 func NewFile(path string) *File {
-	return &File{
-		Path: path,
-	}
+	return &File{Path: path}
 }
 
 // setInfo will collect information about a File and populate the necessary fields.
@@ -278,6 +276,11 @@ func copyFile(srcFile, dstFile *File, opts Options) (err error) {
 		if err != nil {
 			return errors.Wrapf(ErrCannotOpenOrCreateDstFile, "destination file %s: %s", dstFile.Path, err)
 		}
+		defer func() {
+			if closeErr := dstFD.Close(); closeErr != nil {
+				err = closeErr
+			}
+		}()
 
 		opts.logInfo("copying src file %s to dst file %s", srcFD.Name(), dstFD.Name())
 		if _, err = io.Copy(dstFD, srcFD); err != nil {

option.go

@@ -28,7 +28,7 @@ type Options struct {
 	// NoClobber will not let an existing file be overwritten.
 	NoClobber bool
 	// Parents will create source directories in dst if they do not already exist. ErrWithParentsDstMustBeDir
-	// is returned is destination is not a directory.
+	// is returned if destination is not a directory.
 	Parents bool
 	// Recursive will recurse through sub directories if set true.
 	Recursive bool