Fix: Make AUTO_PICKUP_SAFEMODE also consider ignored mobs

[inogenous]

Summary

Bugfixes "Make AUTO_PICKUP_SAFEMODE also consider ignored mobs"

Purpose of change

• Autopickup should not happen when AUTO_PICKUP_SAFEMODE is enabled and a monster is nearby.
• Makes AUTO_PICKUP_SAFEMODE also consider ignored monsters when evaluating whether or not to autopickup.
• Prevents the following situation:

1. Enable options: SAFEMODE, AUTO_PICKUP_ADJACENT, AUTO_PICKUP_SAFEMODE
2. Enable autopickup for rocks
3. Player is 4 tiles from dangerous monster
4. Press key to ignore monster
5. Take one step towards rock on the ground so that rock is now adjacent
6. Rock is picked up, even though player is close to dangerous monster and AUTO_PICKUP_SAFEMODE is enabled

Describe the solution

• In the above scenario, step 4 would set mostseen to 0 since the monster is ignored.
• Before this commit, autopickup checked for mostseen == 0, thus allowing autopickup even if AUTO_PICKUP_SAFEMODE is enabled
• With this commit, autopickup will instead check for whether there is any monster within safemode proximity that is dangerous, regardless of whether it is ignored or not

Describe alternatives you've considered

• It could also be argued that if the player chooses to ignore a monster, then the intention should also be that autopickup should still be allowed since the monster is ignored. However, the (current) wording of AUTO_PICKUP_SAFEMODE does not state anything about ignored monsters: If true, auto pickup is disabled as long as you can see monsters nearby.

Testing

• With this commit, in scenario listed above, step 6 does not pickup the rock.

Additional context