Remove enforcement rules of rule F.17
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
- Tipo di issue
- Documentazione
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- cpp
- Ambito
- documentation
Direzione di ricerca
Inizia individuando le sezioni delle linee guida F.17 e Con.3 e confrontando le relative regole di applicazione. Rimuovi entrambe le regole di applicazione di F.17, preservando le indicazioni e le eccezioni dichiarate in F.20; verifica la coerenza della formattazione e dei riferimenti nel documento circostante.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Rule F.17 (For “in-out” parameters, pass by reference to non-const) and Con.3 (By default, pass pointers and references to consts) have both two enforcement rules of which one of them is nearly identical:
- F.17: Warn about functions regarding reference to non-const parameters that do not write to them
- Con.3 Flag a function that does not modify an object passed by pointer or reference to non-const
This means that 2 rules will fire if a non-const reference parameter is not changed in the corresponding function body. Having 2 rules firing for the same situation is a coding standard design smell.
In order to solve this, I suggest to remove all enforcements from rule F.17 because the rule is about expresssing an intention, i.e. an in-out parameter. That is something you can't check automatically. To take this one step further, one might even consider to remove F.17 completely because we also have rule F.20 (For “out” output values, prefer return values to output parameters). Since F.20 describes some exceptions, removing F.17 might a bit too strong. So I suggest to remove the 2 enforcement rules of rule F.17.
- Lingua principale
- CSS
- Stelle
- 45.4k
- Fork
- 5.6k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di isocpp/CppCoreGuidelines
-
Use short #s-xxx as ref, not long like #sl-the-standard-library or #gsl-guidelines-support-libraryAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
isocpp/CppCoreGuidelines#2340 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
isocpp/CppCoreGuidelines#2205 · 8 commenti ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 64/100
isocpp/CppCoreGuidelines#1649 · 3 commenti ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
isocpp/CppCoreGuidelines#2342 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
isocpp/CppCoreGuidelines#2335 · 2 commenti ·
Tutte le issue di isocpp/CppCoreGuidelines
Issue simili
-
Liens externes cassésAperta
Difficoltà 1/5 1-3 ore Idoneità per principianti 66/100
betagouv/conseillers-entreprises#4770 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
awsdocs/amazon-eks-user-guide#1257 ·
-
eks-recon: six more null-iteration crashes (networking env/args, observability ConfigMap data)Apertadeep-dive-feedback
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
aws-samples/sample-apex-skills#292 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
italia/eid-wallet-it-docs#1253 ·
I maintainer di solito rispondono entro 13 giorni
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno