Refactor: change Hook(h Hook) to Hook(hooks ...Hook) #629

Halimao · 2023-12-24T12:59:16Z

This PR add support for setting multiple hooks at one time, without multiple times of deep copy (compare to call Hook() multi-times)

Also, func (l Logger) Hook(hooks ...Hook) Logger {} is compatible with func (l Logger) Hook(h Hook) Logger {}

…ulti hooks at once

Halimao added 2 commits December 24, 2023 20:52

refactor: change Hook(h Hook) to Hook(hooks ...Hook) to support set m…

8a04417

…ulti hooks at once

return directly when hooks length is 0

8854c80

rs merged commit 3e8ae07 into rs:master Dec 24, 2023

madkins23 pushed a commit to madkins23/zerolog that referenced this pull request Mar 2, 2024

Refactor: change Hook(h Hook) to Hook(hooks ...Hook) (rs#629)

f882be4

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Refactor: change Hook(h Hook) to Hook(hooks ...Hook) #629

Refactor: change Hook(h Hook) to Hook(hooks ...Hook) #629

Halimao commented Dec 24, 2023 •

edited

Loading

Refactor: change Hook(h Hook) to Hook(hooks ...Hook) #629

Refactor: change Hook(h Hook) to Hook(hooks ...Hook) #629

Conversation

Halimao commented Dec 24, 2023 • edited Loading

Halimao commented Dec 24, 2023 •

edited

Loading