F.45 needs another exception for rvalue overloads
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 38/100
- Tipo di issue
- Documentazione
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- cpp
- Ambito
- documentation
Direzione di ricerca
Parti dalla voce del codice sorgente di C++ Core Guidelines relativa alla regola F.45 e confronta la formulazione attuale con l’eccezione proposta. Il lavoro è completato quando la linea guida riconosce gli overload rvalue-reference per gli accessor dei membri come std::get(std::tuple) e std::optional::value(), con una formulazione coerente con la proposta.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
F.45 is titled "Don’t return a T&&" and states:
We don’t know of any other good examples [besides
std::move] of returning&&.
This is ignorant of functions such as:
The recommendations of CppCoreGuidelines shouldn't contradict common practice in modern C++ standard library features. One could argue that even the standard library uses are illegitimate and unsafe, and there is truth to that. However, the user would incur unjust cost for code such as:
// if .value() returned a value instead of an rvalue reference,
// this would unnecessarily create a temporary object
vec.emplace(get_optional().value());
The zero-overhead principle, a core design principle of the language from the very start arguably trumps the slight safety benefit of returning a value.
Proposed Changes
-We don’t know of any other good examples of returning &&.
+##### Exception
+Member accessors such as `std::get(std::tuple)`,
+or `std::optional::value()` have overloads for rvalue references,
+and those overloads return rvalue references.
+This is fine because it simply imitates what happens when accessing a member of an rvalue struct,
+or a member of an rvalue array.
- 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#2335 · 2 commenti ·
-
Provide a simple GSL utility to replace `static_cast<void>`Forse di nuovo libera @GabrielDosReis l’ha presa 227 giorni fa e non c’è nessuna pull request aperta. Aperta
isocpp/CppCoreGuidelines#2319 · 7 commenti · 1 assegnatario ·
Tutte le issue di isocpp/CppCoreGuidelines
Issue simili
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
ianktoo/project-monitor#14 ·
-
documentation good first issue help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
steerion-labs/bounded-agent-runtime#93 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation P2
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Parameterless constructor is not a must-provided by user when a struct has a field initializerAperta:watch: Not Triaged dotnet-csharp/svc fundamentals/subsvc
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
daniel-ospina/tortoise#9038 ·
I maintainer di solito rispondono entro 1 giorno