Minor improvements

include/data.hpp

@@ -10,6 +10,7 @@
 #include <fstream>
 #include <string>
 #include <vector>
+#include <atomic>
 
 #include "cpp_utils/assert.hpp"
 
@@ -333,7 +334,7 @@ struct data_handler {
 
     const char* module;
     const char* path;
-    volatile bool changed = false;
+    std::atomic<bool> changed = false;
     mutable server_lock lock;
     std::vector<T> data_;
 };

include/money.hpp

@@ -36,7 +36,7 @@ struct money {
     }
 
     static money from_double(double dollars) {
-        const long dollars_long = static_cast<long>(dollars);
+        const auto dollars_long = static_cast<long>(dollars);
         return {dollars_long, static_cast<int>((dollars - dollars_long) * SCALE)};
     }
 

include/views.hpp

@@ -384,7 +384,6 @@ bool range_contains(const R& r, const std::ranges::range_value_t<R> & value) {
 #endif
 }
 
-
 // TODO(C+23) In the future, we can simply ranges::to<std::vector> but it is not yet implemented with GCC
 
 template <std::ranges::range R>
@@ -406,7 +405,7 @@ auto to_vector(R&& r) {
 template <std::ranges::range R>
 auto min_with_default(R&& r, std::ranges::range_value_t<R> def) {
     if (r) {
-        return std::ranges::min(r);
+        return std::ranges::min(std::forward<R>(r));
     }
 
     return def;