Minor code cleanup; updated docs and ci settings

Author: alex-alrux

.travis.yml

@@ -1,18 +1,19 @@
 language: go
-
+sudo: false
 go:
+  - 1.8
   - 1.7.5
   - 1.7.4
   - 1.7.3
   - 1.7.2
   - 1.7.1
   - 1.7
+  - tip
   - 1.6.4
   - 1.6.3
   - 1.6.2
   - 1.6.1
   - 1.6
-  - tip
   - 1.5.4
   - 1.5.3
   - 1.5.2
@@ -32,15 +33,19 @@ go:
   - 1.1.2
   - 1.1.1
   - 1.1
-  - 1.0.3
-  - 1.0.2
-  - 1.0.1
-  - 1
-
+before_install:
+  - go get github.com/mattn/goveralls
+script:
+  - $HOME/gopath/bin/goveralls -service=travis-ci
 matrix:
   fast_finish: true
   allow_failures:
     - go: tip
+    - go: 1.6.4
+    - go: 1.6.3
+    - go: 1.6.2
+    - go: 1.6.1
+    - go: 1.6
     - go: 1.5.4
     - go: 1.5.3
     - go: 1.5.2
@@ -60,7 +65,3 @@ matrix:
     - go: 1.1.2
     - go: 1.1.1
     - go: 1.1
-    - go: 1.0.3
-    - go: 1.0.2
-    - go: 1.0.1
-    - go: 1

README.md

@@ -1,19 +1,21 @@
 # Extended `errors` package for Go (golang)
 
