By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 2048 - Removed unused #if / #ifdef blocks
Summary: Removed unused #if / #ifdef blocks
Status: NEW
Alias: None
Product: TDE
Classification: Unclassified
Component: other (any) (show other bugs)
Version: R14.0.x [Trinity]
Hardware: Other Linux
: P5 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2014-05-06 09:57 CDT by Michele Calgaro
Modified: 2018-05-27 10:49 CDT (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michele Calgaro 2014-05-06 09:57:57 CDT 
As a small code enhancement, it would be good to remove unused #if blocks.
For example blocks such as #if 0
Comment 1 Timothy Pearson 2014-05-06 14:35:07 CDT 
Some of those blocks disable useful debug code.  Are we sure we want to remove them?
Comment 2 Michele Calgaro 2014-05-07 07:27:54 CDT 
I guess we will have to distinguish case by case.
Code that is considered useless shall be removed, while for those "#if 0" blocks that contain possible useful code, I propose we change them to "#ifdef USEFUL_DEFINE_NAME", so that we can easily enable/disable the code at compile time without the need for editing the code each time.
Comment 3 Timothy Pearson 2014-05-07 09:12:11 CDT 
Sounds good to me!