]> Cypherpunks.ru repositories - gostls13.git/commitdiff
cmd: some semi-automated cleanups
authorDaniel Martí <mvdan@mvdan.cc>
Thu, 5 Apr 2018 20:28:06 +0000 (21:28 +0100)
committerDaniel Martí <mvdan@mvdan.cc>
Fri, 6 Apr 2018 13:59:29 +0000 (13:59 +0000)
* Remove some redundant returns
* Replace HasPrefix with TrimPrefix
* Remove some obviously dead code

Passes toolstash -cmp on std cmd.

Change-Id: Ifb0d70a45cbb8a8553758a8c4878598b7fe932bc
Reviewed-on: https://go-review.googlesource.com/105017
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
13 files changed:
src/cmd/asm/internal/arch/arm.go
src/cmd/compile/internal/gc/ssa.go
src/cmd/go/internal/test/test.go
src/cmd/internal/obj/arm/asm5.go
src/cmd/internal/obj/arm64/asm7.go
src/cmd/internal/obj/mips/asm0.go
src/cmd/internal/obj/ppc64/asm9.go
src/cmd/internal/obj/x86/a.out.go
src/cmd/internal/obj/x86/asm6.go
src/cmd/internal/objabi/flag.go
src/cmd/internal/objfile/disasm.go
src/cmd/link/internal/loadpe/ldpe.go
src/cmd/vet/method.go

