Empty Method in BigtableContents
While I'm working on a tiff loader issue, I found the following method in this core/data/bigtable.cpp
void BigtableContents::swap(int from, int to)
{
// std::swap(colIndexMap.at(from), colIndexMap.at(to));
}
This method is also called
//! Called upon QHeaderView::sectionMoved.
void BigtableModel::onColumnMove(int from, int to)
{
contents_.swap(from, to);
beginResetModel();
contents_.sortRows();
endResetModel();
}
I'm unsure if this was intentionally changed or not, but if this method is not needed anymore then it should be removed entirely. I'm asking @j.wuttke since you commented this out in this commit. Is/Was there any reason to keep this empty method?