[torquedev] DBPRT statements and #ifdef 0
dbeer at adaptivecomputing.com
Tue Mar 15 15:23:30 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?
Direct Line: 801-717-3386 | Fax: 801-717-3738
1656 S. East Bay Blvd. Suite #300
Provo, UT 84606
More information about the torquedev