cleaned up SetLoggingLevel

levellogger.go

@@ -7,8 +7,10 @@ import (
 	"runtime/debug"
 )
 
-func NewLevelLogger(out io.Writer, level LogLevel) LevelLogger {
-	var ll LevelLogger
+// changed to return a pointer to a new LevelLogger
+func NewLevelLogger(out io.Writer, level LogLevel) *LevelLogger {
+
+	ll := &LevelLogger{}
 
 	ll.CurrentLevel = level
 
@@ -188,35 +190,31 @@ func (ll *LevelLogger) SetFlags(logname string, newFlags int) {
 }
 
 func (ll *LevelLogger) SetLoggingLevel(l int) {
-	if l < 0 || l > 8 {
-		ll.CurrentLevel = 4 // default
-	} else {
-		switch l {
-		case 0:
-			ll.CurrentLevel = LevelLoggerOff
-		case 1:
-			ll.CurrentLevel = PanicLevel
-		case 2:
-			ll.CurrentLevel = TraceLevel
-		case 3:
-			ll.CurrentLevel = DebugLevel
-		case 4:
-			ll.CurrentLevel = InfoLevel
-		case 5:
-			ll.CurrentLevel = SecurityLevel
-		case 6:
-			ll.CurrentLevel = WarnLevel
-		case 7:
-			ll.CurrentLevel = ErrorLevel
-		case 8:
-			ll.CurrentLevel = FatalLevel
-		default:
-			ll.CurrentLevel = InfoLevel
-		}
+	switch l {
+	case 0:
+		ll.CurrentLevel = LevelLoggerOff
+	case 1:
+		ll.CurrentLevel = PanicLevel
+	case 2:
+		ll.CurrentLevel = TraceLevel
+	case 3:
+		ll.CurrentLevel = DebugLevel
+	case 4:
+		ll.CurrentLevel = InfoLevel
+	case 5:
+		ll.CurrentLevel = SecurityLevel
+	case 6:
+		ll.CurrentLevel = WarnLevel
+	case 7:
+		ll.CurrentLevel = ErrorLevel
+	case 8:
+		ll.CurrentLevel = FatalLevel
+	default:
+		ll.CurrentLevel = InfoLevel
 	}
 }
 
-func (ll LevelLogger) Blank(n int) {
+func (ll *LevelLogger) Blank(n int) {
 	outs := " "
 	if n < 1 {
 		n = 1
@@ -229,7 +227,7 @@ func (ll LevelLogger) Blank(n int) {
 	ll.blankLog.Print(outs)
 }
 
-func (ll LevelLogger) CheckFerr(e error, msg string) {
+func (ll *LevelLogger) CheckFerr(e error, msg string) {
 	if e != nil {
 		ll.Fatal(e, msg)
 	}