From 60ba3056abda5a5ad7f9ce9d683ec48a61a261e6 Mon Sep 17 00:00:00 2001 From: Sadie del Solar Date: Mon, 29 Apr 2024 12:40:07 -0700 Subject: [PATCH] Codechange: Compiler check for _fios_colours/DetailedFileType --- src/fileio_type.h | 2 ++ src/fios_gui.cpp | 3 ++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/fileio_type.h b/src/fileio_type.h index c871c1c14b..a353e12b8a 100644 --- a/src/fileio_type.h +++ b/src/fileio_type.h @@ -40,6 +40,8 @@ enum DetailedFileType { DFT_FIOS_DIR, ///< A directory entry. DFT_FIOS_DIRECT, ///< Direct filename. + DFT_END, ///< End of this enum. Supports a compile time size check against _fios_colours in fios_gui.cpp + DFT_INVALID = 255, ///< Unknown or invalid file. }; diff --git a/src/fios_gui.cpp b/src/fios_gui.cpp index 2db7745c65..dda777f643 100644 --- a/src/fios_gui.cpp +++ b/src/fios_gui.cpp @@ -238,7 +238,8 @@ static const TextColour _fios_colours[] = { TC_DARK_GREEN, // DFT_FIOS_DIR TC_ORANGE, // DFT_FIOS_DIRECT }; - +/* This should align with the DetailedFileType enum defined in fileio_type.h */ +static_assert(std::size(_fios_colours) == DFT_END); /** * Sort the collected list save games prior to displaying it in the save/load gui.