Remove if to save or not save the tags.

The way that we work with tags is a bit weird, we have a global
tag_list pointer and every dive copy the tags from there. the problem
is that the tags_changed() function looks at those two places ( the
global and the dive-local lists ) to see if something was changed,
but we shouldn't add anything at the global taglist untill a tag
is really added, so that function is bogus.

The correct way to check if something changed is to check all the
strings in tags against the strings in the QLineEdit and see if they
are the same, then we can add things to the tag_list.

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Tomaz Canabrava 2014-07-08 18:18:33 -03:00 committed by Dirk Hohndel
parent 954d401483
commit 0054304cc8

View file

@ -719,7 +719,6 @@ void MainTab::acceptChanges()
}
}
}
if (tagsChanged(&displayed_dive, cd))
saveTags();
#if 0 // with the new architecture this shouldn't be needed anymore