Casting Uint to uintmax_t now throws range_error not overflow_error

This is for thinking ahead when we will have signed types and we want to have a uniform excpetion API
Underflow exception is used for operations between Uints, range_error will be used when converting to uintmax_t or intmax_t and the value is too big or too small for the result type

Author: saxbophone

CMakeLists.txt

@@ -9,7 +9,7 @@ else()
     set(ARBY_SUBPROJECT ON)
 endif()
 
-project(arby VERSION 0.1.0 LANGUAGES CXX)
+project(arby VERSION 0.2.0 LANGUAGES CXX)
 
 find_program(CCACHE_PROGRAM ccache)
 if(CCACHE_PROGRAM)

arby/include/arby/arby.hpp

@@ -181,13 +181,13 @@ namespace com::saxbophone::arby {
         Uint(std::string digits);
         /**
          * @returns Value of this Uint object cast to uintmax_t
-         * @throws std::overflow_error when Uint value is out of range for
+         * @throws std::range_error when Uint value is out of range for
          * uintmax_t
          */
         explicit constexprvector operator uintmax_t() const {
             // prevent overflow of uintmax_t
             if (*this > std::numeric_limits<uintmax_t>::max()) {
-                throw std::overflow_error("value too large for uintmax_t");
+                throw std::range_error("value too large for uintmax_t");
             }
             uintmax_t accumulator = 0;
             uintmax_t current_radix = 1;

tests/casting.cpp

@@ -13,8 +13,8 @@ TEST_CASE("Casting arby::Uint to uintmax_t", "[casting]") {
     CHECK((uintmax_t)arby::Uint(value) == value);
 }
 
-TEST_CASE("Casting arby::Uint with value higher than UINT_MAX to uintmax_t throws overflow_error", "[casting]") {
+TEST_CASE("Casting arby::Uint with value higher than UINT_MAX to uintmax_t throws range_error", "[casting]") {
     arby::Uint value = arby::Uint(std::numeric_limits<uintmax_t>::max()) + 1;
 
-    CHECK_THROWS_AS((uintmax_t)value, std::overflow_error);
+    CHECK_THROWS_AS((uintmax_t)value, std::range_error);
 }