Merge pull request #1088 from xexyl/1991-fine-chars

1991/fine/Makefile

@@ -39,7 +39,7 @@ include ../../var.mk
 # Common C compiler warnings to silence
 #
 CSILENCE= -Wno-bitwise-op-parentheses -Wno-error -Wno-implicit-function-declaration \
-	-Wno-int-conversion -Wno-pointer-to-int-cast
+	-Wno-int-conversion -Wno-pointer-to-int-cast -Wno-implicit-int
 
 # Common C compiler warning flags
 #
@@ -58,7 +58,7 @@ ARCH=
 
 # Defines that are needed to compile
 #
-CDEFINE=
+CDEFINE= -D'main(a,b)'='int main(a,b)char**b;'
 
 # Include files that are needed to compile
 #

1991/fine/README.md

@@ -19,18 +19,10 @@ echo text | ./fine
 ## Try:
 
 ```sh
-echo "green terra
-vex
-tang
-vend onyx
-cheryl be flashy
-rail
-clerk
-the rug gary lenT
-" | ./fine
+./demo.sh
 ```
 
-Read the last line backwards! Credit goes to [Merlyn LeRoy (Brian
+Make sure that you read the last line backwards! Credit goes to [Merlyn LeRoy (Brian
 Westley)](/winners.html#Brian_Westley) for that. See his 22 August 1992
 [rec.puzzles
 post](https://groups.google.com/g/rec.puzzles/c/z_xVPMRVBtg/m/lyEYSAeBD4gJ). To
@@ -43,13 +35,17 @@ rotated from `the rug gary lent`!
 This filter, 80 chars plus a newline, fits into a single line on most 
 terminals (unless your terminal has a line wrap mis-feature :-)).
 
+Note in 2023: fixing this entry to work with modern systems increased the size,
+originally to 106 characters but dropped down to 85 with clever use of the
+Makefile and removal of a cast that was not strictly necessary.
+
 This entry is likely one of the smallest C implementations of this
 filter, excluding programs that resort to command line or include 
 file tricks.
 
 How does this program work?  Which 3 bytes of C code can be changed
 into 2 bytes, allowing the program to still work, but also stripping 
-the high bit off of some input?
+the high bit off some input?
 
 ## Author's remarks:
 

1991/fine/demo.sh

@@ -0,0 +1,23 @@
+#!/usr/bin/env bash
+
+make all || exit 1
+
+echo -n "Green terra: "
+echo "Green terra" |./fine
+
+echo -n "Vex: "
+echo "Vex" | ./fine
+
+echo -n "Tang: "
+echo "Tang" | ./fine
+
+echo -n "Vend onyx: "
+echo "Vend onyx" | ./fine
+echo -n "Cheryl be flashy: "
+echo "Cheryl be flashy" | ./fine
+echo -n "Rail: "
+echo "Rail" | ./fine
+echo -n "Clerk: "
+echo "Clerk" | ./fine
+echo -n "The rug gary lenT: "
+echo "The rug gary lenT" | ./fine

1991/fine/fine.c

@@ -1 +1 @@
-int main(int a, char **b){while((a=getchar())+1)putchar((b=64^a&223)&&(int)b<27?a&96|((int)b+12)%26+1:a);}
+main(a,b){while((a=getchar())+1)putchar((b=64^a&223)&&b<27?a&96|((int)b+12)%26+1:a);}

thanks-for-fixes.md

@@ -650,6 +650,18 @@ and he also made an [alternate version](1991/dds/dds.alt.c) that works with
 `clang`. The alternate code, described in the README.md file, is what is needed
 for clang.  Reading it might be instructive even if you have gcc.
 
+## [1991/fine](1991/fine/fine.c) ([README.md](1991/fine/README.md))
+
+Cody made it look much more like the original entry even after the fix that
+increased the count in characters from 80 to 106, getting it back down to just 85.
+This was done by redefining `main` at the compiler line so that it looks like
+the original where one didn't have to worry about the type of args of main() and
+there were other fewer restrictions and also by removing a cast that was
+not strictly necessary (this does create a new warning: `ordered comparison
+between pointer and integer ('char **' and 'int')` but it works just fine).
+
+Cody also added the [demo.sh](1991/fine/demo.sh) script which feeds the program
+some fun input for fun but mostly different output.
 
 ## [1991/westley](1991/westley/westley.c) ([README.md](1991/westley/README.md]))