home *** CD-ROM | disk | FTP | other *** search
/ Amiga Format CD 27 / amigaformatcd27.iso / -websites- / amidoom / adoom-0.7.lha / ADoom / amiga_notes.txt < prev    next >
Text File  |  1998-01-16  |  18KB  |  381 lines
  1. ========================================================================
  2. v0.7 15 Jan 98
  3. ------------------------------------------------------------------------
  4. Version 0.6 had a serious bug in c2p8_040_amlaukka.s which caused
  5. crashes on 68040+AGA and 68060+AGA machines.  Fixed with patch
  6. supplied by Aki Laukkanen.
  7.  
  8. ------------------------------------------------------------------------
  9. Version 0.6 had a serious bug in the use_mmu cleanup code which caused
  10. crashes one exit when -mmu or MMU was used.  Fixed with patch supplied
  11. by Aki Laukkanen.
  12.  
  13. ------------------------------------------------------------------------
  14. Commenting out the Macro68 debug statements in the *.s files made the
  15. executable much smaller.
  16.  
  17. ========================================================================
  18. v0.6 15 Jan 98
  19. ------------------------------------------------------------------------
  20. Applied patches to amiga_music.s as supplied by Joe Fenton.
  21. Also added code to set correct frequency for PAL or NTSC instead of
  22. assuming NTSC.
  23.  
  24. ------------------------------------------------------------------------
  25. Applied patches to column and span drawing routines in amiga_draw.s as
  26. supplied by Aki Laukkanen.  This fixed all the bright dots scattered
  27. around floors and ceilings, and the red stairs at the start.  Disabled
  28. various routines in r_draw.c, r_segs.c and r_things.c that are
  29. replaced by routines in amiga_draw.s.
  30.  
  31. ------------------------------------------------------------------------
  32. Now use FPU for FixedMul() on 68060.  Joe Fenton says "2 FP multiplies
  33. beat the hell out of 4 word multiplies, plus the associated adds and
  34. swaps".  Added SetFPMode() routine to amiga_fixed.s to set FPU
  35. rounding mode to "round towards minus infinity".  Otherwise WAD demos
  36. got out of step and went wrong when using FPU for multiply.
  37.  
  38. ------------------------------------------------------------------------
  39. In p_map.c, undid the patch I made in version 0.0 by changing line:
  40.  
  41.     if (ld->special && numspechit < MAXSPECIALCROSS)
  42.  
  43. back to:
  44.  
  45.     if (ld->special)
  46.  
  47. Instead added code in p_map.c and p_enemy.c to check bounds of all
  48. spechit[] array accesses.  With MAXSPECIALCROSS == 8, got lots of
  49. array out of bounds accesses, especially in 2nd level of DOOM II.
  50. Changed MAXSPECIALCROSS from 8 to 20.  That got rid of them.
  51.  
  52. ------------------------------------------------------------------------
  53. After several hours of stepping through with CPR, finally found and
  54. fixed the bug that made the Doom II "Dead Simple" level impossible to
  55. complete.  For the function pointer comparison in the line:
  56.  
  57.     if (th->function.acp1 != (actionf_p1)P_MobjThinker)
  58.  
  59. in p_enemy.c, SAS/C generated code which compared acp1 with the
  60. address of the ALV (JMP instruction) for P_MobjThinker() instead of
  61. the address of the P_MobjThinker() function itself.  Thus the test
  62. always failed, even when the function pointers were the same.  Fixed
  63. by compiling everything with ABSFUNCPOINTER.
  64.  
  65. I reported this to Steve Krueger at SAS.  He replied:
  66.  
  67. "It's not a bug, but rather a limitation of the CODE=NEAR model. We
  68. even document this in the manual, and provide an option to work around
  69. it if you still want to use CODE=NEAR. Check the manual for
  70. information on the ABSFUNCTIONPOINTER option.
  71.  
  72. ------------------------------------------------------------------------
  73. Found and fixed the crash on exit bug when music is enabled.  Register
  74. a0 wasn't initialised to _custom in FreeChannels() in amiga_music.s.
  75.  
  76. ------------------------------------------------------------------------
  77. Added -nosfx option and tooltype for disabling sound effects and
  78. leaving audio channels free.
  79.  
  80. ------------------------------------------------------------------------
  81. Added Aki Laukkanen's code to amiga_video.c for marking screens[0] and
  82. the video raster as "imprecise" with the MMU when the -mmu option is
  83. specified.  This should create a slight speedup for 68060.
  84.  
  85. ------------------------------------------------------------------------
  86. Started adding code for -directcgx option for rendering directly to
  87. framebuffer of any video card running CyberGraphX.  Discovered it
  88. flickers badly and needs double or triple-buffering.  Started adding
  89. code for double-buffering, but didn't get logic for interaction with
  90. I_ReadScreen() right.  Still haven't got LockBitMapTags() and
  91. UnLockBitMap() called from the right places, so sometimes tries to
  92. write to framebuffer (screens[0]) while not locked.  This may corrupt
  93. other screens if you flip or drag screens while running with
  94. -directcgx.
  95.  
  96. ------------------------------------------------------------------------
  97. Applied patches from Aki Laukkanen to get low detail mode working.
  98. This included substantial changes to amiga_draw.s and r_draw.c.  The
  99. assembly routines Aki supplied for R_DrawFuzzColumn(),
  100. R_DrawFuzzColumnLow(), R_DrawTranslatedColumn() and
  101. R_DrawTranslatedColumnLow() didn't appear to work right, so reverted
  102. to C versions in r_draw.c.
  103.  
  104. ========================================================================
  105. v0.5  8 Jan 98
  106. ------------------------------------------------------------------------
  107. Added music code in amiga_music.s based on Joseph Fenton .MUS player.
  108. Changed amiga_sound.s to use only 2 channels for sound effects when
  109. music is enabled, and 4 channels when music is disabled.  Added
  110. MIDI_Instruments and MIDI_Instruments.s to archive.  MIDI_Instruments.s
  111. isn't actually used, but it's helpful for understanding the instrument
  112. format.
  113.  
  114. ------------------------------------------------------------------------
  115. Added Aki Laukkanen's latest 68040 and 68060-optimised column and span
  116. renderers in amiga_draw.s.  Modified r_main.c and r_draw.h to handle
  117. different DrawColumn_040(), DrawColumn_060(), DrawSpan_040() and
  118. DrawSpan_060().  Aki sent me some more fast C2P and MMU routines but I
  119. haven't got those working yet.
  120.  
  121. ------------------------------------------------------------------------
  122. The call to BestModeID() in amiga_video.c had an unterminated taglist.
  123. Fixed.
  124.  
  125. ------------------------------------------------------------------------
  126. Added Aki's 68060 versions of FixedMul() and FixedDiv() in amiga_fixed.s.
  127. 68060 FixedDiv uses FPU.  If there is no FPU, fall back to 68040 version.
  128. Added code to select appropriate routines in amiga_main.c.
  129. Changed FixedMul() definitions in m_fixed.h to look like this:
  130. extern fixed_t (*FixedMul)    (fixed_t a, fixed_t b);
  131. extern fixed_t (*FixedDiv)    (fixed_t a, fixed_t b);
  132. fixed_t FixedMul_040    (fixed_t a, fixed_t b);
  133. fixed_t FixedMul_060    (fixed_t a, fixed_t b);
  134. fixed_t FixedDiv_040    (fixed_t a, fixed_t b);
  135. fixed_t FixedDiv_060fpu    (fixed_t a, fixed_t b);
  136. /* fixed_t FixedDiv2    (fixed_t a, fixed_t b); */
  137.  
  138. ------------------------------------------------------------------------
  139. Added code to amiga_video.c to check version numbers before trying to
  140. call BestModeID() and put up a screenmode requester.  Explicitly
  141. declared LowLevelBase so it doesn't try and autoopen lowlevel.library.
  142.  
  143. ------------------------------------------------------------------------
  144. Fixed I_WaitVBL() in amiga_video.c to call WaitTOF().  It's only
  145. called on exit to delay long enough to hear the final sound effect.
  146.  
  147. ========================================================================
  148. v0.4
  149. ------------------------------------------------------------------------
  150. Gamma correction wasn't working in ECS mode.  Fixed in I_Palette()
  151. in amiga_video.c.
  152.  
  153. ------------------------------------------------------------------------
  154. I think I finally found the cause of the 80000003 crash on exit.  It
  155. was caused by 2 missing commas in the initialisation strings for
  156. endmsg[] in dstrings.c following the strings: "go ahead and leave. see
  157. if i care." and "you're lucky i don't smack\nyou for thinking about
  158. leaving.".  Fixed.
  159.  
  160. ------------------------------------------------------------------------
  161. Added 68020/30 optimised C2P routine in c2p_020.s.  Use blitter for
  162. final passes which means double-buffering is required to eliminate
  163. otherwise severe flicker.  Implemented double-buffering for all planar
  164. modes in amiga_video.c.
  165.  
  166. ------------------------------------------------------------------------
  167. Found that the last QBlit() pass finished while the main program was
  168. in the middle of 3D rendering.  This meant the ChangeScreenBuffer()
  169. call was delayed until the end of rendering.  The program felt very
  170. sluggish, even though the frame-rate was higher than before because
  171. there was a much longer delay between pressing a key and seeing the
  172. result happen on the screen.  To solve this I created a new subtask
  173. which simply calls ChangeScreenBuffer() in a loop when signalled from
  174. the QBlit() cleanup function.  This way the main engine is interrupted
  175. to call ChangeScreenBuffer() at the earliest opportunity.  Extensive
  176. changes are in amiga_video.c.
  177.  
  178. ------------------------------------------------------------------------
  179. Gabry (ggreco@iol.it) emailed me some CD32 joypad handling code.  I
  180. included it in amiga_video.c (disabled by default) but I can't test
  181. it.  To enable it, use the -joypad option or JOYPAD tooltype.
  182.  
  183. ------------------------------------------------------------------------
  184. Several people sent me faster replacements for R_DrawColumn() and
  185. R_DrawSpan() in amiga_draw.s, but so far only one of them worked
  186. right.
  187.  
  188. ========================================================================
  189. v0.3
  190. ------------------------------------------------------------------------
  191. Low detail code doesn't work, so disabled detail level toggle in
  192. m_menu.c.  Previously setting low detail and then resizing screen
  193. corrupted graphics.
  194.  
  195. ------------------------------------------------------------------------
  196. Added ClearPointer() call before CloseWindow() in amiga_video.c.
  197. Let's see if that fixes the crash on exit.  Hmm, seems OK after about
  198. a dozen runs.  Perhaps it was a bug in CyberGraphX.  ... nope, just
  199. got another 80000003 crash on exit  :(
  200.  
  201. ------------------------------------------------------------------------
  202. Added code to get ProgramName in amiga_main.c.  Added definition of
  203. ProgramName in m_args.h and m_args.c.
  204.  
  205. ------------------------------------------------------------------------
  206. Allocated space for myargv[] in amiga_main.c and copy argv[] across.
  207. Parse icon tooltypes and convert to additional entries in myargv[]
  208. and myargc.
  209.  
  210. ------------------------------------------------------------------------
  211. Added -screenmode option to set ScreenMode in amiga_video.c
  212.  
  213. ------------------------------------------------------------------------
  214. Added -forcedemo option in g_game.c to force demos to play even if
  215. the version number doesn't match.
  216.  
  217. ------------------------------------------------------------------------
  218. Changed handling of $HOME in d_main.c to append a '/' if it doesn't
  219. already end in a '/' or a ':'.
  220.  
  221. ------------------------------------------------------------------------
  222. Added trailing slashes to definitions of DEVMAPS and DEVDATA in
  223. dstrings.h.
  224. #define DEVMAPS "devmaps/"
  225. #define DEVDATA "devdata/"
  226.  
  227. ========================================================================
  228. v0.2   1 Jan 1998
  229. ------------------------------------------------------------------------
  230. Changed zone management code to allocate between 2Mb and 6Mb instead
  231. of always 6Mb.  ADoom should now work on Amigas with less memory.
  232. Check result of allocation.  Previous version crashed if 6Mb malloc
  233. failed.
  234.  
  235. ------------------------------------------------------------------------
  236. Changed handling of HOME environment variable in d_main.c
  237.  
  238. ------------------------------------------------------------------------
  239. The Amiga doesn't have F11 and F12 keys.  Changed amiga_video.c to map
  240. the '[' and ']' keys to F11 and F12.  Now gamma correction works.
  241. Also fixed keypad '+'.
  242.  
  243. ------------------------------------------------------------------------
  244. Added joystick code using gameport device in amiga_video.c.  Changed
  245. m_misc.c to enable usejoystick.
  246.  
  247. ------------------------------------------------------------------------
  248. Added AmiTCP networking support by writing a new version of
  249. amiga_net.c based on i_net.c for Linux.  Successfully worked to a PC
  250. running Linux with DOOM compiled from the released DOOM sources.
  251. Performance was really bad, with lots of jerking, even on ethernet.
  252. Tried improving performance by making sends non-blocking as well as
  253. recvs.  AmiTCP 4.3 appears to return errno==EINVAL instead of
  254. errno=EWOULDBLOCK for non-blocking recvfrom() and non-blocking
  255. sendto() --- seems pretty weird to me, or have I done something wrong?
  256.  
  257. ========================================================================
  258. v0.1  30 Dec 97
  259. ------------------------------------------------------------------------
  260. Open audio.device on 4 channels and play sound effects in
  261. amiga_sound.c.  Cache up to 12 sound effects in chipmem at a time.
  262. All other sound effects are cached in fastmem.  Audio period is still
  263. not calculated correctly from pitch.  Separation (balance) is ignored,
  264. so 3D sound effects are not right.  Changed s_sound.c to pass channel
  265. number (cnum) to I_StartSound().  Changed m_misc.c to set 4 channels
  266. instead of 3.
  267.  
  268. ------------------------------------------------------------------------
  269. Deleted calls to I_UpdateSound() and I_SubmitSound() in d_main.c
  270.  
  271. ------------------------------------------------------------------------
  272. Stripped 68040-optimised AGA and ECS(EHB) C2P routines out of Flick
  273. v1.5 and linked them into amiga_video.c.  Changed "dirty_list"
  274. mechanism to a comparison buffer instead.  Added amiga_median.c for
  275. calculating best EHB palette and 8-bit -> 6-bit colour xlate table.
  276. Changed EHB C2P routine to handle colour translate table (probably not
  277. entirely efficiently).  These routines are not quite state-of-the-art
  278. (e.g, old merge macro) but they are not bad either.  Linked in 68020-
  279. and Akiko-optimised routines too, but they are not called yet.
  280.  
  281. ------------------------------------------------------------------------
  282. Made a minor optimisation speedup to R_DrawColumn().
  283.  
  284. ========================================================================
  285. v0.0  28 Dec 97
  286. ------------------------------------------------------------------------
  287. Wrote amiga_main.c, amiga_system, amiga_sound.c, amiga_video.c,
  288. amiga_net.c and smakefile to replace i_main.c, i_system, i_sound.c,
  289. i_video.c, i_net.c and Makefile respectively.  The amiga_sound and
  290. amiga_net routines are dummies.
  291.  
  292. ------------------------------------------------------------------------
  293. Amiga is big-endian.  Compiled with DEFINE=__BIG_ENDIAN__.  It looks
  294. like the code hasn't been tested with __BIG_ENDIAN__ for a while.  The
  295. test in m_swap.c for whether swap routines should be included was
  296. around the wrong way.  SwapSHORT() returned an int instead of an
  297. unsigned short.
  298.  
  299. ------------------------------------------------------------------------
  300. Fixed point arithmetic routines in m_fixed.c used (long long) 64-bit
  301. datatype not available in SAS/C.  Replaced with assembly routines in
  302. amiga_fixed.s.  Added check for divide by 0 in FixedDiv() because
  303. Amiga crashes on divide by 0.
  304.  
  305. ------------------------------------------------------------------------
  306. The Unix function alloca() was called from r_data.c, w_wad.c and some
  307. other files.  I replaced with calls to malloc() and added calls to
  308. free() where each routine returns.
  309.  
  310. ------------------------------------------------------------------------
  311. Changed the cache routines in w_wad.c to use the more efficient
  312. fopen(), fread(), fseek(), fclose() instead of open(), read(),
  313. lseek(), close().  Changed handles from (int) to (FILE *) everywhere
  314. including lumpinfo_t structure in w_wad.h.  Fixed the filelength()
  315. function to align stat structure on 4-byte boundary.  Used SAS/C
  316. fileno() function to get file number from (FILE *).
  317.  
  318. ------------------------------------------------------------------------
  319. In d_main.c, changed the handling of slashes and double-dots in
  320. filenames to Amiga conventions.
  321.  
  322. ------------------------------------------------------------------------
  323. Moved definition of RANGECHECK from doomdef.h to smakefile.  Disabled
  324. SNDSERV and SNDINTR in doomdef.h.
  325.  
  326. ------------------------------------------------------------------------
  327. SAS/C 6.58 appeared to generate bad code for the line:
  328.  
  329.     offsetangle = abs(rw_normalangle-rw_angle1);
  330.  
  331. in r_segs.c, resulting in corrupted graphics.  This took me hours to
  332. track down.  Putting printf() instructions before and after, the code
  333. calulated abs(1073741824-1927746431)=1073741824 instead of the correct
  334. answer of 854004607.  As far as I could see, it was using the stdlib.h
  335. definition of abs(), which looks fine.  As a workaround, I changed
  336. abs() to iabs() (SAS/C function) throughout in all sources.
  337.  
  338. On 5 Jan 98, Michael van Elst <mlelstv@serpens.swb.de> offered the
  339. following explanation:
  340.  
  341. >It doesn't. It is an artefact of ANSI C. rw_normalangle is defined
  342. >as unsigned, rw_angle1 is defined as int. The result of the difference
  343. >is therefore unsigned. You need something like:
  344. >
  345. >    offsetangle = abs((int)rw_normalangle - (int)rw_angle1);
  346. >or  offsetangle = abs((long)rw_normalangle - (long)rw_angle1);
  347. >
  348. >and make sure that rw_normalangle fits into the positive int or
  349. >long values.
  350. >[...]
  351. >gcc probably doesn't use the macro but an abs(int) function. You
  352. >get the same with SAS/C if you #undef abs.
  353.  
  354. ------------------------------------------------------------------------
  355. Got READ enforcer hits in p_enemy.c at the line:
  356.  
  357.         ld = spechit[numspechit];
  358.  
  359. Crashed badly shortly afterwards if Enforcer was not running.  As far
  360. as I can tell, the variable numspechit should be in the range 0..7.
  361. Somehow numspechit gets the value 149551680 (0x8e9fa40).  Without
  362. really knowing what I was doing, I attempted to fix the problem by
  363. changing the line:
  364.  
  365.     if (ld->special)
  366.  
  367. in p_map.c to:
  368.  
  369.     if (ld->special && numspechit < MAXSPECIALCROSS)
  370.  
  371. So far it seems to work.
  372.  
  373. ------------------------------------------------------------------------
  374. Disassembled R_DrawColumn() and R_DrawSpan() with OMD, created
  375. amiga_draw.s and hand-optimised a bit.
  376.  
  377. ------------------------------------------------------------------------
  378. Program still often crashes on exit   :(
  379.  
  380. ------------------------------------------------------------------------
  381.