<div dir="ltr">I often use DEBUG_MRA_FOO, and then use judgement and taste as to leave it in or not.<div><br></div><div>I use "is it worth writing the documentation for it" as part of deciding to leave it in or not, and also "am I absolutely sure I have characterized and fixed this bug such that I'm sure it's fixed, and will not recur".</div><div><br></div><div>Often, I once I'm done, I will remove the DEBUG_MRA_FOO and replace it with an assert, again using judgement and taste.</div><div><br></div><div>I will leave it to your judgement and taste.</div><div><br></div><div>..m</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, May 19, 2017 at 12:21 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>
Have you thought about logging and/or printout for debugging?<br>
<br>
We have DEBUG and DPRINTF.  I almost never use them because the signal to<br>
noise is so low - it prints stuff from all over but I'm only interested in<br>
one small corner.<br>
<br>
I occasionally add various msyslog lines when I'm chasing a bug and throw<br>
them away after I have found it.<br>
<br>
Would it make sense to preserve them with something like an ifdef DEBUG_FOO<br>
symbol?<br>
<br>
Or if (DEBUG_FOO) so it only adds one extra line of source code rather than<br>
2?  (no endif)<br>
<br>
Or just put a // in the front of the ones you don't want.<br>
<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><div dir="ltr">-- <br></div><div 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 SMS & Signal</p>
</div></div>