Use smallest usable type (uint8_t) for some enums

Author: joto

src/expire-config.hpp

@@ -10,9 +10,10 @@
  * For a full list of authors see the git log.
  */
 
+#include <cstdint>
 #include <cstdlib>
 
-enum class expire_mode
+enum class expire_mode : uint8_t
 {
     full_area,     // Expire all tiles covered by polygon.
     boundary_only, // Expire only tiles covered by polygon boundary.

src/flex-table.hpp

@@ -23,6 +23,7 @@
 #include <cassert>
 #include <chrono>
 #include <cstddef>
+#include <cstdint>
 #include <future>
 #include <limits>
 #include <memory>
@@ -35,7 +36,7 @@
  * output. This is not a real primary key, because the values are not
  * necessarily unique.
  */
-enum class flex_table_index_type {
+enum class flex_table_index_type : uint8_t {
     no_index,
     node, // index by node id
     way, // index by way id
@@ -57,7 +58,7 @@ class flex_table_t
      * Table creation type: interim tables are created as UNLOGGED and with
      * autovacuum disabled.
      */
-    enum class table_type {
+    enum class table_type : uint8_t {
         interim,
         permanent
     };

src/logging.hpp

@@ -18,10 +18,11 @@
 #include <fmt/color.h>
 
 #include <atomic>
+#include <cstdint>
 #include <cstdio>
 #include <utility>
 
-enum class log_level
+enum class log_level : uint8_t
 {
     debug = 1,
     info = 2,

src/middle.hpp

@@ -13,6 +13,7 @@
 #include <osmium/memory/buffer.hpp>
 #include <osmium/osm/entity_bits.hpp>
 
+#include <cstdint>
 #include <memory>
 
 #include "osmtypes.hpp"
@@ -171,7 +172,7 @@ class middle_t
     }
 
 #ifndef NDEBUG
-    enum class middle_state
+    enum class middle_state : uint8_t
     {
         constructed,
         started,

src/options.hpp

@@ -21,7 +21,7 @@
 
 class reprojection;
 
-enum class command_t
+enum class command_t : uint8_t
 {
     help,
     version,