wneessen/go-mail
1
1
Fork 0
mirror of https://github.com/wneessen/go-mail.git synced 2024-11-22 13:50:49 +01:00
Code Issues Projects Releases Packages Wiki Activity

Replace hardcoded '2' in Output with const CallDepth

Browse source 
A `const CallDepth` placeholder has been introduced and has been set to 2. This is used as an argument in Output method calls in various log levels like debug, info, warn, and error in place of previously hardcoded '2'. This change improves the readability of the code. The '2' represented the call depth of log.Logger's Output method, which wasn't immediately clear with the hardcoded value.
This commit is contained in:
Winni Neessen 2023-07-24 15:27:39 +02:00
parent 2b27643187
commit 7d69ec8faa
Signed by: wneessen
GPG key ID: 5F3AF39B820C119D
1 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
log/stdlog.go
View file

@ -33,6 +33,10 @@ const (
LevelDebug LevelDebug
) )
// CallDepth is the call depth value for the log.Logger's Output method
// This defaults to 2 and is only here for better readablity of the code
const CallDepth = 2
// New returns a new Stdlog type that satisfies the Logger interface // New returns a new Stdlog type that satisfies the Logger interface
func New(o io.Writer, l Level) *Stdlog { func New(o io.Writer, l Level) *Stdlog {
lf := log.Lmsgprefix | log.LstdFlags lf := log.Lmsgprefix | log.LstdFlags
@ -48,27 +52,27 @@ func New(o io.Writer, l Level) *Stdlog {
// Debugf performs a Printf() on the debug logger // Debugf performs a Printf() on the debug logger
func (l *Stdlog) Debugf(f string, v ...interface{}) { func (l *Stdlog) Debugf(f string, v ...interface{}) {
if l.l >= LevelDebug { if l.l >= LevelDebug {
_ = l.debug.Output(2, fmt.Sprintf(f, v...)) _ = l.debug.Output(CallDepth, fmt.Sprintf(f, v...))
} }
} }
// Infof performs a Printf() on the info logger // Infof performs a Printf() on the info logger
func (l *Stdlog) Infof(f string, v ...interface{}) { func (l *Stdlog) Infof(f string, v ...interface{}) {
if l.l >= LevelInfo { if l.l >= LevelInfo {
_ = l.info.Output(2, fmt.Sprintf(f, v...)) _ = l.info.Output(CallDepth, fmt.Sprintf(f, v...))
} }
} }
// Warnf performs a Printf() on the warn logger // Warnf performs a Printf() on the warn logger
func (l *Stdlog) Warnf(f string, v ...interface{}) { func (l *Stdlog) Warnf(f string, v ...interface{}) {
if l.l >= LevelWarn { if l.l >= LevelWarn {
_ = l.warn.Output(2, fmt.Sprintf(f, v...)) _ = l.warn.Output(CallDepth, fmt.Sprintf(f, v...))
} }
} }
// Errorf performs a Printf() on the error logger // Errorf performs a Printf() on the error logger
func (l *Stdlog) Errorf(f string, v ...interface{}) { func (l *Stdlog) Errorf(f string, v ...interface{}) {
if l.l >= LevelError { if l.l >= LevelError {
_ = l.err.Output(2, fmt.Sprintf(f, v...)) _ = l.err.Output(CallDepth, fmt.Sprintf(f, v...))
} }
} }