Removed global formatter for error messages as it is potentially dang…

…erous. One can unexpectedly change error messages from other packages if they also use simplerr. If error checks are using the error message then it could break things

errors.go

@@ -44,7 +44,10 @@ func New(_fmt string, args ...interface{}) *SimpleError {
 
 // Error satisfies the `error` interface. It uses the `simplerr.Formatter` to generate an error string.
 func (e *SimpleError) Error() string {
-	return Formatter(e)
+	if parent := e.Unwrap(); parent != nil {
+		return fmt.Sprintf("%s: %s", e.GetMessage(), parent.Error())
+	}
+	return e.msg
 }
 
 // Message sets the message text on the error. This message it used to wrap the underlying error, if it exists.

errors_test.go

@@ -517,17 +517,6 @@ func (s *TestSuite) TestErrorFormatting() {
 	serr3 := New("something")
 	s.Equal("something", serr3.Error())
 
-	// Change the error formatting style
-	Formatter = func(e *SimpleError) string {
-		parent := e.Unwrap()
-		if parent == nil {
-			return e.GetMessage()
-		}
-		return strings.Join([]string{e.GetMessage(), parent.Error()}, "\n")
-	}
-	s.Equal("wrapper 1\noriginal", serr1.Error())
-	s.Equal("wrapper 2\nwrapper 1\noriginal", serr2.Error())
-	Formatter = DefaultFormatter
 }
 
 func (s *TestSuite) TestStackTrace() {