Commit add3c0b1 authored by Kohei Yoshida's avatar Kohei Yoshida

Actually use the clipped row range. Still untested.

The test comes next.
parent 34b67366
Pipeline #38152278 passed with stage
in 4 minutes and 46 seconds
......@@ -317,7 +317,8 @@ public:
}
const column_store_t& col = **m_it_cols;
m_current_pos = col.position(0);
m_current_pos = col.position(m_row_first);
m_end_pos = col.position(m_row_last+1);
}
bool has() const override
......@@ -331,10 +332,10 @@ public:
void next() override
{
m_update_current_cell = true;
m_current_pos = column_store_t::advance_position(m_current_pos, 1);
m_current_pos = column_store_t::next_position(m_current_pos);
const column_store_t* col = *m_it_cols;
if (m_current_pos.first != col->cend())
if (m_current_pos != m_end_pos)
// It hasn't reached the end of the current column yet.
return;
......@@ -344,7 +345,8 @@ public:
// Reset the position to the first cell in the new column.
col = *m_it_cols;
m_current_pos = col->position(0);
m_current_pos = col->position(m_row_first);
m_end_pos = col->position(m_row_last+1);
}
const model_iterator::cell& get() const override
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment