Clean up:
[dragonfly.git] / games / larn / Fixed.Bugs
1 This is a list of the fixes/enhancements made to larn V11.0 in Version 12.0.
2 (Version numbers consist of 2 parts: ver.subver.  When the save file format
3 changes, ver must be bumped.  This is why the next release of Larn is 12.0
4 and not 11.1. This is used in the savefile routines to check for out-of-date
5 save files).  This list was mainly meant to be a record of what changed,
6 for my own sanity.  It's included for your benefit (Warning: SPOILER!):
7
8 0.  lprintf() in fileio.c (now called io.c) has been changed to use varargs
9         so that its variable number of arguments usage is now portable.  Pyramids
10         primarily had this problem.
11
12 1.      Panic handler was added to signal.c.  This routine catches fatal errors
13         like segmentation faults, bus errors, illegal instructions, etc., and
14         trys to performs a savegame() before dumping core.  This helps prevent
15         the loss of a good game due to a game malfunction.  Also, the name of the
16         signal received is printed, instead of just its number.
17
18 2.      The version number of the program is now selectable from the Makefile.
19         see the symbols VER and SUBVER.
20
21 3.      When at an altar, pray and donate 3000000000 gp. and ye used to receive
22         a whopping amount of gold due to a wraparound problem with the signed 
23         ints.  This has been fixed by using unsigned longs when asking for money
24         amounts.
25
26 4.      It was possible that when compiled with work hours checking, checkpointing
27         enabled, and having "play-day-play" in the .larnopts file a segmentation
28         fault would occur at its first attempt to do a checkpoint.  This was due
29         to an improperly declared savefilename array in tok.c.  This has been fixed.
30
31 5.      on level H, casting a missile weapon (mle cld ssp bal lit) off the edge of
32         the level would mess up the display, as it didn't know when to stop.  This
33         is needless to say, fixed.  Absolute bounds are now in effect for missile
34         type spells, see godirect() in monster.c.
35
36 6.  The create monster routine will now create monsters in random positions
37         around the player.  Before, the 1st one would always be created to the
38         upper left.
39
40 7.      If you vpr or lit at a throne, it would summon a gnome king that you
41         would have to deal with.  However, as each throne has only one king with it,
42         successive vpr's should not create more gnome kings.  Presently, successive
43         vpr's will create more kings.  This has been fixed.
44
45 8.      The mechanism to manage spheres of annihilation has been reworked to provide
46         a cleaner design and to eliminate some possible problems.
47
48 9.      The spell gen (genocide monsters) has been implemented.
49
50 10.     When dropping a ring of strength and having been weakened to STR=3 the
51         player might end up with a negative strength.  Strength is now stored
52         in 2 variables, real strength, and strength bonuses.  Only real strength
53         can now be weakened down to a minimum of 3, so unless you have a ring of
54         strength -3 or less, strengths below 3 should not occur.
55
56 11. larn -h will now print out a list of all available command line options.
57
58 12. larn -o<optsfile> now lets you specify a .larnopts file on the command
59         line.  This was necessary as part of the solution to number 14 below.
60
61 13.     The "savefile:" statement has been aded to the .larnopts format to allow
62         specifying the savefilename (full path) for the savegame operation.
63         This too was needed as part of # 14 below.
64
65 14. A player id facility has been added to larn.  The complaint was that
66         the game used the userid to order the scoreboard, thus only one scoreboard
67         entry was allowed for each userid.  If the compile time symbol UIDSCORE
68         is defined at compilation time (see Makefile), this will still be true.
69         However, if this define is omitted, the game will create and manage a
70         file called ".playerids" where names are taken from the specified
71         .larnopts file (now a command line option) and assigned a unique playerid.
72         playerid's will now be used to govern scoreboard entry posting.  This
73         feature makes it easy for one person to have many characters, each
74         appearing on the scoreboard.  Be kind to your fellow players!
75         The philosophy of one score per player gives more players the opportunity
76         to bask in glory for all to see!
77
78 15. It is no longer required that the player be WIZID to create the scoreboard
79         or to examine the logfile.  Anyone with the correct wizard's password can
80         now use these command line options (password is only needed to create/clear
81         the scoreboard).  If you want to prevent players from zeroing the 
82         scoreboard, change the wizard's password. (in config.c) By the way, wizards
83         may be alot of fun, but they are prevented from being placed on any
84         scoreboard. (for clarification)
85
86 16. Monsters now have intelligence, that is some of them.  This determines if
87         the monster moves using the previously stupid movement method, or by using
88         the new IMM (intelligent monster movement) algorithm.  With IMM, monsters
89         will move around corners, avoid pits, traps, etc.  With increasing levels
90         of difficulty, more monsters will be using IMM.  Beware of IMM when 
91         aggravated!  Those little beasties can really find you!
92
93 17. Added the scroll of life protection.
94
95 18. Larn now consults the file ".holiday" to check for holidays if the TIMECHECK
96         option (no playing during working hours) is enabled.  Before, larn knew
97         nothing about holidays.  It should now let people play if it is a holiday.
98         The format for a .holiday entry is: "mmm dd yyyy comments . . .".
99
100 19. In nap() and napms() it is possible that with nap(0) or napms(0) there
101         would be an infinite loop and the game would hang.  The case of nap(0)
102         is now looked for.
103
104 20. The granularity of gold piles has been increased.  iarg[] has been changed
105         from char's to short's, so instead of 255 x 10^n granularity we now have
106         32767 x 10^n granularity.  This also means more than 255000 gp can be
107         dropped in one place.  Not realistic, but it prevents a worthless
108         annoyance.  Who said games were supposed to be realistic?
109
110 21. Termcap capability has been added to larn.  If the symbol VT100 is defined
111         in the makefile, the game will be compiled to use only VT100 compatible
112         terminals (Much more efficient).  If the symbol VT100 is omitted, the game
113         will be compiled to use the termcap entry for whatever terminal you are
114         using.  This involves an extra layer of output interpretation, as every
115         byte sent to the terminal must be inspected for control tokens.
116         Only 3 termcap entries need be found for the game to be functional:
117         CM (cursor movement), CE (clear to end of line), and CL (clear screen).
118         For a better display, the following are optional: AL (insert line), DL
119         (delete line), SO (Standout begin), SE (Standout end), and CD (clear to end
120         of screen).  The .larn.help file was left as is, with VT100 escape
121         sequences in it.  If the termcap version of larn reads it, it is translated
122         for the desired terminal type.  The .mail60* files have been removed, and
123         their text is now included in bill.c so it can be used with any terminal.
124         Note:  If compiled for termcap, and using a VT100, the display will act
125         a little different.  This is because the VT100 does not have insert line/
126         delete line codes, and the scrolling region must be simulated with vertical
127         wraparound instead of scrolling.  Thanks goes to Michiel Huisjes for the
128         original termcap patch.
129
130 22. When playing as wizard, if you go down stairs on 10 or V3, or up stairs
131         on H, 1, or V1, etc. you would be placed in a phantom zone where the display
132         was really weird ([-1] subscripting), and would eventually lead to a
133         segmentation fault.  Stairs and volcano shafts now check for the level
134         they are being used on.
135
136 23. In response to some sites having only unsigned chars (flame the
137         manufacturer), the chars that were used to store positive and negative
138         numbers have been changed to shorts.  This includes diroffx[], diroffy[],
139         iarg[][][], ivenarg[], and some others.  I believe the changes are correct,
140         but I have none of these machines to try it out on. (Volunteers?)
141
142 24. The function fullhit(n) in monster.c was supposed to return the damage
143         done by n full hits on a monster.  It only returned the damage for ONE hit,
144         thus severely limiting the usefulness of the web and sle spells.
145
146 25. Someone said that they were getting segmentation faults when they were
147         reading scrolls as the wizard.  I couldn't find the problem, which may
148         have had something to do with the signed char problem mentioned above.
149         However, I've added a check in read_scroll() and quaff_potion() to trap
150         any scroll or potion types that are not in the game.
151
152 26. "vt125" has been added to the acceptable terminal list
153         (checked only if compiled with -DVT100).
154
155 27. In savegame() and restoregame(), there was a 6 hardwired into the i/o
156         statements which assumed the size of struct cel was 6.  On some machines
157         this caused the rightmost part of each level to not be saved in a savefile.
158         These 6's have been replaced with sizeof(struct cel), and should now be
159         portable.
160
161 28. The option "no-beep" has been added to the .larnopts file.  When specified,
162         beeping is inhibited at the terminal.
163
164 29. When becoming wizard, no longer to you wear the ring of protection, and
165         null scrolls and potions are no longer created.
166
167 30. Many spelling errors have been fixed, both in player messages, and in the
168         code itself.  A thanks goes to Mars Gralia who sent me a detailed list of
169         the mistakes.
170
171 31. When a player wins a game, if getlogin() fails, a segmentation fault will
172         result, because the NULL returned from getlogin() is used as a pointer.
173         This call has been replaced (now using loginname already determined).
174         Also, the mail creation upon winning has been rewritten, mainly to allow
175         termcapping of the text.
176
177 32. The Larn Revenue Service will now always appear on level H.  Before, it
178         was only created if the player had outstanding taxes.  In that multiple
179         save files per player are now more possible, this was seen as incorrect.
180
181 33. Input buffer flushing is now in effect.  If the input char queue exceeds
182         5 bytes, the excess is discarded.  Also, if the player hits or gets hit
183         all input bytes are flushed (within 1).  This relieves the situation
184         where many moves have been typed ahead of the display and the player keeps
185         getting hit while the queue of moves is processed.
186
187 34. When a savefile has been altered, a warning message is displayed to the
188         effect that you've cheated, and you will not be placed on the normal
189         scoreboard.  If you then save the game, and start 'er up again, memory 
190         of the cheating was lost.  This has been fixed, by letting the scoreboard
191         routines consult the cheating flag.  Also, the I node number of the
192         savefile is written into the savefile, so cp'ing, etc., will avail the
193         cheater not.  If high security is needed, the game should be run suid.
194         This suid mode has not been made the default because most installations
195         do not want to install it that way.
196
197 35. The sources have been run through lint, and most of lint's complaints have
198         been taken care of.  An attempt was made to adjust the code for 16 bit int
199         machines.  Many casts to long have been put in.  I don't know if it will
200         run on a 16 bitter, but it should be closer to that end.
201
202 36. When larn starts up, if it can't find the scoreboard, it will now make a
203         blank one instead of complaining that there is no scoreboard.  It is not
204         necessary to do "larn -c" to initially create the scoreboard.
205
206 37. When listing out the logfile (larn -l), the error message "error reading
207         from input file" has been fixed.  Also, the date & time of a player's
208         demise is now included in the logfile.
209
210 38. When casting web or sle into a mirror, the game will no longer bash the
211         player.  Instead, the player will either fall asleep or get stuck in his
212         web.
213
214 39. Items like cookies, books, chests, swords of slashing, and Bessmann's
215         flailing hammer can now be sold at the trading post.
216