nitpicks

I think the point of kr_log_is_debug() before kr_log_req() was optimization, in case the call happens on hotter code-paths. Otherwise the function does relatively lots of work before deciding that nothing should be logged. (new mempool, construct formatted message, destroy mempool)

added 1 commit

• b03be744 - lib/resolve: clarify why debug level is checked

Compare with previous version

resolved all threads

added 1 commit

• e0017573 - doc: remove outdated note about survery

Compare with previous version

added 1 commit

• 09c7af68 - ci build:macOS: more delay

Compare with previous version

added 1 commit

• 648e004b - doc logging: add a simple example

Compare with previous version

resolved all threads

added 24 commits

• 648e004b...e4633d64 - 19 commits from branch master
• 0c2e0521 - modules/nsid: improve sanity check
• bb0a7711 - lib/resolve: clarify why debug level is checked
• d141d4e2 - doc: remove outdated note about survery
• aa4df308 - ci build:macOS: more delay
• 4e01f882 - doc logging: add a simple example

Compare with previous version

added 1 commit

• bc6feaa8 - ci: allow extra sync time for macos GH action

Compare with previous version

added 1 commit

• 498ca708 - ci/distrotests: add fedora35, ubuntu2110

Compare with previous version

added 9 commits

• 498ca708...8c0564ba - 2 commits from branch master
• 3d60a990 - modules/nsid: improve sanity check
• 8cc68478 - lib/resolve: clarify why debug level is checked
• 8ee8747c - doc: remove outdated note about survery
• 68510ac9 - ci build:macOS: more delay
• 0fb48e75 - doc logging: add a simple example
• c4b5aa19 - ci: allow extra sync time for macos GH action
• 6e3e6dd3 - ci/distrotests: add fedora35, ubuntu2110

Compare with previous version

merged

mentioned in commit c17e9541