Minor fixes to 1994/ldb
Remove warning from Makefile that this entry uses gets() as this is no longer true. Update README.md pointing out that this will print a newline after the output.
This commit is contained in:
1 parent
2614ae0c9c
commit
15a62a83fc
2 files changed
+4
-2
No files matched your search
@@ -132,7 +132,6 @@ all: data ${TARGET}
|
||||
supernova deep_magic magic charon pluto
|
||||
|
||||
${PROG}: ${PROG}.c
|
||||
@echo "NOTE: this entry uses gets() so you might get a warning compiling and/or running this."
|
||||
${CC} ${CFLAGS} $< -o $@ ${LIBS}
|
||||
|
||||
# alternative executable
|
||||
|
||||
+4
-1
@@ -14,7 +14,10 @@ USA
|
||||
compile and work with modern compilers. The problem was that `srand()` returns
|
||||
void but it was used in a `||` expression. Thus the comma operator was needed.
|
||||
Cody also changed the entry to use `fgets()` instead of `gets()` to make it
|
||||
safe for lines greater than 231 in length. Thank you Cody for your assistance!
|
||||
safe for lines greater than 231 in length. Note that this now prints a newline
|
||||
after the output but this seems like a worthy compromise for making it safer
|
||||
(fixing it is more problematic than it is worth). Thank you Cody for your
|
||||
assistance!
|
||||
|
||||
|
||||
## To run:
|
||||
|
||||
Reference in new issue
Block a user