#152 `bomb' lets players find subs too easily
Normally, planes have to make sub detection check.
Except for
pin_bomb(), which always tells whether subs are in the
sector.
Possible fix:
0. Mark all planes uncommitted
1. If no uncommitted planes remain, this bombing run
ends. Else show targets, except for subs at sea.
2. Ask for kind of target; if there are uncommitted
planes that can bomb subs at sea, and this is `at sea',
then accept `ships' even if there are none (so you
don't learn anything)
3. While there are uncommitted planes
- Choose one
- If bombing ships at sea and can bomb subs, search
for subs with this plane (commits plane)
- Offer targets (including subs this plane found)
and offer to back out
- If player chooses `back out', go to 1.
- If player chooses a target, bomb it with this
plane (commits plane)
This deals with player errors rather gracefully, yet
doesn't ever reveal subs for free.
It also lets you bomb different kinds of targets in in
the same bombing run, which may or may not an improvement.
