<br><br><div class="gmail_quote">2011/3/15 &quot;Mgr. Šimon Tóth&quot; <span dir="ltr">&lt;<a href="mailto:toth@fi.muni.cz">toth@fi.muni.cz</a>&gt;</span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">&gt; Someone mentioned a while back removing the DBPRT statements from TORQUE. We are considering removing these statements for TORQUE 4.0. Of course, we would like to know if these statements are still useful to anyone or are used by anyone? If the answers to these questions are no and no, can anyone think of another reason why these should stay in the code?<br>

&gt;<br>
&gt; Additionally, one of the developers pointed out the existence of code segments wrapped by #if 0 (see start_exec.c for examples). Most of these haven&#39;t been changed for about two years. Obviously, before anything can be deleted it has to be inspected and considered in context, but does anyone have objections to moving towards removing this code? Would anyone volunteer to inspect some of the code to see if it has any value or a reason to keep it?<br>

&gt;<br>
&gt; I suppose there are two ways you can approach this problem:<br>
&gt;<br>
&gt; 1. There is no reason to keep extra clutter in the code that just makes it harder to understand.<br>
&gt; 2. There is no real benefit other than aesthetic for deleting code, and there is a risk of losing information when some code is deleted.<br>
&gt;<br>
&gt; What are your thoughts?<br>
<br>
</div>Some of the DBPRT should be probably converted into standard logs (debug<br>
level).<br></blockquote></div><br>I agree. <br><br>While we&#39;re looking at loggin I also think TORQUE is guilty of some excessive logging in some cases (is it really necessary to print the version every 30 seconds or so)<br>