From 637489067497662d27ab17f90effba5feb6a7079 Mon Sep 17 00:00:00 2001
From: Cody Boone Ferguson <53008573+xexyl@users.noreply.github.com>
Date: Wed, 29 May 2024 11:36:55 -0700
Subject: [PATCH] Restore bug status to 2001/ollinger
Once upon a time after adding the entry to the bugs file it was changed
so that it should be a bug to be fixed and when verifying things since
it was very quick to fix I added it. It now has been restored to the
original.
The bugs file was updated and the thanks file was updated accordingly
along with the index.html and README.md files.
---
2001/ollinger/README.md | 12 +++++
2001/ollinger/index.html | 4 ++
2001/ollinger/ollinger.c | 1 -
bugs.html | 114 +++++++++++++++++++++------------------
bugs.md | 12 +++++
thanks-for-help.html | 40 +++++++-------
thanks-for-help.md | 39 +++++++-------
7 files changed, 126 insertions(+), 96 deletions(-)
diff --git a/2001/ollinger/README.md b/2001/ollinger/README.md
index 702d3d79e..1e2b74b9c 100644
--- a/2001/ollinger/README.md
+++ b/2001/ollinger/README.md
@@ -5,6 +5,18 @@
```
+### Bugs and (Mis)features:
+
+The current status of this entry is:
+
+```
+ STATUS: INABIAF - please **DO NOT** fix
+```
+
+For more detailed information see [2001/ollinger in bugs.html](../../bugs.html#2001_ollinger).
+
+
+
## To use:
```
diff --git a/2001/ollinger/index.html b/2001/ollinger/index.html
index 015cbdb5b..814c63c8d 100644
--- a/2001/ollinger/index.html
+++ b/2001/ollinger/index.html
@@ -403,6 +403,10 @@ Location: FR - French Republic ( The current status of this entry is: For more detailed information see 2001/ollinger in bugs.html.To build:
+ makeBugs and (Mis)features:
+
+ STATUS: INABIAF - please **DO NOT** fixTo use:
./ollinger integerTry:
diff --git a/2001/ollinger/ollinger.c b/2001/ollinger/ollinger.c
index 29d45a4d0..318773a22 100644
--- a/2001/ollinger/ollinger.c
+++ b/2001/ollinger/ollinger.c
@@ -11,7 +11,6 @@ char e[]="**3
Although it is independent of endianness both systems need the same character set. In other words both have to be ASCII or EBCDIC - not one of each.
+This program will very likely crash or do something unexpected if you do not +provide enough args.
The glob pattern must match the whole string. See the author’s comments for @@ -2129,13 +2137,13 @@ compile under clang. The following patch was applied:
gavin_files: boot.b lilo.conf prim gavin_install.txtThe current (Makefile was modified to try and fit into the current IOCCC build environment.
-See known features in the index.html for things that are not bugs but documented (mis)features.
The author stated that:
@@ -2160,7 +2168,7 @@ things that are not bugs but documented (mis)features.The generated code will very likely segfault or do something not intended if not @@ -2173,7 +2181,7 @@ given the right args. See the index.html file for the correct syntax.
This program sometimes will create unsolvable puzzles :-) just to hook you.
@@ -2187,7 +2195,7 @@ dimensions. Try 100 100 100 for instance and see what happens!
This entry will very likely segfault or do something strange if the source code @@ -2196,7 +2204,7 @@ does not exist.
The author states:
@@ -2230,7 +2238,7 @@ for running itself.Cody Boone Ferguson notes that, though @@ -2244,7 +2252,7 @@ website itself.
The author stated the below points of interest.
@@ -2285,7 +2293,7 @@ with at leastcomputer.tofu input file:
The string specified must be <= 42 characters and may only consist of the @@ -2306,12 +2314,12 @@ but you are welcome to try and fix it.
provide it as an additional alt version. Fixing this is very likely to be very challenging and in some systems it will not be possible to fix but you are welcome to try and fix it if you wish to! -Incorrect formulas will ungracefully crash the program.
The author stated:
@@ -2328,7 +2336,7 @@ welcome to try and fix it if you wish to!Cody Boone Ferguson fixed this program to @@ -2375,7 +2383,7 @@ case-sensitive.
For a great amount of data points the program will crash, depending on your @@ -2398,7 +2406,7 @@ bins at the edges.
instead being something else entirely. The Internet Wayback Machine, although it archived it, did not load scripts. Do you know if the domain was moved? Do you have an archive or mirror? Please provide us it! Thank you. -The author states the following:
Bad input (e.g. nonexistent files, non-numeric number of iterations, etc.) @@ -2411,7 +2419,7 @@ tends to result in empty output.
The author stated that there are a number of features and limitations. As the @@ -2583,7 +2591,7 @@ defined.
The author stated that the program will crash if no argument is passed to the @@ -2597,7 +2605,7 @@ fire :-)
The author stated:
@@ -2606,7 +2614,7 @@ fire :-)The author stated:
@@ -2639,7 +2647,7 @@ fire :-)The author stated:
@@ -2653,7 +2661,7 @@ doesn’t make it possible to lock drawing to the display refresh rate.)The author stated:
@@ -2674,7 +2682,7 @@ doesn’t make it possible to lock drawing to the display refresh rate.)The author stated that the program will crash if no argument is passed to the @@ -2690,7 +2698,7 @@ fire :-)
The author stated:
@@ -2710,7 +2718,7 @@ and endianness conversion would make the source too large for IOCCC rule 2).This program will possibly crash or draw something strange with 0 args. Then @@ -2729,7 +2737,7 @@ used.
From the author:
@@ -2745,7 +2753,7 @@ section.From the author:
@@ -2758,14 +2766,14 @@ such asC2E2.
Invalid input files will very likely crash the program.
This program will not terminate on its own; you must kill hou (but not Qiming
@@ -2773,7 +2781,7 @@ Hou :-) ) yourself. This should not be fixed.
The author reminds us that if you kill the program you will have to wait a short @@ -2790,7 +2798,7 @@ it out as a known limitation it is not a bug but a feature.
The author noted that in macOS the colours might be wrong. They gave a solution. @@ -2814,7 +2822,7 @@ player become bigger, stay away from blocks!
This program will very likely crash if no arg is given.
@@ -2855,7 +2863,7 @@ rely on the entry as the below shows. One should be able to do:The author stated:
@@ -2880,7 +2888,7 @@ result. Error messages become garbled, though.The program doesn’t look at the header of files so if it’s passed something hat @@ -2889,7 +2897,7 @@ is not compressed data it’s likely to crash.
The program assumes that The author wrote: When you run it on a JSON file you will see something like:EOF is -1. This can be fixed but at this time it is
@@ -2922,7 +2930,7 @@ use 8 * sizeof(typ) bits per place. It does not work when CHA
2018/algmyr
STATUS: INABIAF - please DO NOT fix
+STATUS: INABIAF - please DO NOT fix
Source code: 2018/algmyr/prog.c
Information: 2018/algmyr/index.html
2018/hou
STATUS: INABIAF - please DO NOT fix
+STATUS: INABIAF - please DO NOT fix
Source code: 2018/hou/prog.c
Information: 2018/hou/index.html
where [] is the cursor. When this happens if you hit enter (this is necessary
or else it’ll happen again) and then exit again (ctrl-e) and run prog again
it’ll be okay.
The author stated that if you make a typo it can happen that the boot loader can crash and halt. If this is the case type ctrl-e to quit the emulator and try again.
@@ -2981,7 +2989,7 @@ in the case of compiled code it won’t be executable).The author wrote the following:
@@ -3031,7 +3039,7 @@ this program has nothing to do with a hand.Certain input can crash this program. The file @@ -3039,7 +3047,7 @@ this program has nothing to do with a hand.
The author pointed out that some implementations of wc(1) show different
@@ -3047,7 +3055,7 @@ values but his implementation matches that of macOS and FreeBSD.
Cody Boone Ferguson fixed the scripts so @@ -3079,7 +3087,7 @@ ideal if this was not the case.
There are two things to be aware of with this entry.
@@ -3108,7 +3116,7 @@ in the directory:As a backtrace quine this entry is SUPPOSED to segfault so this should not be @@ -3116,7 +3124,7 @@ touched either.
The author stated the following:
@@ -3146,7 +3154,7 @@ touched either.The author wrote that there are a number of differences from what one might @@ -3156,7 +3164,7 @@ remarks in the sections Syntax and
The author wrote that if you decide to change networks or use a different input @@ -3176,7 +3184,7 @@ a crash.
This program will not validate input so it might fail or get stuck if invoked @@ -3185,7 +3193,7 @@ erroneously.
The author noted that if the program runs out of memory it is likely to crash.
@@ -3200,7 +3208,7 @@ ignored except line feeds (preserved) and tabs (expanded to 8 spaces).’This entry is known to crash if no arg is specified. Although easy to fix it is @@ -3211,7 +3219,7 @@ either. But can you figure out why this happens?
The author stated that bad things happen if the entered move is outside of the @@ -3225,7 +3233,7 @@ move.
There are some things that might appear to be bugs but are actually features or @@ -3235,7 +3243,7 @@ things that are misinterpreted as bugs. See the
The author noted that the program only supports WAV files that have @@ -3244,7 +3252,7 @@ audio channels.
The author listed the following limitations:
diff --git a/bugs.md b/bugs.md index 737e77fd4..c6c06c89f 100644 --- a/bugs.md +++ b/bugs.md @@ -2296,6 +2296,18 @@ Although it is independent of endianness both systems need the same character set. In other words both have to be ASCII or EBCDIC - not one of each. +Cody fixed this to compile with clang by adding another function that is allowed to
-have a third arg as an int, not a char **. He chose pain() because it’s a four
+have a third arg as an int, not a char **. He chose pain() because it’s a four
letter word that would match the format and because it’s pain that clang forces
this. :-) This fix makes a point of the author’s notes on portability no longer
valid, BTW.
and for some reason it was being reported as an error. This was not really worth
-a thanks per se but it was originally done by adding int. The better solution
-is to disable the warning which is what was done later on to make it more like
-the original.
and for some reason it was being reported as an error.
+Cody also added the try.sh script.
Yusuke provided a proper command line for those without /dev/audio
or /dev/sound/dsp (which is most everyone nowadays, it seems, and especially
those with macOS) (to do with sound; see his
-/2013/endoh3/index.html entry where he also refers to
+2013/endoh3/index.html entry where he also refers to
sound devices in macOS).
Cody added the try.sh.
e() at the
end of main().
-Cody also added the alt code that adds +
Cody also added the alt code that adds vi(m) movement keys.
The author provided two changes: one to speed it up and one to make it not crash -on losing. Cody provided an alternate version which does the former but he felt -that the idea of crashing on losing (see the index.html for details on why that -might be) too good to get rid of so he kept that in.
-He also points out that there is a way to get the computer to automatically lose +on losing. Cody provided an alternate +version which does the former but +not the latter as he he felt that the idea of crashing on losing (see the +index.html for details on why that might be) too good to get rid. The author +explains how to make this change, however.
+Cody also points out that there is a way to get the computer to automatically lose very quickly. Do you know what it is?
Cody fixed this so that the when compiling the code the program is not executed itself by itself which just showed the usage string and exited. The script -herrmann1.sh is used to compile the program but +herrmann1.sh is how one should compile the program but it’s also how you invoke the program.
Cody provided the file plus1.turing based on the author’s remarks.
@@ -2865,7 +2864,9 @@ shellcheck. In particular there were quite a few:Cody fixed this to work with both 64-bit and 32-bit compilers by changing most
of the ints (all but that in main(int ...)) to longs. He also fixed it to
-compile with clang by changing the args of main to be int and char **, respectively, and changing specific references to theargvarg, casting tolong(wasintbut the 64-bit fix requireslong`) which was its old type.
clang by changing the args of main to be int and char **,
+respectively, and changing specific references to the argv arg, casting to
+long (was int but the 64-bit fix requires long) which was its old type.
Cody also added the try.sh script which might be more useful than any other place as the command to try is quite long with C code.
For some reason the original code was missing (presumingly because it had been
@@ -2874,27 +2875,24 @@ added to .gitignore by accident) but Cody restored it from the arch
Cody improved the Makefile to allow one to more easily set up the port, +
Cody improved the Makefile to allow one to more easily set up the port,
speed and socket(2) call that the author had set up.
Cody also slowed down the ball just a tad (it was already a -D macro that was used
in the code) as it went too fast for the speed at which the paddles move even
when holding down the movement keys (but see below).
Cody also provided an alternate version which lets you use
the arrow keys on your keyboard instead of the more awkward ‘,’ and ‘.’.
He updated both versions to have #ifndef..#endif pairs for the macros so one
+
Cody updated both versions to have #ifndef..#endif pairs for the macros so one
can more easily configure different settings without having to specify all of
them (though this change became unnecessary with an improvement on how it was
-done). The speed, SPEED, will be set to 50 if it’s not defined at the compiler
-line as 50 is what it used to be set to. This way it’s more to the original but
+done). The speed, SPEED, will be set to 50 if it’s not defined at the compiler
+line as 50 is what it used to be set to. This way it’s more to the original but
without having to sacrifice playability by running make.
Cody also fixed it to not crash if not enough args, exiting 1 instead. This was -done at a time where it was said to be a bug that should be fixed, for better or -worse.