]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile/internal/gc: clean up errcmp
authorHÃ¥vard Haugen <havard.haugen@gmail.com>
Wed, 2 Sep 2015 18:25:46 +0000 (20:25 +0200)
committerBrad Fitzpatrick <bradfitz@golang.org>
Sun, 13 Sep 2015 23:50:25 +0000 (23:50 +0000)
Change-Id: Id07811a25bf4aa3ff834e7254a3dfb04522b2926
Reviewed-on: https://go-review.googlesource.com/14174
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dave Cheney <dave@cheney.net>
src/cmd/compile/internal/gc/subr.go

index f82fed4586b04f0fd07393f4b0fa55cfda354df1..68cee5231d16277e36ba2b11a39c4d9c31392b56 100644 (file)
@@ -59,26 +59,21 @@ func adderr(line int, format string, args ...interface{}) {
        })
 }
 
+// errcmp sorts errors by line, then seq, then message.
 type errcmp []Error
 
-func (x errcmp) Len() int {
-       return len(x)
-}
-
-func (x errcmp) Swap(i, j int) {
-       x[i], x[j] = x[j], x[i]
-}
-
+func (x errcmp) Len() int      { return len(x) }
+func (x errcmp) Swap(i, j int) { x[i], x[j] = x[j], x[i] }
 func (x errcmp) Less(i, j int) bool {
        a := &x[i]
        b := &x[j]
        if a.lineno != b.lineno {
-               return a.lineno-b.lineno < 0
+               return a.lineno < b.lineno
        }
        if a.seq != b.seq {
-               return a.seq-b.seq < 0
+               return a.seq < b.seq
        }
-       return stringsCompare(a.msg, b.msg) < 0
+       return a.msg < b.msg
 }
 
 func Flusherrors() {
@@ -86,7 +81,7 @@ func Flusherrors() {
        if len(errors) == 0 {
                return
        }
-       sort.Sort(errcmp(errors[:len(errors)]))
+       sort.Sort(errcmp(errors))
        for i := 0; i < len(errors); i++ {
                if i == 0 || errors[i].msg != errors[i-1].msg {
                        fmt.Printf("%s", errors[i].msg)