Skip to content

Commit 7d77b63

Browse files
authored
fix: use file.opt instead of opt when setting logger creating a file instance. (#97)
* fix: use file.opt instead of opt when setting logger. If pe.New function is called with nil as opts parameter, panic occurs due to comparing opts.logger(which opt is nil), so use file.opts instead. * fix: use file.opts.logger for NewBytes function
1 parent f9f7d40 commit 7d77b63

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

file.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -144,12 +144,12 @@ func New(name string, opts *Options) (*File, error) {
144144
}
145145

146146
var logger log.Logger
147-
if opts.Logger == nil {
147+
if file.opts.Logger == nil {
148148
logger = log.NewStdLogger(os.Stdout)
149149
file.logger = log.NewHelper(log.NewFilter(logger,
150150
log.FilterLevel(log.LevelError)))
151151
} else {
152-
file.logger = log.NewHelper(opts.Logger)
152+
file.logger = log.NewHelper(file.opts.Logger)
153153
}
154154

155155
file.data = data
@@ -176,7 +176,7 @@ func NewBytes(data []byte, opts *Options) (*File, error) {
176176
}
177177

178178
var logger log.Logger
179-
if opts.Logger == nil {
179+
if file.opts.Logger == nil {
180180
logger = log.NewStdLogger(os.Stdout)
181181
file.logger = log.NewHelper(log.NewFilter(logger,
182182
log.FilterLevel(log.LevelError)))

0 commit comments

Comments
 (0)