refactor (*UUID).UnmarshalText (#58)

prochac · web-flow · commit cb32006e483f · 2020-05-19T09:17:26.000-05:00
Now it's easier to set breakpoint on error for debugging.
diff --git a/marshal.go b/marshal.go
@@ -16,10 +16,11 @@ func (uuid UUID) MarshalText() ([]byte, error) {
 // UnmarshalText implements encoding.TextUnmarshaler.
 func (uuid *UUID) UnmarshalText(data []byte) error {
 	id, err := ParseBytes(data)
-	if err == nil {
-		*uuid = id
+	if err != nil {
+		return err
 	}
-	return err
+	*uuid = id
+	return nil
 }
 
 // MarshalBinary implements encoding.BinaryMarshaler.

Original file line number	Diff line number	Diff line change
`@@ -16,10 +16,11 @@ func (uuid UUID) MarshalText() ([]byte, error) {`
`16`	`16`	`// UnmarshalText implements encoding.TextUnmarshaler.`
`17`	`17`	`func (uuid *UUID) UnmarshalText(data []byte) error {`
`18`	`18`	`id, err := ParseBytes(data)`
`19`		`- if err == nil {`
`20`		`- *uuid = id`
	`19`	`+ if err != nil {`
	`20`	`+ return err`
`21`	`21`	`}`
`22`		`- return err`
	`22`	`+ *uuid = id`
	`23`	`+ return nil`
`23`	`24`	`}`
`24`	`25`
`25`	`26`	`// MarshalBinary implements encoding.BinaryMarshaler.`