mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-11-30 22:20:21 +00:00
Code Issues Projects Releases Packages Wiki Activity

Cylinder model: fix an obscure case that would access cylinder -1

Browse source 
If we remove a cylidner for a unique gas and that is allowable, then don't
try to copy from cylinder with index same_gas (which is still -1).

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2015-06-21 21:30:38 -07:00
parent ce1be9c2eb
commit c01c35ca0c
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
qt-models/cylindermodel.cpp
View file

@ -345,6 +345,9 @@ void CylindersModel::remove(const QModelIndex &index)
} }
beginRemoveRows(QModelIndex(), index.row(), index.row()); // yah, know, ugly. beginRemoveRows(QModelIndex(), index.row(), index.row()); // yah, know, ugly.
rows--; rows--;
// if we didn't find an identical gas, point same_gas at the index.row()
if (same_gas == -1)
same_gas = index.row();
if (index.row() == 0) { if (index.row() == 0) {
// first gas - we need to make sure that the same gas ends up // first gas - we need to make sure that the same gas ends up
// as first gas // as first gas