C++ now requires overridden virtual functions to be marked with override.
In a previous commit, last() calls were translated to end() calls, but that is incorrect since end() points one "past" the last element of the array. One of these cases has already been fixed, but the other still needs to be included.
Finally, a missing assignment of a cell's cluster to itself caused cell_locs.at() to crash.
override
.last()
calls were translated toend()
calls, but that is incorrect sinceend()
points one "past" the last element of the array. One of these cases has already been fixed, but the other still needs to be included.cell_locs.at()
to crash.