<div dir="ltr"><div dir="ltr"><div><br clear="all"></div><div><div dir="ltr" class="gmail_signature"><div dir="ltr"><font face="tahoma, sans-serif" color="#666666">Cheers,<br>Dave Hart</font></div></div></div><br></div><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Jul 3, 2025 at 3:15 AM Richard Laager (@rlaager) via vc <<a href="mailto:vc@ntpsec.org">vc@ntpsec.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="msg9204065444261560541"><u></u>










<div style="font-size:inherit;font-family:"GitLab Sans",-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Noto Sans",Ubuntu,Cantarell,"Helvetica Neue",sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji"">
<div>

<ul><li><pre style="white-space:pre-wrap;display:block;font-size:14px;color:rgb(58,56,63);font-family:"GitLab Mono","JetBrains Mono",Menlo,"DejaVu Sans Mono","Liberation Mono",Consolas,"Ubuntu Mono","Courier New","andale mono","lucida console",monospace;font-variant-ligatures:none;word-break:break-all;background-color:rgb(251,250,253);border-radius:2px;margin:0px;padding:8px 12px;border:1px solid rgb(220,220,222)">[Reverted changed to non-Pythonic == idiom.  Tweaked commit message.]
</pre>
</li>
</ul>
<h4 style="margin-top:10px;margin-bottom:10px">
1 changed file:
</h4>
<ul>
<li>
<a href="#m_9204065444261560541_5d7267c06a32341332961589a5641bb0142dca20">
ntpclients/ntpviz.py</a></li></ul></div></div></div></blockquote><div> </div><div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif">I learned to put an rvalue on the left-hand side of == comparisons as a safety idiom for C/C++, sure, but I also find it speeds code comprehension.</div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif"><br></div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif">if (ENOENT == errno)</div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif">vs.</div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif">if (errno == ENOENT)</div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif"><br></div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif">errno is cognitive noise.  Just my two cents.  I'm not advocating churning existing code but you might consider it writing new code, regardless of whether it's C/C++.</div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif"><span style="color:rgb(102,102,102);font-family:tahoma,sans-serif"><br></span></div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif"><span style="color:rgb(102,102,102);font-family:tahoma,sans-serif">Cheers,</span><br style="color:rgb(102,102,102);font-family:tahoma,sans-serif"><span style="color:rgb(102,102,102);font-family:tahoma,sans-serif">Dave Hart</span><br class="gmail-Apple-interchange-newline"></div></div></div></div>