Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance logging - add millisecs (add %f specifier for timestamp-format) #1087

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

michaelortmann
Copy link
Member

@michaelortmann michaelortmann commented Jan 3, 2021

Found by: michaelortmann
Patch by: michaelortmann
Fixes:

One-line summary:
Add %f specifier for timestamp-format

Additional description (if needed):
I always wanted to have the possibility to let eggdrop log showing milliseconds. Often i found myself adding this while debugging. Last time i missed it was with ident race debugging. This PR adds %f as an additional format specifier for timestamp-format.
@geo: you probaly wanna add some doc for it.

Test cases demonstrating functionality (if applicable):
conf:
set timestamp-format {[%H:%M:%S.%f]}

$ ./eggdrop -t BotA.conf
[...]
*** -HQ joined the party line.
[22:19:02.736] main: entering loop
[...]
[22:19:39.804] #test6888: mode change '+t ' by [email protected]

Test %f vs. %%f:

conf:
set timestamp-format {[%H:%M:%S.%f]}:
result:
[12:57:54.481]

conf:
set timestamp-format {[%%H:%%M:%%S.%%f]}
result:
[%H:%M:%S.%f]

@michaelortmann michaelortmann changed the title Add %f specifier for timestamp-format Enhance logging - add millisecs (add %f specifier for timestamp-format) Jan 22, 2021
src/misc.c Outdated Show resolved Hide resolved
michaelortmann pushed a commit to michaelortmann/eggdrop that referenced this pull request Nov 21, 2023
michaelortmann pushed a commit to michaelortmann/eggdrop that referenced this pull request Nov 21, 2023
@michaelortmann michaelortmann mentioned this pull request Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants