zero value for zerolog.Logger

[rs]

zero value for zerolog.Logger

[bazuker]

@soulne4ny It does not seem like you initialized the logger.
Try something like this

package main

import (
	"github.com/rs/zerolog"
)

func main() {
	log := zerolog.New(os.Stdout).With().Timestamp().Logger()
	log.Info().Msg("crash")
}

[toravir]

@oxygenD Do you think it is ok to change methods like Info(), Debug() etc to take pointer receivers ? So that one can check for nil pointers and not crash. That will also avoid compiler making a copy of the logger for the invocation.

[bazuker]

@toravir It's zero log, I guess if pointers were used instead, they would cause allocations which we are trying to avoid.

[toravir]

@oxygenD i tried that and ran benchmark and i don't see an extra alloc - cpu usage isn't much different either. See:
With "(l * Logger) Info() *Event" :

BenchmarkInfo-8   	30000000	        39.2 ns/op	       0 B/op	       0 allocs/op

Without the change (base code):

BenchmarkInfo-8   	30000000	        41.8 ns/op	       0 B/op	       0 allocs/op

[rs]

Fixed by #18.