-This is a drop-in replacement for the standard [Go](http://golang.org) package with the same name, providing all the standard functionality as well as additional features.
+[![Release](https://img.shields.io/github/release/agext/errors.svg?style=flat)](https://github.com/agext/errors/releases/latest)
+[![GoDoc](https://img.shields.io/badge/godoc-reference-blue.svg?style=flat)](https://godoc.org/github.com/agext/errors) 
+[![Build Status](https://travis-ci.org/agext/errors.svg?branch=master&style=flat)](https://travis-ci.org/agext/errors)
+[![Coverage Status](https://coveralls.io/repos/github/agext/errors/badge.svg?style=flat)](https://coveralls.io/github/agext/errors)
+[![Go Report Card](https://goreportcard.com/badge/github.com/agext/errors?style=flat)](https://goreportcard.com/report/github.com/agext/errors)
 
-## Maturity
+This is a drop-in replacement for the standard [Go](http://golang.org) package with the same name, providing all the standard functionality as well as additional features.
 
-[![Build Status](https://travis-ci.org/agext/errors.svg?branch=master)](https://travis-ci.org/agext/errors)
+## Project Status
 
-v1.0 Stable: Guaranteed no breaking changes to the API in future v1.x releases. No known bugs or performance issues. Probably safe to use in production, though provided on "AS IS" basis.
+v1.0.1 Stable: Guaranteed no breaking changes to the API in future v1.x releases. Probably safe to use in production, though provided on "AS IS" basis.
 
 This package is being actively maintained. If you encounter any problems or have any suggestions for improvement, please [open an issue](https://github.com/agext/errors/issues). Pull requests are welcome.
 
 ## Overview
 
-[![GoDoc](https://godoc.org/github.com/agext/errors?status.png)](https://godoc.org/github.com/agext/errors)
-
 When you need to retain more information about an error message than a single string allows, just substitute this package for the one in the standard library.
 
 The `New` function still accepts a single string as argument, so no code will be broken. Where you need to include additional information, you can provide it to `New` in a `Desc` structure instead of the string, or you can add it to the error message using one of its setter methods.

errors.go

@@ -101,60 +101,60 @@ func newFromE(desc *Desc) Error {
 // Log sends the error to the provided log, using the appropriate
 // logging function: FATAL conditions are logged using Fatal(), PANIC using
 // Panic(), and anything else using Print().
-func (this *errorMessage) Log(log Logger) Error {
-	switch this.level {
+func (em *errorMessage) Log(log Logger) Error {
+	switch em.level {
 	case FATAL:
-		log.Fatal(this)
+		log.Fatal(em)
 	case PANIC:
-		log.Panic(this)
+		log.Panic(em)
 	default:
-		log.Print(this)
+		log.Print(em)
 	}
-	return this
+	return em
 }
 
 // Level returns the error level.
-func (this *errorMessage) Level() int8 {
-	return this.level
+func (em *errorMessage) Level() int8 {
+	return em.level
 }
 
 // SetLevel sets the error level.
-func (this *errorMessage) SetLevel(l int8) Error {
+func (em *errorMessage) SetLevel(l int8) Error {
 	if l >= minLevel && l <= maxLevel {
-		this.level = l
+		em.level = l
 	}
-	return this
+	return em
 }
 
 // Code returns the error code.
-func (this *errorMessage) Code() int {
-	return this.code
+func (em *errorMessage) Code() int {
+	return em.code
 }
 
 // SetCode sets the error code.
-func (this *errorMessage) SetCode(c int) Error {
-	this.code = c
-	return this
+func (em *errorMessage) SetCode(c int) Error {
+	em.code = c
+	return em
 }
 
 // Text returns the error text.
-func (this *errorMessage) Text() string {
-	return this.text
+func (em *errorMessage) Text() string {
+	return em.text
 }
 
 // SetText sets the error text.
-func (this *errorMessage) SetText(t string) Error {
-	this.text = t
-	return this
+func (em *errorMessage) SetText(t string) Error {
+	em.text = t
+	return em
 }
 
 // Info returns the error info.
-func (this *errorMessage) Info() []string {
-	return this.info
+func (em *errorMessage) Info() []string {
+	return em.info
 }
 
 // addInfo adds (more) error info.
-func (this *errorMessage) addInfo(calldepth int, s ...string) Error {
+func (em *errorMessage) addInfo(calldepth int, s ...string) Error {
 	for i, line := range s {
 		if line == "debug.stack" {
 			calldepth *= 2
@@ -180,20 +180,20 @@ func (this *errorMessage) addInfo(calldepth int, s ...string) Error {
 			break
 		}
 	}
-	this.info = append(this.info, s...)
-	return this
+	em.info = append(em.info, s...)
+	return em
 }
 
 // AddInfo adds (more) error info.
-func (this *errorMessage) AddInfo(s ...string) Error {
-	return this.addInfo(2, s...)
+func (em *errorMessage) AddInfo(s ...string) Error {
+	return em.addInfo(2, s...)
 }
 
 // Error returns a text containing the error message and code;
 // it is useful for satisfying the `error` interface.
-func (this *errorMessage) Error() string {
-	if this.code != 0 {
-		return this.text + fmt.Sprintf(" (code: 0x%04x)", this.code)
+func (em *errorMessage) Error() string {
+	if em.code != 0 {
+		return em.text + fmt.Sprintf(" (code: 0x%04x)", em.code)
 	}
-	return this.text
+	return em.text
 }

errors_test.go

@@ -24,14 +24,14 @@ type mockLogger struct {
 	log string
 }
 
-func (this *mockLogger) Print(s ...interface{}) {
-	this.log += fmt.Sprintln(s...)
+func (ml *mockLogger) Print(s ...interface{}) {
+	ml.log += fmt.Sprintln(s...)
 }
-func (this *mockLogger) Fatal(s ...interface{}) {
-	this.log += "[FATAL] " + fmt.Sprintln(s...)
+func (ml *mockLogger) Fatal(s ...interface{}) {
+	ml.log += "[FATAL] " + fmt.Sprintln(s...)
 }
-func (this *mockLogger) Panic(s ...interface{}) {
-	this.log += "[PANIC] " + fmt.Sprintln(s...)
+func (ml *mockLogger) Panic(s ...interface{}) {
+	ml.log += "[PANIC] " + fmt.Sprintln(s...)
 }
 
 func TestNew(t *testing.T) {

helpers.go

@@ -44,6 +44,7 @@ func levelName(l int8) string {
 	return "?"
 }
 
+// Logger defines the interface expected by the Log method of Error
 type Logger interface {
 	Fatal(...interface{})
 	Panic(...interface{})