Cruft: There are 3 routines that convert time_t to text

Gary E. Miller gem at rellim.com
Thu Mar 16 06:35:53 UTC 2017


Yo Hal!

On Wed, 15 Mar 2017 23:21:59 -0700
Hal Murray <hmurray at megapathdsl.net> wrote:

> gem at rellim.com said:
> > I like checks for things that can never happen.  Somehow many of
> > them eventually do get tripped, then the error is more obvious,
> > assuming a good error message.   
> 
> I do too, but there can be too much of a good thing.  There is a real
> talent for putting in the right checks without adding too much
> clutter.

It is always a judgement call.

> That particular trap is very unlikely to go off.  I'd be happy if the
> calls to that routine were changed to use the new code and a similar
> trap were installed over there.


I'm all for calling the functions in timespecops.h.  Way too many places
where handling timespec's are one-off, each done slightly wrong.

But not this cycle, quiete period before release.  We need to keep
changes to the very simple and the very critical.

RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
	gem at rellim.com  Tel:+1 541 382 8588

	    Veritas liberabit vos. -- Quid est veritas?
    "If you can’t measure it, you can’t improve it." - Lord Kelvin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ntpsec.org/pipermail/devel/attachments/20170315/79885d2c/attachment.bin>


More information about the devel mailing list