OpenXRay/xray-16

Compiler warnings elimination

Ouverte

#713 ouverte le 8 janv. 2021

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (1 personne assignée)C++ (519 forks)auto 404
Code QualityHelp wantedPortability

Métriques du dépôt

Stars
 (3 524 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Starting from the creation of this task, #116 is a subtask of this task.

MSVC:

  • MSBuild
    • Enable "warnings are errors" mode (is it possible?)
  • Compiler
    • Enable conformance mode
    • Enable "warnings are errors" mode
    • Raise the warning level to 4
      • Level 1
      • Level 2
      • Level 3
      • Level 4
  • Linker
    • Enable "warnings are errors" mode

GCC:

  • CMake and make
    • Enable "warnings are errors" mode (is it possible?)
  • Compiler
    • Disable permissive mode
    • Maybe enable pedantic mode
    • Enable "warnings are errors" mode
    • Raise the warning level to 4
      • Level 1
      • Level 2
      • Level 3
      • Level 4
  • Linker
    • Enable "warnings are errors" mode

For history:

  • List all warnings elimination commits in this issue

Tools: PVS-Studio CppDepend (includes cppcheck, Clang-Tidy, Vera++) ReSharper C++ (includes Clang and Clang-Tidy check plus it's own analysis) Visual Assist (has it's own code inspection)

https://habr.com/ru/articles/75123/ https://habr.com/ru/companies/pvs-studio/articles/303664/

Guide contributeur

Direction de recherche
Examinez la sortie des avertissements du compilateur pour chaque configuration de construction. Identifiez les codes et les indicateurs d'avertissement, puis modifiez les fichiers CMakeLists.txt ou de projet MSBuild pour ajouter les indicateurs de compilateur appropriés afin d'activer les avertissements en tant qu'erreurs. Testez chaque système de construction localement.
Stack technique
ccpp
Domaine
build system
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++CMakeMSBuildGCC
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40