index 6e86ac0fbe278acf434ea0053853eac19fe16cf7..b0e985f0bc92a5ac5b11d06483b029692dcfa534 100644 (file)
@@ -223,9 +223,7 @@ func ParseARMCondition(cond string) (uint8, bool) {
 }
 
 func parseARMCondition(cond string, ls, scond map[string]uint8) (uint8, bool) {
-       if strings.HasPrefix(cond, ".") {
-               cond = cond[1:]
-       }
+       cond = strings.TrimPrefix(cond, ".")
        if cond == "" {
                return arm.C_SCOND_NONE, true
        }
index a2a67ee72823442c7f5ce0555b7b0a31809c3fc8..fcb6223528e88cdfb8c13773a5d2a51458facf59 100644 (file)
@@ -226,7 +226,6 @@ func (s *state) updateUnsetPredPos(b *ssa.Block) {
                p.Pos = bestPos
                s.updateUnsetPredPos(p) // We do not expect long chains of these, thus recursion is okay.
        }
-       return
 }
 
 type state struct {
index 7f14ce3cd73fba3c5003c21207a164dec45e0e6b..b82484a12cfcf0e7fbe5554a398108f65c39057c 100644 (file)
@@ -722,9 +722,7 @@ func runTest(cmd *base.Command, args []string) {
                buildTest, runTest, printTest, err := builderTest(&b, p)
                if err != nil {
                        str := err.Error()
-                       if strings.HasPrefix(str, "\n") {
-                               str = str[1:]
-                       }
+                       str = strings.TrimPrefix(str, "\n")
                        failed := fmt.Sprintf("FAIL\t%s [setup failed]\n", p.ImportPath)
 
                        if p.ImportPath != "" {
index ef4655357d41623625ab34a2f902de44ba161359..a88ae74d311d5efc06fb7f37a55dd5c8fcf430f3 100644 (file)
@@ -2844,7 +2844,6 @@ func (c *ctxt5) asmout(p *obj.Prog, o *Optab, out []uint32) {
        out[3] = o4
        out[4] = o5
        out[5] = o6
-       return
 }
 
 func (c *ctxt5) movxt(p *obj.Prog) uint32 {
@@ -3315,8 +3314,7 @@ func (c *ctxt5) ofsr(a obj.As, r int, v int32, b int, sc int, p *obj.Prog) uint3
 
 // MOVW $"lower 16-bit", Reg
 func (c *ctxt5) omvs(p *obj.Prog, a *obj.Addr, dr int) uint32 {
-       var o1 uint32
-       o1 = ((uint32(p.Scond) & C_SCOND) ^ C_SCOND_XOR) << 28
+       o1 := ((uint32(p.Scond) & C_SCOND) ^ C_SCOND_XOR) << 28
        o1 |= 0x30 << 20
        o1 |= (uint32(dr) & 15) << 12
        o1 |= uint32(a.Offset) & 0x0fff
index 98d61416322f60fd8e0d8d89fb3a72b0d2884129..45d793ce5934a467abec3162831479110e9d6202 100644 (file)
@@ -1533,9 +1533,6 @@ func (c *ctxt7) oplook(p *obj.Prog) *Optab {
        }
 
        c.ctxt.Diag("illegal combination: %v %v %v %v, %d %d", p, DRconv(a1), DRconv(a2), DRconv(a3), p.From.Type, p.To.Type)
-       if ops == nil {
-               ops = optab
-       }
        // Turn illegal instruction into an UNDEF, avoid crashing in asmout
        return &Optab{obj.AUNDEF, C_NONE, C_NONE, C_NONE, 90, 4, 0, 0, 0}
 }
@@ -4264,7 +4261,6 @@ func (c *ctxt7) asmout(p *obj.Prog, o *Optab, out []uint32) {
        out[2] = o3
        out[3] = o4
        out[4] = o5
-       return
 }
 
 /*
index 94296f9351192c247bf03923230d96e91ef316e0..0a391b978bb5cbda11226002dfd9e92df4822908 100644 (file)
@@ -739,9 +739,6 @@ func (c *ctxt0) oplook(p *obj.Prog) *Optab {
 
        c.ctxt.Diag("illegal combination %v %v %v %v", p.As, DRconv(a1), DRconv(a2), DRconv(a3))
        prasm(p)
-       if ops == nil {
-               ops = optab
-       }
        // Turn illegal instruction into an UNDEF, avoid crashing in asmout.
        return &Optab{obj.AUNDEF, C_NONE, C_NONE, C_NONE, 49, 4, 0, 0}
 }
@@ -1602,7 +1599,6 @@ func (c *ctxt0) asmout(p *obj.Prog, o *Optab, out []uint32) {
        out[1] = o2
        out[2] = o3
        out[3] = o4
-       return
 }
 
 func (c *ctxt0) vregoff(a *obj.Addr) int64 {
index 92365e917861d06a18b390c17771b29f7cb5e7ec..bfa17fc3b0a2fe6367c3af38c851992370c48091 100644 (file)
@@ -3560,7 +3560,6 @@ func (c *ctxt9) asmout(p *obj.Prog, o *Optab, out []uint32) {
        out[2] = o3
        out[3] = o4
        out[4] = o5
-       return
 }
 
 func (c *ctxt9) vregoff(a *obj.Addr) int64 {
index 13bef85c298f24df1c6f6d17cb04892596908992..f39882c82f2af53f122bcac74720a9231742988b 100644 (file)
@@ -32,11 +32,6 @@ package x86
 
 import "cmd/internal/obj"
 
-// mark flags
-const (
-       DONE = 1 << iota
-)
-
 const (
        REG_NONE = 0
 )
index 136dd0e35edb725973aa0d24dcb59164cc81eeb4..d817a89af7cf1f48d36d9cadcd5ae5f7c9cf4fb1 100644 (file)
@@ -3168,7 +3168,6 @@ bas:
 bad:
        ctxt.Diag("asmidx: bad address %d/%d/%d", scale, index, base)
        ab.Put1(0)
-       return
 }
 
 func (ab *AsmBuf) relput4(ctxt *obj.Link, cursym *obj.LSym, p *obj.Prog, a *obj.Addr) {
@@ -3436,7 +3435,6 @@ putrelv:
 
 bad:
        ctxt.Diag("asmand: bad address %v", obj.Dconv(p, a))
-       return
 }
 
 func (ab *AsmBuf) asmand(ctxt *obj.Link, cursym *obj.LSym, p *obj.Prog, a *obj.Addr, ra *obj.Addr) {
@@ -4400,8 +4398,6 @@ func (ab *AsmBuf) doasm(ctxt *obj.Link, cursym *obj.LSym, p *obj.Prog) {
                                        ab.PutInt32(0)
                                }
 
-                               break
-
                        case Zbyte:
                                v = vaddr(ctxt, p, &p.From, &rel)
                                if rel.Siz != 0 {
@@ -4776,7 +4772,6 @@ bad:
 
        ctxt.Diag("invalid instruction: %v", p)
        //      ctxt.Diag("doasm: notfound ft=%d tt=%d %v %d %d", p.Ft, p.Tt, p, oclass(ctxt, p, &p.From), oclass(ctxt, p, &p.To))
-       return
 }
 
 // byteswapreg returns a byte-addressable register (AX, BX, CX, DX)
index d0522c8f4314ac3ab8f4bf4841e72fa9ca88bb68..ecb9e39a6b809d963221b9a9e17c946837125b8e 100644 (file)
@@ -106,21 +106,6 @@ func (c *count) IsCountFlag() bool {
        return true
 }
 
-type fn0 func()
-
-func (f fn0) Set(s string) error {
-       f()
-       return nil
-}
-
-func (f fn0) Get() interface{} { return nil }
-
-func (f fn0) String() string { return "" }
-
-func (f fn0) IsBoolFlag() bool {
-       return true
-}
-
 type fn1 func(string)
 
 func (f fn1) Set(s string) error {
index 6ddf8d6cd7df9912097dffa4fd30558a29e766fe..ef8ef4324b8c3e97eca0e937331ca6591d1ae5b1 100644 (file)
@@ -141,9 +141,7 @@ func (fc *FileCache) Line(filename string, line int) ([]byte, error) {
        // Clean filenames returned by src.Pos.SymFilename()
        // or src.PosBase.SymFilename() removing
        // the leading src.FileSymPrefix.
-       if strings.HasPrefix(filename, src.FileSymPrefix) {
-               filename = filename[len(src.FileSymPrefix):]
-       }
+       filename = strings.TrimPrefix(filename, src.FileSymPrefix)
 
        // Expand literal "$GOROOT" rewrited by obj.AbsFile()
        filename = filepath.Clean(os.ExpandEnv(filename))
index 09c9e31c2e3518ff7caa46a3e7af0faf3345d014..c8fae3789839e59567772d74c7bbbed6f07c5273 100644 (file)
@@ -406,9 +406,7 @@ func readpesym(arch *sys.Arch, syms *sym.Symbols, f *pe.File, pesym *pe.COFFSymb
                name = sectsyms[f.Sections[pesym.SectionNumber-1]].Name
        } else {
                name = symname
-               if strings.HasPrefix(name, "__imp_") {
-                       name = name[6:] // __imp_Name => Name
-               }
+               name = strings.TrimPrefix(name, "__imp_") // __imp_Name => Name
                if arch.Family == sys.I386 && name[0] == '_' {
                        name = name[1:] // _Name => Name
                }
index b13ce2fcb56cc0d5ea8a552c8dfe34e184eae2b8..5783278d2c29b635e40bc3740980abe9b6a4a12f 100644 (file)
@@ -166,9 +166,7 @@ func (f *File) matchParams(expect []string, actual []ast.Expr, prefix string) bo
 
 // Does this one type match?
 func (f *File) matchParamType(expect string, actual ast.Expr) bool {
-       if strings.HasPrefix(expect, "=") {
-               expect = expect[1:]
-       }
+       expect = strings.TrimPrefix(expect, "=")
        // Strip package name if we're in that package.
        if n := len(f.file.Name.Name); len(expect) > n && expect[:n] == f.file.Name.Name && expect[n] == '.' {
                expect = expect[n+1:]