Updated error messages [skip ci]

ankane · ankane · commit 2659e4512a03 · 2026-03-06T22:18:22.000-08:00
diff --git a/include/pgvector/sparsevec.hpp b/include/pgvector/sparsevec.hpp
@@ -25,7 +25,7 @@ class SparseVector {
     /// Creates a sparse vector from a span.
     explicit SparseVector(std::span<const float> value) {
         if (value.size() > std::numeric_limits<int>::max()) {
-            throw std::invalid_argument{"too many dimensions"};
+            throw std::invalid_argument{"sparsevec cannot have more than max int dimensions"};
         }
         dimensions_ = static_cast<int>(value.size());
 
@@ -42,7 +42,7 @@ class SparseVector {
     /// Creates a sparse vector from a map of non-zero elements.
     SparseVector(const std::unordered_map<int, float>& map, int dimensions) {
         if (dimensions < 0) {
-            throw std::invalid_argument{"sparsevec dimensions cannot be negative"};
+            throw std::invalid_argument{"sparsevec cannot have negative dimensions"};
         }
         dimensions_ = dimensions;
 
diff --git a/test/pqxx_test.cpp b/test/pqxx_test.cpp
@@ -258,7 +258,7 @@ void test_sparsevec_from_string() {
 
     assert_exception<pqxx::conversion_error>([] {
         auto _ = pqxx::from_string<pgvector::SparseVector>("{}/-1");
-    }, "sparsevec dimensions cannot be negative");
+    }, "sparsevec cannot have negative dimensions");
 
     assert_exception<pqxx::conversion_error>([] {
         auto _ = pqxx::from_string<pgvector::SparseVector>("{:}/1");
diff --git a/test/sparsevec_test.cpp b/test/sparsevec_test.cpp
@@ -28,7 +28,7 @@ static void test_constructor_map() {
 
     assert_exception<std::invalid_argument>([&]{
         SparseVector(map, -1);
-    }, "sparsevec dimensions cannot be negative");
+    }, "sparsevec cannot have negative dimensions");
 
     assert_exception<std::invalid_argument>([&]{
         SparseVector(map, 4);
