Enum.7 Reason text is misleading
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 48/100
- Tipo di issue
- Documentazione
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- cpp
- Ambito
- documentation
Direzione di ricerca
Cerca nel repository Enum.7 e il testo Reason citato, quindi leggi la linea guida collegata e la motivazione che la circonda. Aggiorna la formulazione della linea guida per evitare di implicare che int sia sempre il valore predefinito e verifica che la documentazione renderizzata mantenga la spiegazione prevista.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The current "Reason" for Enum.7 is currently
**Reason** The default is the easiest to read and write. int is the default integer type. int is compatible with C enums.
This feels misleading, as a quick and natural reading of it strongly implies that the default type for an enum is int, when it is not. Both clang and gcc default to unsigned int on linux, while MSVC defaults to int on x64 for enums with smallish values (see https://godbolt.org/z/fPWczEon3).
The default type is also going to change based on the actual values in the enum.
I don't believe that the more complex language in the standard about "the underlying type is an integral type that can represent all the enumerator values defined in the enumeration" is useful in this guideline, however maybe you could just say something like the following (just leave int out of the reason)?
**Reason** The default is the easiest to read and write and is compatible with C enums.
Or maybe rework this guideline entirely in the face of the potentially-confusing underlying behavior?
[edit: some grammar]
- Lingua principale
- CSS
- Stelle
- 45.3k
- 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
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 75/100
isocpp/CppCoreGuidelines#2338 ·
-
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 42/100
isocpp/CppCoreGuidelines#2336 · 4 commenti ·
-
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
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
inu-appcenter/memorIN-frontend#106 ·
I maintainer di solito rispondono entro 1 giorno
-
workflow: a tick's dispatch counts as 'only this step', and no review self-grants a round unattendedApertaworkflow
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
kristofdegrave/homeassistant-smart-charging#1505 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
openfoodfacts/score-my-recipe#80 ·
I maintainer di solito rispondono entro 2 giorni
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
githubnext/gh-aw-workshop#3968 ·
I maintainer di solito rispondono entro 1 giorno