cleaned up SetLoggingLevel

levellogger.go

@@ -7,8 +7,10 @@ import (
 	"runtime/debug"
 )
 
-func NewLevelLogger(out io.Writer, level LogLevel) LevelLogger {
+// changed to return a pointer to a new LevelLogger
-	var ll LevelLogger
+func NewLevelLogger(out io.Writer, level LogLevel) *LevelLogger {
+
+	ll := &LevelLogger{}
 
 	ll.CurrentLevel = level
 
@@ -188,9 +190,6 @@ 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
@@ -213,10 +212,9 @@ func (ll *LevelLogger) SetLoggingLevel(l int) {
 	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)
 	}