ElementsProject / ElementsProject/elements
Seemingly useless assignment in reissueasset_base()
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C++
- Sterne
- 1.2k
- Forks
- 417
- Ø Merge
- 1 T. 23 Std.
- Gemergte PRs (30 T.)
- 15
Beschreibung
https://github.com/ElementsProject/elements/blob/e5ab941489c564cd78845658b8e8688dc56817f5/src/rpc/rawtransaction.cpp#L2896-L2912
the assignment
```
mtx.vin[issuance_input_index].assetIssuance.nAmount = asset_amount;
```
at line 2912 is the same as the assignment at line 2898
neither `asset_amount` nor `assetIssuance.nAmount` seem to be modified between the lines.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in src/rpc/rawtransaction.cpp bei reissueasset_base(), insbesondere in den Zeilen 2896-2912. Vergleiche die Zuweisungen in den Zeilen 2898 und 2912 und überprüfe, ob sich asset_amount oder assetIssuance.nAmount zwischen ihnen ändert. Erledigt ist die Aufgabe, wenn geklärt ist, ob die spätere Zuweisung redundant ist, und die entsprechende Bereinigung vorgenommen oder dokumentiert wurde, warum sie erforderlich ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- blockchain
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 50/100