<div dir="ltr">Putting the globals into a controlled struct make them easier to reason about, both for humans and for source code analysis.  And even if the resulting struct is little more than the "globals dumping ground", it does force that they all be declared in one single place, in a place where you have to admit "this is a global".</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Feb 21, 2018 at 10:58 AM Hal Murray via devel <<a href="mailto:devel@ntpsec.org">devel@ntpsec.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> I've been looking at the code around mode 6 generation and discovered  that<br>
> in some areas it's still globals all the way down. Translating  these<br>
> globals will make future refactoring/translating easier.<br>
<br>
I'm missing the big idea.<br>
<br>
The current case is that we have a lot of global variables.<br>
<br>
What does packaging them in a struct solve?  We aren't going to pass a<br>
pointer to the struct around all over the place.<br>
<br>
Should we cleanup the names so it's obvious which variables are global?<br>
Should we reorganize the header files so it's easier to find all of them?<br>
<br>
<br>
--<br>
These are my opinions.  I hate spam.<br>
<br>
<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@ntpsec.org" target="_blank">devel@ntpsec.org</a><br>
<a href="http://lists.ntpsec.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ntpsec.org/mailman/listinfo/devel</a><br>
</blockquote></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><p dir="ltr">Mark Atwood<br>
<a href="http://about.me/markatwood">http://about.me/markatwood</a><br>
+1-206-604-2198</p>
</div></div>