#1 Pointless reassignement of lastMinerRecord

Offen
vor 3 Jahren geöffnet von TooManySugar · 0 Kommentare
TooManySugar kommentierte vor 3 Jahren

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.
Anmelden, um an der Diskussion teilzunehmen.
Kein Label
Kein Meilenstein
Niemand zuständig
1 Beteiligte
Laden…
Abbrechen
Speichern
Hier gibt es bis jetzt noch keinen Inhalt.