]> Cypherpunks repositories - gostls13.git/commitdiff
encoding/json: remove unnecessary else conditions
authorbontequero <bontequero@gmail.com>
Mon, 9 Apr 2018 08:06:18 +0000 (11:06 +0300)
committerBrad Fitzpatrick <bradfitz@golang.org>
Mon, 9 Apr 2018 19:28:23 +0000 (19:28 +0000)
Fixes golint warning about "if block ends with a return statement, so drop this else and outdent its block".

Change-Id: Id17ad0bf37ba939386b177b709e9e3c067d8ba21
Reviewed-on: https://go-review.googlesource.com/105736
Run-TryBot: Iskander Sharipov <iskander.sharipov@intel.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Daniel Martí <mvdan@mvdan.cc>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/encoding/json/decode.go

index d3ada54b693d86e0370c12beb7d67abb202f59a5..9479e1a5c6675b7a9fde03eee5da3900f7c24be4 100644 (file)
@@ -896,9 +896,8 @@ func (d *decodeState) literalStore(item []byte, v reflect.Value, fromQuoted bool
                if !ok {
                        if fromQuoted {
                                return fmt.Errorf("json: invalid use of ,string struct tag, trying to unmarshal %q into %v", item, v.Type())
-                       } else {
-                               return errPhase
                        }
+                       return errPhase
                }
                err := ut.UnmarshalText(s)
                if err != nil {
@@ -952,9 +951,8 @@ func (d *decodeState) literalStore(item []byte, v reflect.Value, fromQuoted bool
                if !ok {
                        if fromQuoted {
                                return fmt.Errorf("json: invalid use of ,string struct tag, trying to unmarshal %q into %v", item, v.Type())
-                       } else {
-                               return errPhase
                        }
+                       return errPhase
                }
                switch v.Kind() {
                default:
@@ -985,9 +983,8 @@ func (d *decodeState) literalStore(item []byte, v reflect.Value, fromQuoted bool
                if c != '-' && (c < '0' || c > '9') {
                        if fromQuoted {
                                return fmt.Errorf("json: invalid use of ,string struct tag, trying to unmarshal %q into %v", item, v.Type())
-                       } else {
-                               return errPhase
                        }
+                       return errPhase
                }
                s := string(item)
                switch v.Kind() {
@@ -1001,9 +998,8 @@ func (d *decodeState) literalStore(item []byte, v reflect.Value, fromQuoted bool
                        }
                        if fromQuoted {
                                return fmt.Errorf("json: invalid use of ,string struct tag, trying to unmarshal %q into %v", item, v.Type())
-                       } else {
-                               return &UnmarshalTypeError{Value: "number", Type: v.Type(), Offset: int64(d.readIndex())}
                        }
+                       return &UnmarshalTypeError{Value: "number", Type: v.Type(), Offset: int64(d.readIndex())}
                case reflect.Interface:
                        n, err := d.convertNumber(s)
                        if err != nil {