#1 Pointless reassignement of lastMinerRecord

Aperto
aperto 3 anni fa da TooManySugar · 0 commenti
TooManySugar ha commentato 3 anni fa

https://gogs.veloe.link/Veloe/EthemineTelegramBotCore/src/master/EthermineBotTelegramCore/UpdateData.cs#L88-L90

lastMinerRecord already assigned at line 42. Assignment here assigns the same value as was before.

Otherwise if emit Botdb.SaveChangesAsync() after Botdb.miners.Add(newMinerRecord) (line 46) this assignment would always lead to equality with newMinerRecord.

https://gogs.veloe.link/Veloe/EthemineTelegramBotCore/src/master/EthermineBotTelegramCore/UpdateData.cs#L88-L90 lastMinerRecord already assigned at line 42. Assignment here assigns the same value as was before. Otherwise if emit `Botdb.SaveChangesAsync()` after `Botdb.miners.Add(newMinerRecord)` (line 46) this assignment would always lead to equality with newMinerRecord.
Sign in to join this conversation.
Nessuna etichetta
Nessuna milestone
Nessun assegnatario
1 Partecipanti
Caricamento...
Annulla
Salva
Non ci sono ancora contenuti.