From fed2c24687664f672504325a20b9ad3b0664322f Mon Sep 17 00:00:00 2001 From: Christopher Haster Date: Tue, 28 Jan 2025 11:58:23 -0600 Subject: [PATCH] Replaced LFS_LITTLER with LFS_NO_LOG This no longer includes LFS_NO_ASSERT, but at least 2 defines is better than 5 right? LFS_NO_LOG => LFS_NO_DEBUG LFS_NO_INFO LFS_NO_WARN LFS_NO_ERROR The benefit is a much more obvious name, and avoids confusion when mixed with other feature aliases (LFS_LITTLER + LFS_BIGGEST?). --- lfs_util.h | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/lfs_util.h b/lfs_util.h index e47d6c6b..6405e35b 100644 --- a/lfs_util.h +++ b/lfs_util.h @@ -24,17 +24,6 @@ // Some convenient macro aliases // TODO move these to something like lfs_cfg.h? -// LFS_LITTLER disables non-essential/debugging utils -// -// note your system may already make these zero cost in release mode -#ifdef LFS_LITTLER -#define LFS_NO_DEBUG -#define LFS_NO_INFO -#define LFS_NO_WARN -#define LFS_NO_ERROR -#define LFS_NO_ASSERT -#endif - // LFS_BIGGEST enables all opt-in features #ifdef LFS_BIGGEST #define LFS_CKPROGS @@ -44,6 +33,14 @@ #define LFS_GC #endif +// LFS_NO_LOG disables all logging macros +#ifdef LFS_NO_LOG +#define LFS_NO_DEBUG +#define LFS_NO_INFO +#define LFS_NO_WARN +#define LFS_NO_ERROR +#endif + // System includes #include