Silently ignore reversed ranges in AddRange for consistency

Replace the std::invalid_argument throw with a silent return on
reversed ranges (pos_start > pos_end), consistent with how Add()
handles invalid positions. The clamping already reduces the range
to empty when start > end after clamping, so the explicit throw
is removed entirely.

Author: Baunsgaard

src/iceberg/deletes/roaring_position_bitmap.cc

@@ -22,7 +22,6 @@
 #include <cstring>
 #include <exception>
 #include <limits>
-#include <stdexcept>
 #include <utility>
 #include <vector>
 
@@ -129,11 +128,6 @@ void RoaringPositionBitmap::Add(int64_t pos) {
 }
 
 void RoaringPositionBitmap::AddRange(int64_t pos_start, int64_t pos_end) {
-  if (pos_start > pos_end) {
-    throw std::invalid_argument("AddRange requires pos_start <= pos_end, got [" +
-                                std::to_string(pos_start) + ", " +
-                                std::to_string(pos_end) + ")");
-  }
   pos_start = std::max(pos_start, int64_t{0});
   pos_end = std::min(pos_end, kMaxPosition + 1);
   if (pos_start >= pos_end) {

src/iceberg/deletes/roaring_position_bitmap.h

@@ -67,8 +67,8 @@ class ICEBERG_EXPORT RoaringPositionBitmap {
   /// \brief Sets a range of positions [pos_start, pos_end).
   /// \param pos_start the start of the range (inclusive), clamped to 0
   /// \param pos_end the end of the range (exclusive), clamped to kMaxPosition + 1
-  /// \throws std::invalid_argument if pos_start > pos_end
-  /// \note If pos_start == pos_end, this method does nothing.
+  /// \note If pos_start > pos_end, the call is silently ignored.
+  ///       If pos_start == pos_end, this method does nothing.
   ///       Positions outside [0, kMaxPosition] are silently ignored.
   void AddRange(int64_t pos_start, int64_t pos_end);
 

src/iceberg/test/roaring_position_bitmap_test.cc

@@ -25,7 +25,6 @@
 #include <fstream>
 #include <random>
 #include <set>
-#include <stdexcept>
 #include <string>
 #include <vector>
 
@@ -211,9 +210,10 @@ INSTANTIATE_TEST_SUITE_P(
       return info.param.name;
     });
 
-TEST(RoaringPositionBitmapTest, TestAddRangeReversedThrows) {
+TEST(RoaringPositionBitmapTest, TestAddRangeReversedIsNoOp) {
   RoaringPositionBitmap bitmap;
-  ASSERT_THROW(bitmap.AddRange(100, 50), std::invalid_argument);
+  bitmap.AddRange(100, 50);
+  ASSERT_TRUE(bitmap.IsEmpty());
 }
 
 struct OrParams {