From 7a96697c4c991b169644e7b1c7de1f04923d2468 Mon Sep 17 00:00:00 2001 From: Cody Boone Ferguson <53008573+xexyl@users.noreply.github.com> Date: Fri, 7 Jul 2023 10:47:11 -0700 Subject: [PATCH] Fix 1990/theorem/Makefile for clang in linux Disabled a warning that defaulted to an error. In this case it was easier to do it this way due to the way the entry works. --- 1990/theorem/Makefile | 2 +- thanks-for-fixes.md | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/1990/theorem/Makefile b/1990/theorem/Makefile index bc35188b9..0afb8d29f 100644 --- a/1990/theorem/Makefile +++ b/1990/theorem/Makefile @@ -41,7 +41,7 @@ include ../../var.mk CSILENCE= -Wno-deprecated-declarations -Wno-empty-body -Wno-error -Wno-unused-value \ -Wno-implicit-function-declaration -Wno-incompatible-pointer-types \ -Wno-macro-redefined -Wno-parentheses -Wno-return-type -Wno-unused-parameter \ - -Wno-empty-translation-unit + -Wno-empty-translation-unit -Wno-int-conversion # Common C compiler warning flags # diff --git a/thanks-for-fixes.md b/thanks-for-fixes.md index d65c9a16d..1949c0779 100644 --- a/thanks-for-fixes.md +++ b/thanks-for-fixes.md @@ -424,6 +424,10 @@ and to prevent a warning about `gets()` at linking or runtime. Since this program is so incredible the extra fixes were deemed worth having and this is why it was done. +Cody disabled a warning in the Makefile that proved to be a problem only with +clang in linux but which was defaulting to an error. This way was the simplest +way to deal with the problem in question due to the way the entry works. + Yusuke pointed out that `atof` nowadays needs `#include ` which was used in order to get this to work initially (prior to this output was there but incomplete).