skip transform if options don't request it

Fixes #64

data.go

@@ -72,8 +72,6 @@ type Options struct {
 	ScaleUp bool
 }
 
-var emptyOptions = Options{}
-
 func (o Options) String() string {
 	buf := new(bytes.Buffer)
 	fmt.Fprintf(buf, "%v%s%v", o.Width, optSizeDelimiter, o.Height)
@@ -101,6 +99,13 @@ func (o Options) String() string {
 	return buf.String()
 }
 
+// transform returns whether o includes transformation options.  Some fields
+// are not transform related at all (like Signature), and others only apply in
+// the presence of other fields (like Fit and Quality).
+func (o Options) transform() bool {
+	return o.Width != 0 || o.Height != 0 || o.Rotate != 0 || o.FlipHorizontal || o.FlipVertical
+}
+
 // ParseOptions parses str as a list of comma separated transformation options.
 // The following options can be specified in any order:
 //

data_test.go

@@ -19,6 +19,8 @@ import (
 	"testing"
 )
 
+var emptyOptions = Options{}
+
 func TestOptions_String(t *testing.T) {
 	tests := []struct {
 		Options Options

transform.go

@@ -35,7 +35,7 @@ var resampleFilter = imaging.Lanczos
 // encoded image in one of the supported formats (gif, jpeg, or png).  The
 // bytes of a similarly encoded image is returned.
 func Transform(img []byte, opt Options) ([]byte, error) {
-	if opt == emptyOptions {
+	if !opt.transform() {
 		// bail if no transformation was requested
 		return img, nil
 	}