Changed to using an enum for the compression level names

src/output_options.c

@@ -34,6 +34,10 @@
 #include "part_type.h"
 #include "swift.h"
 
+/* Compression level names. */
+const char* compression_level_names[compression_level_count] = {
+    "off", "on", "low", "med", "high"};
+
 /**
  * @brief Initialise the output options struct with the information read
  *        from file. Only rank 0 reads from file; this data is then broadcast
@@ -137,10 +141,12 @@ int output_options_should_write_field(struct output_options* output_options,
           part_type_names[part_type]);
 
   char compression_level[PARSER_MAX_LINE_SIZE];
-  parser_get_opt_param_string(output_options->select_output, field,
-                              compression_level, compression_level_default);
+  parser_get_opt_param_string(
+      output_options->select_output, field, compression_level,
+      compression_level_names[compression_level_default]);
 
-  int should_write = strcmp(compression_do_not_write, compression_level);
+  int should_write = strcmp(compression_level_names[compression_do_not_write],
+                            compression_level);
 
 #ifdef SWIFT_DEBUG_CHECKS
   message("Determining if %s is to be written. Returning %d from %s.", field,

src/output_options.h

@@ -23,15 +23,25 @@
 #include "part_type.h"
 
 /* Compression level names */
-#define compression_do_not_write "off"
-#define compression_write_lossless "on"
-#define compression_write_low_lossy "low"
-#define compression_write_med_lossy "med"
-#define compression_write_high_lossy "high"
+enum compression_levels {
+  compression_do_not_write = 0,
+  compression_write_lossless,
+  compression_write_low_lossy,
+  compression_write_med_lossy,
+  compression_write_high_lossy,
+  /* Counter, always leave last */
+  compression_level_count,
+};
 
 /* Default value for SelectOutput */
 #define compression_level_default compression_write_lossless
 
+/**
+ * @brief Names of the compression levels, used in the select_output.yml
+ *        parameter file.
+ **/
+extern const char* compression_level_names[];
+
 /**
  * @brief Output selection properties, including the parsed files.
  **/