Manually targeting a square should target any target_able monster

[PowerWyrm]

When use_old_target is set to "yes" and you aim a wand of teleport other at a monster, the next time you aim the wand it uses up a charge but does nothing else. I guess it's because the target is still set to the now out-of-LOS teleported monster.

Probably related to #332.

[PowerWyrm]

In fact this happens when using manual targeting (p) on a square with a monster. The game now records the coordinates instead of the monster and simply aims at the same square if use_old_target is set.

This is in fact a side effect of fixing #1792. On my local copy, I don't have this problem, because I fixed #1792 differently (target_set_location):

    /* Legal target */
    if (cave_in_bounds_fully(cave, y, x))
    {
        int m_idx = cave->m_idx[y][x];

        /* Save target info */
        target_set = TRUE;
        target_who = NULL;
        if (m_idx)
        {
            struct monster *mon = cave_monster_at(cave, y, x);
            if (target_able(mon))
                target_who = mon;
        }
        target_y = y;
        target_x = x;
    }

Before commit 61ca4d8, the monster was targeted properly, but the coordinates were not set. Commit 61ca4d8 did set the coordinates properly, but allowed non target_able monsters to be chosen. Commit 4901b95 disallowed any monster to be manually targeted. I think the above fix should work properly by setting the coordinates properly and allowing any target_able monster to be targeted.

[nckmccnnll]

This looks like it needs fixing

[nckmccnnll]

Fix as above applied in d43c3fb

[nckmccnnll]

After the discussion referenced in #1946, this was reverted in 19d624f.