fix Result ctor order warnings

Author: SizzinSeal

include/common/result.hpp

@@ -79,17 +79,17 @@ class Result {
         requires Sentinel<T> && (std::constructible_from<U, Errs> || ...)
                      && (!std::convertible_to<U, T>)
     constexpr Result(U&& error)
-        : value(sentinel_v<T>),
-          error(std::forward<U>(error)) {}
+        : error(std::forward<U>(error)),
+          value(sentinel_v<T>) {}
 
     // Construct a Result with an error, initializing the normal value to its sentinel value.
     // Constraint: type T has a sentinel value
     // Constraint: some type in Errs can be constructed from type U
     template<typename U>
         requires Sentinel<U> && (std::constructible_from<U, Errs> || ...)
     explicit constexpr Result<U>(U&& error)
-        : value(sentinel_v<T>),
-          error(std::forward<U>(error)) {}
+        : error(std::forward<U>(error)),
+          value(sentinel_v<T>) {}
 
     // Get the given error type, if it exists
     // Constraint: some type Errs can be constructed from type U

tests/Result.cpp

@@ -17,7 +17,5 @@ void initialize() {
     static_assert(a.get() == a.get<int>());
     constexpr zest::Result<int, MyError2> b(2);
     static_assert(a == b);
-
-    zest::Result<int, IntError> c(2);
-    zest::Result<int, IntError> d(IntError(2));
+    auto c = a.get<MyError>();
 }