From 9fc83efe064cf7900bfa7ec488eb67b3ef197229 Mon Sep 17 00:00:00 2001 From: Rangi42 Date: Fri, 8 Aug 2025 19:00:13 -0400 Subject: [PATCH] Make `rgbasm -Wlarge-constant` enabled by default --- man/rgbasm.1 | 4 +--- src/asm/warning.cpp | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/man/rgbasm.1 b/man/rgbasm.1 index 416f3169..805a2ca6 100644 --- a/man/rgbasm.1 +++ b/man/rgbasm.1 @@ -370,10 +370,8 @@ Warn when is called with an empty string as its second argument (the substring to replace). This warning is enabled by .Fl Wall . -.It Fl Wlarge-constant +.It Fl Wno-large-constant Warn when a constant too large to fit in a signed 32-bit integer is encountered. -This warning is enabled by -.Fl Wall . .It Fl Wmacro-shift Warn when shifting macro arguments past their limits. This warning is enabled by diff --git a/src/asm/warning.cpp b/src/asm/warning.cpp index 01c7c9ab..94cbfb45 100644 --- a/src/asm/warning.cpp +++ b/src/asm/warning.cpp @@ -34,7 +34,7 @@ Diagnostics warnings = { {"empty-data-directive", LEVEL_ALL }, {"empty-macro-arg", LEVEL_EXTRA }, {"empty-strrpl", LEVEL_ALL }, - {"large-constant", LEVEL_ALL }, + {"large-constant", LEVEL_DEFAULT }, {"macro-shift", LEVEL_EXTRA }, {"nested-comment", LEVEL_DEFAULT }, {"obsolete", LEVEL_DEFAULT },