(WIP) Add -logtimemillis command line option #9062

I find micros are too granular, with no line ever being less than 1 millisecond apart from its neighbouring lines, therefore this option will save 3 bytes per line of debug.log :)

Seems overkill to have three time formats

Agree with @laanwj

@laanwj you have a point - shall I add to this the removal of logtimemicros? I just didn't want to inconvenience people who already had it in their bitcoin.conf, though. Or make it so that logtimemicros shows only the first three decimal places?

IMO we should just keep things as is. Either use microsecond precision, or second precision, or keep this as a custom patch. You could even postprocess the log to chop off three positions. For the rest of the world this just isn't so important.

I'd fathom the rest of the world, too, has more urgent things to discuss too than even more flavors of logging format. This shouldn't compete for attention with bugfixes and new features.

@laanwj You are probably right, however, many other projects default to logging only milliseconds. I do still question the logic in logging 3 additional insignificant decimal places, given that lines (in my experience) are never less than 1ms apart.

I like the idea of postprocessing, although ideally rather than post-processing this could be done by piping so that the unprocessed debug log isn't created. Is there currently an option to debug to stdout or stderr?, as this would then be possible.

src/init.cpp:    strUsage += HelpMessageOpt("-printtoconsole", _("Send trace/debug info to console instead of debug.log file"));