BlindTransaction() and VerifyAmounts() need detailed error return codes
Nessuno ha ancora preso questa issue.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
The functions BlindTransaction() and VerifyAmounts() each have several failure paths, but only use -1 and false to indicate to the caller that an operation failed. This makes it difficult to report helpful error messages to the user.
The error paths could return an enum indicating the reason for failure, and the callers could then surface error strings to the user.
BlindTransaction(): https://github.com/ElementsProject/elements/blob/master/src/blind.cpp#L286
VerifyAmounts(): https://github.com/ElementsProject/elements/blob/master/src/confidential_validation.cpp#L134
- Lingua principale
- C++
- Stelle
- 1.2k
- Fork
- 418
- Merge medio
- 1g 12h
- PR unite (30g)
- 13
Guida per i contributori
Apri 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 ElementsProject/elements
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 58/100
ElementsProject/elements#1555 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
ElementsProject/elements#1521 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 32/100
ElementsProject/elements#1516 · 5 commenti ·
-
Compilation error on WSL Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
ElementsProject/elements#1502 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 38/100
ElementsProject/elements#1500 · 2 commenti ·
Tutte le issue di ElementsProject/elements
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
ros2/message_filters#338 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
subsurface/subsurface#4984 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
google-ai-edge/LiteRT-LM#3739 ·