Byron/trash-rs

Use references in `restore_all` and `purge_all`

Aperta

#54 aperta il 17 lug 2022

Vedi su GitHub
 (12 commenti) (0 reazioni) (0 assegnatari)Rust (40 fork)auto 404
enhancementgood first issuehelp wanted

Metriche repository

Star
 (247 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

restore_all and purge_all should take references because they don't need the ownership of TrashItems. This could reduce cloning when calling the functions. restore_all might need an additional clone on a RestoreCollision error because there is remaining_items. My use case is specified in #53 .

Guida contributor

Direzione di ricerca
Modifica `restore all` e `purge all` per accettare `&[TrashItem]` invece di `Vec<TrashItem>`. Adatta il codice interno per lavorare con i riferimenti e gestisci l'errore `RestoreCollision` clonando gli elementi rimanenti se necessario.
Tech stack
rust
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75