Code review comment for lp:~mhr3/dee-qt/changeset-support

398 + // there's an extra row in the backend, skip it by incrementing
399 + row++;
400 + }
401 + } else {
402 + if (d->m_rowBeingRemoved >= 0 && row >= d->m_rowBeingRemoved) {
403 + // we need to skip the about-to-be-removed row
404 + row++;

Why are we skipping only one row? The changeset may span multiple rows, shouldn't this be taken into account?