Cody fixed this to compile with more recent perl versions; the symbol that’s now
-PL_na was once na. He notes that this entry crashes under macOS but it works
-under Linux after this change.
PL_na was once na.
Cody also added the try.sh script.
Xlib.h and keysym.h.
Cody fixed this for modern compilers. Depending on the compiler it would either
-segfault when run or not compile at all (gcc and clang respectively).
Cody also provided alternate code that supports the southern hemisphere.
+segfault when run or not compile at all (gcc and clang respectively). The
+compiler fix is due to clang being more strict about arg types to main().
+Cody also provided alternate code that +supports the southern hemisphere, based on the author’s remarks.
Cody also provided the try.sh and try.alt.sh scripts that show the Moon phase in artistic ways. It is recommended one try both a number of times in a row on @@ -2604,7 +2605,7 @@ itself a one-line like the code is in the original entry. Now it looks much more like the original entry but with the two fixes.
Finally Cody fixed the Makefile that had the -Wno-foo options in the CDEFINE
variable which although works it is incongruent with the other Makefiles and is
-more confusing (even if not confusing).
Black position and direction: illegalThis was fixed by having the scanf(3) read in a string and then use atoi(3)
-on it to assign to the ints, much like with 1987/lievaart. The strings are
-char[5] and the % specifier is %4s which is enough for the game.
ints, much like with 1987/lievaart.
+The strings are char[5] and the % specifier is %4s which is enough for the
+game.
ints, much like with 1987/lievaart
Cody fixed the code and added an appropriate make rule so that the
SDL version works independent from the curses version
-(using the same code).
-Cody also added alt code to allow one to slow down the code. This code also, at
-least for the curses mode, allows one to quit at any time by pressing ‘q’. See
-below for more details.
+(using the same code). See below for an interesting problem that occurred that
+had to be resolved for SDL1.
+Cody also added alt code to allow
+one to slow down the code to get a better idea of what the program looked like
+back in 2000, with modern systems. This code also, at least for the curses mode, allows
+one to quit at any time by pressing ‘q’. Note that this alternate version only
+will impact the curses and the SDL versions as Cody does not have a DOS system
+to test the other version in.
Due to a terrible design choice of the SDL1 developers something had to be
-changed. As was noted in the log:
+changed. As was noted in the log at the time:
The SDL version did not work for a number of reasons. First of all the
code requires that SDL is defined. Second the path[sic] wrong header file was
included. Third the SDL1 developers thought it would be a great idea
@@ -2663,16 +2669,11 @@ changed. As was noted in the log:
make: *** [thadgavin_sdl] Error 1
Thus main() was changed to 'int main(int argc, char **argv)'.
-Cody also added an alternate version to help see what is going on in more modern
-systems and in case you’re sensitive to rapidly moving swirling. See the
-index.html for details on that. Note that this alternate version only will impact
-the curses and the SDL versions as Cody does not have a DOS system to test the
-other version in.
-Cody added the alt code based on the
+
Cody added the alt code based on the
author’s remarks with a fix for modern systems and he also added the two
scripts, try.sh and try.alt.sh for
the main code and the alt code respectively.
@@ -2689,7 +2690,7 @@ details) the source code is now executable by default.
segfaulted and once that was fixed only the binary was modified; it was not run
but according to the author’s remarks it should be executed). He managed to do
this with Linux but it will not work with any system that does not allow one to
-compile 32-bit ELF binaries. See bugs.html for
+compile 32-bit ELF binaries. See 2001/anonymous in bugs.html for
why this is; this is not a bug, it’s a feature inherent in what it does!
The following had to be done to fix this:
@@ -2699,7 +2700,8 @@ this prevented main() from entering an infinite recursive loop.
Without these it would crash and prevent modification of the 32-bit
ELF (not elf :-) )
-binary. But again see bugs.html here.
+binary. But again see 2001/anonymous in bugs.html
+for this.
These also had to be done:
- when
open()ing the file the file descriptor had to be saved so it could be
@@ -2734,11 +2736,12 @@ wish to see, try the following commands from the 2001/anonymous dir
git diff d2a42f42e8f477f29e9d5ed09ce2bb349eaf7397..4bc03de321612869aebf855850c6500df95cb6ef anonymous.c
or to not use git:
make diff_orig_prog
-Cody also added a program
-like anonymous.ten.c
-Ten Green Bottles but which
-sings Ten in the Bed instead.
-As well he added the try.sh so that one can
+
Cody also added a program like
+anonymous.ten.c which ‘sings’ the
+lyrics to Ten Green Bottles
+but which sings Ten in the Bed
+instead.
+As well he added the try.sh script so that one can
attempt to use the program as it was designed but if compiling as 32-bit fails
it will at least run the supplementary program as a 64-bit program directly.
diff --git a/thanks-for-help.md b/thanks-for-help.md
index 6e3df04ae..40e7b8b5f 100644
--- a/thanks-for-help.md
+++ b/thanks-for-help.md
@@ -3103,8 +3103,7 @@ He also added the [try.sh](%%REPO_URL%%/2000/dhyang/try.sh) script.
[Cody](#cody) fixed this to compile with more recent perl versions; the symbol that's now
-`PL_na` was once `na`. He notes that this entry crashes under macOS but it works
-under Linux after this change.
+`PL_na` was once `na`.
Cody also added the [try.sh](%%REPO_URL%%/2000/dlowe/try.sh) script.
@@ -3124,9 +3123,11 @@ includes of `Xlib.h` and `keysym.h`.