cleaned up SetLoggingLevel

levellogger.go

@@ -7,6 +7,7 @@ import (
 	"runtime/debug"
 )
 
+// changed to return a pointer to a new LevelLogger
 func NewLevelLogger(out io.Writer, level LogLevel) *LevelLogger {
 
 	ll := &LevelLogger{}
@@ -189,31 +190,27 @@ func (ll *LevelLogger) SetFlags(logname string, newFlags int) {
 }
 
 func (ll *LevelLogger) SetLoggingLevel(l int) {
-	if l < 0 || l > 8 {
+	switch l {
-		ll.CurrentLevel = 4 // default
+	case 0:
-	} else {
+		ll.CurrentLevel = LevelLoggerOff
-		switch l {
+	case 1:
-		case 0:
+		ll.CurrentLevel = PanicLevel
-			ll.CurrentLevel = LevelLoggerOff
+	case 2:
-		case 1:
+		ll.CurrentLevel = TraceLevel
-			ll.CurrentLevel = PanicLevel
+	case 3:
-		case 2:
+		ll.CurrentLevel = DebugLevel
-			ll.CurrentLevel = TraceLevel
+	case 4:
-		case 3:
+		ll.CurrentLevel = InfoLevel
-			ll.CurrentLevel = DebugLevel
+	case 5:
-		case 4:
+		ll.CurrentLevel = SecurityLevel
-			ll.CurrentLevel = InfoLevel
+	case 6:
-		case 5:
+		ll.CurrentLevel = WarnLevel
-			ll.CurrentLevel = SecurityLevel
+	case 7:
-		case 6:
+		ll.CurrentLevel = ErrorLevel
-			ll.CurrentLevel = WarnLevel
+	case 8:
-		case 7:
+		ll.CurrentLevel = FatalLevel
-			ll.CurrentLevel = ErrorLevel
+	default:
-		case 8:
+		ll.CurrentLevel = InfoLevel
-			ll.CurrentLevel = FatalLevel
-		default:
-			ll.CurrentLevel = InfoLevel
-		}
 	}
 }