xZero707/droidfish
1
0
Fork 0
mirror of https://github.com/peterosterlund2/droidfish.git synced 2024-11-23 11:31:33 +01:00
Code Issues Packages Projects Releases Wiki Activity

Don't try to fixup invalid e.p. square if the position is invalid

Browse Source 
En passant fixup requires generating a list of legal moves, which can
fail with an array index out of bounds exception if there is a pawn on
the first/last rank.
This commit is contained in:
Peter Osterlund 2020-01-03 10:32:49 +01:00
parent 61317fa78d
commit 1afda36d85
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
DroidFishApp/src/main/java/org/petero/droidfish/activities/EditBoard.java
View File

@ -442,7 +442,8 @@ public class EditBoard extends Activity {
private void setPosFields() {
setEPFile(getEPFile()); // To handle sideToMove change
TextIO.fixupEPSquare(cb.pos);
if (isValid())
TextIO.fixupEPSquare(cb.pos);
TextIO.removeBogusCastleFlags(cb.pos);
}
@ -478,6 +479,16 @@ public class EditBoard extends Activity {
return false;
}
/** Return true if the position is valid. */
private boolean isValid() {
try {
TextIO.readFEN(TextIO.toFEN(cb.pos));
return true;
} catch (ChessParseError e) {
return false;
}
}
private String getParseErrString(ChessParseError e) {
if (e.resourceId == -1)
return e.getMessage();