#1 Pointless reassignement of lastMinerRecord

オープン
3 年 前TooManySugar によって開かれました · 0 コメント
TooManySugar3 年 前 にコメントしました

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.
会話に参加するには サインイン してください。
ラベルなし
マイルストーンなし
担当者なし
1 参加者
読み込み中…
キャンセル
保存
まだコンテンツがありません