Six not properly documented patches for ntpviz

Richard Laager rlaager at wiktel.com
Thu Jul 3 00:15:56 UTC 2025


NAK on 0001-ntpviz-Make-some-conditionals-Yoda-style.patch and the 
similar part of the change in 
0001-ntpviz-correct-usaage-for-no-os.errno-ever.patch. That style is not 
common in Python, as it is unnecessary. Python does not allow "if x = 
y", so you cannot make the mistake that style prohibits (in C).


For 0001-ZZZ-ntp.ntpstats-ntpviz-change-refclock-name-punting.patch, I 
like that it expands coverage to other refclock types. I'm not sure 
where this is used, exactly, so I'm unsure about how I feel about the 
change from PPS(1) to pps(1).


For the rest, I'm prepared to merge them, if you can sanity check my 
small changes: https://gitlab.com/NTPsec/ntpsec/-/merge_requests/1474

-- 
Richard
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ntpsec.org/pipermail/devel/attachments/20250702/930c66d5/attachment.htm>


More information about the devel mailing list