Bugzilla – Bug 257
log time printer, can we optionally support output of time in seconds?
Last modified: 2008-11-25 09:38:33 EST
Currently log time is printed in nanoseconds, which is very hard to read. Would be much better if we could have an option to print time in seconds.
Created attachment 274 [details] proposed patch is this something you would be happy with ? I would even be fine with changing the default from PREFIX_TIME to PREFIX_TIME_S
+1
The patch looks fine to me. One vote for changing the default from PREFIX_TIME to PREFIX_TIME_S.
+1, I guess that this patch is needed to enable the output Gustavo posted for bug 405. +1 on changing the default to be seconds-based.
(In reply to comment #4) > +1, I guess that this patch is needed to enable the output Gustavo posted for > bug 405. +1 on changing the default to be seconds-based. Since it looks like everyone would be happy with changing the default, how about just doing this and getting rid of the ns output ?
The output in seconds from bug 405 is actually not a result of this patch but a quick and dirty patch I have done in my tree. But the result should be more or less the same... +1 to change the default to seconds. But then you should change the patch to add PREFIX_TIME_NS instead of PREFIX_TIME_S. I guessing there was a reason to have time in nanoseconds in the first place. I am guessing the reason still remains, although we have come to the conclusion that in most cases time in seconds is what we want and time in nanoseconds is the exception and not the rule.
(In reply to comment #6) > The output in seconds from bug 405 is actually not a result of this patch but a > quick and dirty patch I have done in my tree. But the result should be more or > less the same... > > +1 to change the default to seconds. But then you should change the patch to > add PREFIX_TIME_NS instead of PREFIX_TIME_S. I guessing there was a reason to > have time in nanoseconds in the first place. I am guessing the reason still > remains, although we have come to the conclusion that in most cases time in > seconds is what we want and time in nanoseconds is the exception and not the > rule. > I would be happy with either removing the ns variant or just making it non-default.
changed the default to output seconds. changeset: 736b807ae8c0