Remove FatalLevel

Doing `os.Exit(-1)` inside a library doesn't sit right.

Author: bep

handlers/cli/cli.go

@@ -23,7 +23,6 @@ var Colors = [...]*color.Color{
 	log.InfoLevel:  color.New(color.FgBlue),
 	log.WarnLevel:  color.New(color.FgYellow),
 	log.ErrorLevel: color.New(color.FgRed),
-	log.FatalLevel: color.New(color.FgRed),
 }
 
 // Strings mapping.
@@ -32,7 +31,6 @@ var Strings = [...]string{
 	log.InfoLevel:  "•",
 	log.WarnLevel:  "•",
 	log.ErrorLevel: "⨯",
-	log.FatalLevel: "⨯",
 }
 
 // Handler implementation.

handlers/text/text.go

@@ -33,7 +33,6 @@ var Colors = [...]int{
 	log.InfoLevel:  blue,
 	log.WarnLevel:  yellow,
 	log.ErrorLevel: red,
-	log.FatalLevel: red,
 }
 
 // Strings mapping.
@@ -42,7 +41,6 @@ var Strings = [...]string{
 	log.InfoLevel:  "INFO",
 	log.WarnLevel:  "WARN",
 	log.ErrorLevel: "ERROR",
-	log.FatalLevel: "FATAL",
 }
 
 // Handler implementation.

levels.go

@@ -19,15 +19,13 @@ const (
 	InfoLevel
 	WarnLevel
 	ErrorLevel
-	FatalLevel
 )
 
 var levelNames = [...]string{
 	DebugLevel: "debug",
 	InfoLevel:  "info",
 	WarnLevel:  "warn",
 	ErrorLevel: "error",
-	FatalLevel: "fatal",
 }
 
 var levelStrings = map[string]Level{
@@ -36,7 +34,6 @@ var levelStrings = map[string]Level{
 	"warn":    WarnLevel,
 	"warning": WarnLevel,
 	"error":   ErrorLevel,
-	"fatal":   FatalLevel,
 }
 
 // String implementation.

levels_test.go

@@ -18,7 +18,6 @@ func TestParseLevel(t *testing.T) {
 		{"warn", WarnLevel, 2},
 		{"warning", WarnLevel, 3},
 		{"error", ErrorLevel, 4},
-		{"fatal", FatalLevel, 5},
 	}
 
 	for _, c := range cases {

logger.go

@@ -3,7 +3,6 @@ package log
 import (
 	"fmt"
 	stdlog "log"
-	"os"
 	"time"
 
 	"github.com/bep/clocks"
@@ -93,7 +92,7 @@ func NewLogger(cfg LoggerConfig) Logger {
 		panic("handler cannot be nil")
 	}
 
-	if cfg.Level <= 0 || cfg.Level > FatalLevel {
+	if cfg.Level <= 0 || cfg.Level > ErrorLevel {
 		panic("log level is out of range")
 	}
 
@@ -167,7 +166,4 @@ func (l *logger) log(e *Entry, s fmt.Stringer) {
 		stdlog.Printf("error logging: %s", err)
 	}
 
-	if e.Level == FatalLevel {
-		os.Exit(1)
-	}
 }