Simplify with C++17
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia eseguendo un audit di mp/util.h e dei pattern indicati per AUTO_RETURN, Priority<0/1/2> e l’indicizzazione delle tuple; esamina anche gli utilizzi di kj::mvCapture. Suddividi il lavoro in sostituzioni mirate usando le funzionalità di C++17 e considera la conservazione del comportamento esistente dopo ogni modifica.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Saving note from an email, there are a ton of a simplifications that can be made now that C++17 is required:
A lot of code needs better comments, and there are also big swathes that are overcomplicated because they were written to use c++11, and can now be simplified with c++17. Examples:
- AUTO_RETURN macro uses can be replaced with auto return types
- Priority<0>, Prioriity<1>, Priority<2> overloads used for conditional compilation can all go away and be replaced with if constexpr
- std::tuple indexing code can go away and be replaced with std::apply
Basically all of mp/util.h can go away. Also kj::mvCapture uses can replaced with shorter direct captures.
- Lingua principale
- C++
- Stelle
- 62
- Fork
- 58
- Merge medio
- 9g 9h
- PR unite (30g)
- 3
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 bitcoin-core/libmultiprocess
-
Listeners can't be stopped, so bitcoin-node can hang after shutdown when an IPC client reconnectsApertabug
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
bitcoin-core/libmultiprocess#385 ·
I maintainer di solito rispondono entro 1 giorno
-
ListenConnections stops listening for good after one temporary accept() failure such as EMFILEApertabug hardening
Difficoltà 3/5 1-2 giorni Idoneità per principianti 57/100
bitcoin-core/libmultiprocess#384 ·
I maintainer di solito rispondono entro 1 giorno
-
Calling an IPC object after or during destroy() makes the server use a null or freed implementationForse già presa @xyzconstant l’ha presa 9 giorni fa. Apertabug hardening
Difficoltà 4/5 3-5 giorni Idoneità per principianti 40/100
bitcoin-core/libmultiprocess#378 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
iwyu: --check_also=*.h ?Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
bitcoin-core/libmultiprocess#329 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 62/100
bitcoin-core/libmultiprocess#328 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di bitcoin-core/libmultiprocess
Issue simili
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
EchoTools/nevr-runtime#2042 ·
I maintainer di solito rispondono entro 1 giorno
-
Download size and speed show scientific notation between 1000 and 1023 unitsForse già presa @RaphaelFakhri l’ha presa oggi. Aperta
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 88/100
kiwix/kiwix-desktop#1570 ·
I maintainer di solito rispondono entro 1 giorno
-
Device Request
Difficoltà 2/5 Mezza giornata Idoneità per principianti 62/100
dresden-elektronik/deconz-rest-plugin#8686 ·
I maintainer di solito rispondono entro 1 giorno
-
nixos/noctalia-greeter: greeter finds no sessions (session dirs not passed via XDG_DATA_DIRS)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
aminya/project_options#303 ·