From 77cc50df28fc6374304a29c83e6bbd3b3e522d3a Mon Sep 17 00:00:00 2001 From: SuperDude88 <82904174+SuperDude88@users.noreply.github.com> Date: Sun, 17 May 2026 22:41:09 -0400 Subject: [PATCH] Error for Default Migration - Add error message in the config log for default migration Not a fatal, just a clear signal that you should handle your enum cases explicitly to make sure they're right --- src/dusk/config.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/dusk/config.cpp b/src/dusk/config.cpp index e67365b0d2..fc7a82961f 100644 --- a/src/dusk/config.cpp +++ b/src/dusk/config.cpp @@ -58,6 +58,8 @@ template void ConfigImpl::loadFromJson(ConfigVar& cVar, const json& jsonValue) { if constexpr (std::is_enum_v) { if (jsonValue.is_boolean()) { + DuskConfigLog.error("Doing default migration of CVar {} from bool, enum values may not be what is expected!", cVar.getName()); + using Underlying = std::underlying_type_t; const bool b = jsonValue.get();