move_ground() has code to handle getsect() failure. It's dead,
because it assumes getsect() returns a negative value on failure. It
actually returns zero then. The mistake can be traced back all the
way back to BSD Empire 1.1. Fix it.
Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
curx = start->sct_x;
cury = start->sct_y;
total_mcost = 0.0;
curx = start->sct_x;
cury = start->sct_y;
total_mcost = 0.0;
- if (getsect(curx, cury, §) < 0) {
+ if (!getsect(curx, cury, §)) {
logerror("move_path: getsect %d,%d", curx, cury);
return -1;
}
logerror("move_path: getsect %d,%d", curx, cury);
return -1;
}
*/
tmpx = curx + diroff[dir][0];
tmpy = cury + diroff[dir][1];
*/
tmpx = curx + diroff[dir][0];
tmpy = cury + diroff[dir][1];
- if (getsect(tmpx, tmpy, &next) < 0) {
+ if (!getsect(tmpx, tmpy, &next)) {
pr("You can't go there...\n");
*movstr = 0;
continue;
pr("You can't go there...\n");
*movstr = 0;
continue;