X-Git-Url: https://jxself.org/git/?a=blobdiff_plain;f=doc%2FHACKING;h=abb692a03c587f1713645034d9ac8aa82058e7cb;hb=HEAD;hp=133f72f6e84c9bbe93f7540252a2d9466978d175;hpb=c8a6ef704e18389b06b0bbd5c755db606ac1c981;p=super-star-trek.git

diff --git a/doc/HACKING b/doc/HACKING
index 133f72f..e940439 100644
--- a/doc/HACKING
+++ b/doc/HACKING
@@ -40,11 +40,15 @@ Dave also says the Space Thingy should not be preserved across saved
 games, so you can't prove to others that you've seen it.  He says it
 shouldn't fire back, either.  It should do nothing except scream and
 disappear when hit by photon torpedos.  It's OK that it may move
-when attacked, but it didn't in the original.  (Whether the Thingy
-can fire back is now controlled by OPTION_THINGY and turned off if the
-game type is "plain" or "almy".  The no-save behavior has been restored.)
+when attacked, but it didn't in the original. (The no-save behavior
+has been restored.)
 
-The Faerie Queen, black holes, and time warping were in the original.
+Dave says the Faerie Queen, black holes, and time warping were in the
+original.  However, either he is misremembering about time warping or
+the original implementation was lost during the FORTRAN-to-C
+translation, as Stas Sergeev added a time-warping feature not
+previiusly present in 2006.  Tom Almy also claims to have added
+timewarping!
 
 Here are Tom Almy's changes:
 
@@ -232,7 +236,7 @@ coverage of the suite.
 Two things to do to the log before dropping it in the test directory:
 
 (a) Strip out all lines beginning with "#curses:" - these are for debugging
-the curses interface and are not needed or regression testing; removing
+the curses interface and are not needed for regression testing; removing
 them makes it easier to see the game commands.
 
 (b) Add a header comment line explaining what the log is a test for.
@@ -298,3 +302,7 @@ clockface-to-radian conversions.  For this reason, and others, the
 trig code is still a bit wacky and obscure.  Modify with caution
 and test thoroughly.
 
+Adding or remobing random() calls is very likely to break lots of
+regression tests. If you have to add new one, wrap them in a
+save/restore of the LCG state; there's an example of this around
+the setup for the Space Thingy.