[torquedev] DBPRT statements and #ifdef 0

"Mgr. Šimon Tóth" toth at fi.muni.cz
Tue Mar 15 15:26:14 MDT 2011


> 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?
> 
> 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'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? 
> 
> I suppose there are two ways you can approach this problem: 
> 
> 1. There is no reason to keep extra clutter in the code that just makes it harder to understand.
> 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.
> 
> What are your thoughts? 

Some of the DBPRT should be probably converted into standard logs (debug
level).

Deleting code isn't an issue since the code isn't going anywhere, it's
still in SVN.

-- 
Mgr. Simon Toth


More information about the torquedev mailing list