Skip to content

Commit 3c55027

Browse files
committed
Fix clang-format violations
Adjust line breaks and alignment to satisfy clang-format.
1 parent 7214fa1 commit 3c55027

File tree

2 files changed

+9
-12
lines changed

2 files changed

+9
-12
lines changed

src/iceberg/deletes/roaring_position_bitmap.cc

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -130,9 +130,9 @@ void RoaringPositionBitmap::Add(int64_t pos) {
130130

131131
void RoaringPositionBitmap::AddRange(int64_t pos_start, int64_t pos_end) {
132132
if (pos_start > pos_end) {
133-
throw std::invalid_argument(
134-
"AddRange requires pos_start <= pos_end, got [" + std::to_string(pos_start) +
135-
", " + std::to_string(pos_end) + ")");
133+
throw std::invalid_argument("AddRange requires pos_start <= pos_end, got [" +
134+
std::to_string(pos_start) + ", " +
135+
std::to_string(pos_end) + ")");
136136
}
137137
if (pos_start == pos_end) {
138138
return;
@@ -150,8 +150,8 @@ void RoaringPositionBitmap::AddRange(int64_t pos_start, int64_t pos_end) {
150150
for (int32_t key = start_key; key <= end_key; ++key) {
151151
uint64_t low_start = (key == start_key) ? Pos32Bits(pos_start) : uint64_t{0};
152152
uint64_t low_end = (key == end_key)
153-
? static_cast<uint64_t>(Pos32Bits(pos_end - 1)) + 1
154-
: (uint64_t{1} << 32);
153+
? static_cast<uint64_t>(Pos32Bits(pos_end - 1)) + 1
154+
: (uint64_t{1} << 32);
155155
impl_->bitmaps[key].addRange(low_start, low_end);
156156
}
157157
}

src/iceberg/test/roaring_position_bitmap_test.cc

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -204,13 +204,10 @@ TEST_P(RoaringPositionBitmapAddRangeNoOpTest, IsNoOp) {
204204

205205
INSTANTIATE_TEST_SUITE_P(
206206
AddRangeNoOpScenarios, RoaringPositionBitmapAddRangeNoOpTest,
207-
::testing::Values(AddRangeNoOpParams{.name = "equal", .start = 100, .end = 100},
208-
AddRangeNoOpParams{.name = "zero_length_at_zero",
209-
.start = 0,
210-
.end = 0},
211-
AddRangeNoOpParams{.name = "negative_both",
212-
.start = -10,
213-
.end = -5}),
207+
::testing::Values(
208+
AddRangeNoOpParams{.name = "equal", .start = 100, .end = 100},
209+
AddRangeNoOpParams{.name = "zero_length_at_zero", .start = 0, .end = 0},
210+
AddRangeNoOpParams{.name = "negative_both", .start = -10, .end = -5}),
214211
[](const ::testing::TestParamInfo<AddRangeNoOpParams>& info) {
215212
return info.param.name;
216213
});

0 commit comments

Comments
 (0)