cpluspluscom/ChessPlusPlus

Redundant `to` parameter in Piece::moveUpdate()

LB-- opened this issue · 0 comments

LB-- commented

When Piece::moveUpdate() is called, the piece is already at the position indicated by to. This redundancy allows for subtle bugs if a coding mistake causes to to not be the actual position of the piece.