Minor bug fixes, preparatory to finding out why replay doesn't work.

Avoid divide-by-zero error.

Many small port fixes in the Python.

Get rid of game.ithere, we just test whether there is a valid
coordinate object for the Tholian instead.  Also, use deepcopy to make
snapshots.

OK, warp flight between quadrants works now.

Various minor fixes to movement code.

...and avoidance of errors raised by indexing with a float.

Local navigation now works in Python.

Flight to other quadrants doesn't yet.  (The bugs all appear to be
consequences of moving to zero indexing.)

Get rid of some dead code.

Planet pointer mess cleaned up.  planets now works.

status command is almost working.  Must resolve the mess with planet pointers.

lrscan, scescan, and chart are working in the Python version.

Eliminate iqhere global in order to get rid of spurious Python warning.

Where we need this flag, we now deduce it from the location of the Thing.

0-origin code gets through initialization OK.

Checkpoint Python version before trying to move to zero-origin indexing.

Python IO queue appears to work.

Python version did its first I/O.

Python translation is syntactically correct.  Now for the grotty
part...

setup.py code added to Python translation.

Python translation now incorporates reports.c.

planets.c code incorporated into Python translation.

Code from moving.c incorporated into Python translation.

Incor[porated io.c code into Python translation.

finish.c code incorporated into Python translation.

Correct minor bug in figure-of-merit scoring.

Inhabited-planet kills weren't being figured in. Also make the code clearer.

More of Stas's notes about the time recomputation bug.

Incorporated events.c into Python.

Small cleanups discovered during Python translation.

Another rollup patch.

- fixed manual moving
- quick hack to avoid division by zero when calculating game.state.remtime
- fixed some problems with sensors() on ncurses
- small translation fixes

battle.c incorporated into Python translation.

adjustments

Eliminate uses of cramlc().

oops, revert ocasional change

Another rollup patch.

- Enabled more gcc warnings
- Fixed -Wundef warnings
- Fixes to inhabited-worlds code, the attempt to enslave the
  uninhabited planet is no longer made.
- Replaced pause_game(true) with announce() and killed the pause_latch()
  trick as it leaded to the lost scroll-pauses
- Fixed calculation of destroyed planets
- Made scan() locale-safe
- Translation updates

Gameplay changes:
- Removed the Romulans-guarding-inhabited-worlds behaviour that probably
  came from bsd-trek. Frankly, I find it much more annoying than enjoying.
- Added the behaveour of transporter consuming the ship's energy - inspired
  by Star Control 2's landing habit. If someone finds this too harsh -
  feel free to remove, but I personally find that logical. (!OPTION_PLAIN)

Python translation now includes ai.c.

Get rid of some macros that block mechanical translation.

Create blank objects in all arrays during Python game state initialization.

Reset all structures for stupid literal translation of the game.

Get rid of DESTROY macro.

Get rid of the iterator macros I wrote.

They interfere with mechanical translation.

Hack boxed xcomments so they're easier for a mechanical translator to recognize.

Clean up ugly FORTRANISH names.

It's been years since C public names had a  6-character significance limit.

More cleanup towards automated Python translation.

More elimination of FORTAN-style if statements.

Reset the translation, we're going to redo it with ctopy.

Get rid of obnoxious visible "have we paused?" state.

This change shouldn't make any behavioral difference, but it
simplifies the call sequence and code of a couple of functions.

Change some ugly and misleading names.

Clean up nasty FORTRAN-style if statements.

E.g., with the body on the same line as the guard.  They're bad style
and make the code hard to read, and I got fed up,

Parallel cleanups in the C and scratch Python code.

bugfix in fry() and translation

better fix for is_valid()

Yet another rollup patch.

- proper localization of dreprt()
- fixed preport() which didn't work at all
- fixed a nasty bug in photon() which caused a misfired torpedo to proceed
- fixed a very nasty bug in is_valid()
- many translation fixes

few assorted adjustments and a fix for #8883

fix makehelp.py invocation

Trial Python now incorporates ai.c

Another rollup patch.

- use ARRAY_SIZE
- cleaned up srscan()
- introduced a separate ncurses window for status
- removed completed TODO entries
- fixed SIGSEGV in tty mode when env LINES is undefined (always!)
- made "Please enter password" message visible
- made the above message to appear for OPTION_PLAIN instead of !OPTION_CURSES
- killed enqueue()
- localized status() (added helper prstat() )
- enabled gcc to check the format arguments, which uncovered a lot of
  the real bugs!
- fixed the bugs uncovered by the gcc format checking (the last one in
  ai.c is left for Eric :)
- fixed problem with skip(): it relied on "linecount", but "linecount"
  doesn't increment when you enter the multiline answers, like, for example,
  the torpedo courses. The fix works only in curses mode though - no idea
  how to fix this for tty.
- localized prouts(), however, not very cleanly. Any help is appreciated,
  see the added comments.

Beginning of experimental Python translation.

The initial file is definitions corresponding to sst.h.

Added appending on development timeline.

Abstract away the operations involving a sentinel value for coordinates.

Besides making the code more readable, the sentinel may change someday
if we go zero-origin everywhere.

Make the "crystals' member into an enumeration.

Also, add explanatory comments to the attack code.  Remove some redundant
logic to push you out of dock on an attack.

More applications of same().

Get rid of obnoxious iattak global.

This was a kluge to prevent recursion through an infinit regress of
moves leading to exit attacks forcing moves, etc.

On Stas Sergeev's suggestion, change where the logfile is written.

allow gcc to still do the bounds checking

Clean up some boolean arguments.

and this gettextized in while I was resolving the rejects...

this gettextized by mistake

Rollup patch.

- completed the first pass on the gettextization and translation
- better (IMHO) fix for the initialization of the gettextized string arrays
- do not clearscreen on exit to leave the "Galaxy Bird" message visible
- implemented scroll-pausing for "help"
- fixed enqueue() which didn't always work
- added IHW to cramen()

Incorporate BSD-Trek visual-scan code, just in case.

Chase more booleans and enums.

A bit of documentation swiped and edited from the BSD code.

BSD-Trek-like critical hits with weighting.

Replace more #defines with enums.

Change the condition field from an uint with #defines to an enum.

More boolean-chasing and coord applications.

Still more application of coord type.

More uses of distance() macro. Eliminate some magic numbers.

More boolean cleanups.  More consistent use of coord type

Anong other things, this allows us to write a distance() macro and get
rid of a lot of gnarly expressions.

Clean up more booleans.

Clean up the rather nasty logoc of snova(), and fix a minor navigation bug.

...(inability to select manual mode) introduced by a recent commit.

More boolean cleanups and explanator comments.

More type cleanup and explanatory comments.

More type cleanup.

More boolean cleanups.

Avoid a nasty bug in planet-name display, and fix another boolean.

More BSD features.

You can now lose if your entire crew dies in battle.  When abandoning
ship in a game with inhabited worlds enabled, they must have one in
the quadrant to beam down to; otherwise they die in space and this
counts heavily against your score (BSD Trek had score adjustment as a
to-do).  Docking at a starbase replenishes your crew.

more gettextizing and translating

Abstract out damage checks.

Implement BSD-Trek check in abandon-ship routine that transporter is
operational and an inhabited planet is nearby before crew can beam
down, otherwise they die in space.  (But this isn't counted against
your score yet.)

Klingon ship births are working.

More booleans cleanup.

FDISTR/FENSLV/FREPRO sequence looks like it's working.

Fix two minor bugs, one in the shove code and one in debugme().

Improvements to the testing machinery, including the replay option.

Improve the world list.

More int-to-boolean cleanup.  Make the FDISTR event work.

Checkpoint with several changes, committed so Stas can play with debug code.

1. Change planet member of quadrant struct back to integer so it can be saved.
2. Add code for torpedoing an inhabited world.  Score drops by 400 when you
   do that, it's the only outcome worse than being killed.
3. Events are now structures and can hold quadrant coordinates.
4. EXPERIMENTAL tag is gone, code for BSD-Trek events is in good enough
   shape to test now.
5. User input is always logged.
6. SEED command is implemented.

Input capture to 'sst-input.log' now happens automatically when idebug is on.

New debug-mode support.

Get rid of DEBUG, the instrumentation is cheap and we want to cut down
on the possibilities for unanticipated logic changes due to
configuration vagaries.  Take the debugging bit out of the game
structure.  Start on the logger.

Revert an incorrect bool type change.

More type cleanup.

Use stdbool facility from C99 so code checkers can do type-consistency
analysis.

Introduce 'coord' data structure, an (x,y) tuple.

Use it consistently in both data structures and arguments of functions
that handle coordinates.

The main benefit of this is that lots of conditional expressions become
significantly less gnarly.  We get a bit of reduction in the line count
by cutting down on pairs of coordinate assignments.

Abstract all references to the future array (outside of events.c) away.

This has two benefits:

1. New code is easier to read.
2. It prepares us for implementing stateful events with a different
underlying data structure.  We'll need this for some BSD-Trek features.