PowerToys icon indicating copy to clipboard operation
PowerToys copied to clipboard

[Analyzers][CPP] turn on Warning 4100

Open sosssego opened this issue 2 years ago • 0 comments

Turn on warning 4100, unreferenced formal parameter.

PR Checklist

  • [x] Continues towards: #940
  • [x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
  • [x] Tests: Added/updated and all pass - no new tests
  • [x] Localization: All end user facing strings can be localized - no new strings
  • [ ] Dev docs: Added/updated
  • [ ] New binaries: Added on the required places
  • [ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Turn on 4100 warning and fix the code based on C++ core guidelines

Remove unused global variable from src\modules\MeasureTool\MeasureToolModuleInterface\dllmain.cpp

Validation Steps Performed

Unit tests and CI

sosssego avatar Oct 25 '22 21:10 sosssego