Warning flag
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 58/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- cpp
- Domaine
- documentation
Piste de recherche
Ouvrez 02-Use_the_Tools_Available.md à la liste des options d’avertissement GCC / Clang et examinez les indications associées à nullptr dans 03-Style.md. Documentez -Wzero-as-null-pointer-constant de GCC avec la portée correcte du compilateur ; l’issue est terminée lorsque l’avertissement apparaît correctement dans la liste concernée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hello,
In your 03-Style.md best practice file you talk about using nullptr instead of 0 or NULL which is a very good things to do.
There is a flag in GCC that warn the user about using 0 (or NULL) instead of nullptr: -Wzero-as-null-pointer-constant. Unfortunately this flag does not exists for clang and I don't know if it does for MSVC.
I think this should be a good idea to add this flag in your warning flag list (https://github.com/lefticus/cppbestpractices/blob/master/02-Use_the_Tools_Available.md#gcc--clang).
Best regards
- Langage dominant
- Aucune donnée de langage
- Étoiles
- 8.9k
- Forks
- 903
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de cpp-best-practices/cppbestpractices
-
Broken Link Chapter 2: Use_the_Tools_AvailablePeut-être à nouveau libre @vishesharma20 l’a pris il y a 172 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 64/100
cpp-best-practices/cppbestpractices#169 · 1 commentaire ·
-
Broken Link - 11 Further ReadingPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 55/100
cpp-best-practices/cppbestpractices#168 · 1 commentaire ·
-
C++ PracticeOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 10/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 35/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
cpp-best-practices/cppbestpractices#160 · 1 commentaire ·
Toutes les issues de cpp-best-practices/cppbestpractices
Issues similaires
-
area-multi-tenancy p1-recommended technical-debt
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
Docs: 2 broken linksOuverteBug pulumi/pulumi-terraform-bridge R2 S2
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
pulumi/pulumi-terraform-bridge#3713 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
[Sync EN] Document the timezone of DateTime(Immutable)::createFromTimestamp() (#6074)Peut-être pris @lacatoire l’a pris aujourd’hui. Ouvertesync-en
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
[Sync EN] Document the timezone of DateTime(Immutable)::createFromTimestamp() (#6074)Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertesync-en
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Les mainteneurs répondent en général sous 1 jour