View Issue Details

IDProjectCategoryView StatusLast Update
00029833 - Current Dev ListMaintenancepublic2019-10-27 21:31
ReporterK7ZCZAssigned To 
PrioritynormalSeverityminorReproducibilityalways
Status newResolutionopen 
Product Version 
Target VersionFixed in Version 
Summary0002983: Maintenance: some modules suppress dangerous warnings
Description
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.


TagsNo tags attached.
ModuleAll
Sub-ModuleVarious
TestingNot Started

Relationships

related to 0003400 closedK7ZCZ Ham Radio Deluxe remove HRDManager and HRDManagerService from project 

Activities

K7ZCZ

2019-10-20 15:30

administrator   ~0008882

Last edited: 2019-10-20 15:30

View 2 revisions

HRDManager and HRDManagerServce have been removed from the project, which helps -- but HRDStation and Setup2 remain. They still needs to be cleaned-up

Issue History

Date Modified Username Field Change
2018-12-12 12:09 K7ZCZ New Issue
2019-10-20 15:30 K7ZCZ Relationship added related to 0003400
2019-10-20 15:30 K7ZCZ Note Added: 0008882
2019-10-20 15:30 K7ZCZ Note Edited: 0008882 View Revisions