Odds and ends

[Eric S. Raymond]

Hal Murray via devel <devel at ntpsec.org>: 
> I occasionally add msyslog lines when debugging.  The DPRINTF stuff isn't 
> interesting - too much junk I don't want.  When I'm cleaning up, should I 
> disable them with "if (0)", or delete them?  Is there a better way? ...

What I do is delete them unless I think they might have continuing value,
in which case I put them under DPRINTF.

> Some/many of the prototypes in our header files are not very useful.  I think the problem is the lack of names on the parameters.  If the types are all different, that's enough, but if there are several integer parameters that doesn't help.  Here is an example:
> extern  int     clocktime       (int, int, int, int, int, time_t, uint32_t, uint32_t *, uint32_t *);

Fair point.  Are you advocating outting the formal names back in?

> There is still some broadcast/multicast stuff around.  grep for CAST will find them.  Should we leave enough around so that our ntpq can decode things when talking to old servers?  If so, I vote we move them to ntpclients/

I agree with moving them to ntpclients.  Anything  that reeduces the amount of
ceuft ubthe ntpd code is good.

> Some of the counters that ntpq can display get written to a log file every hour.  That resets the counters so ntpq can only see what has happened since then rather than the totals since startup.  We should probably maintain and display both.

Reasonable. Would you make ntpq parse the logfile?

> It would be nice if ntpq would show something if a server is using authentication.  I'd be willing to steal a character from the name column.

No need. Just boldface the ones with authentication.

> The code for freeing up key strings zeros them out first.  How do we do that in python or go?

Same way you would in C, iterate over them writing zeros.  Am I
missing something?
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>

My work is funded by the Internet Civil Engineering Institute: https://icei.org
Please visit their site and donate: the civilization you save might be your own.