View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0002983||3 - Current Dev List||Maintenance||public||2018-12-12 12:09||2019-10-27 21:31|
|Target Version||Fixed in Version|
|Summary||0002983: Maintenance: some modules suppress dangerous warnings|
The HRDStation, HRDManagerService, and Setup2 solutions have, in total, more than 100 invocations of the "#pramga warning(suppress" directive to turn off warnings.
Blindly turning off warnings without using them to improve the codebase actually does more damage because these directives hide potentially dangerous code from being tagged by the compiler in each build, or in speical static-analysis builds. The directives should probably be summarily removed, and we should start over with properly addresing Code Analyzer builds. Or, they should be visited and examined to determin their viability.
|Steps To Reproduce|
1) Load the HRD Solution
2) Search the entire solution for "#pragma warning(suppres"
BUG#1) About 105 total hits, none with comments indicating a solid reason for suppressing the error.
|Tags||No tags attached.|