TocModel: clean-up

src/frontends/qt4/GuiView.cpp

@@ -1664,6 +1664,8 @@ void GuiView::updateTocItem(string const & type, DocIterator const & dit)
 
 void GuiView::structureChanged()
 {
+	// FIXME: This is slightly expensive, though less than the tocBackend update
+	// (#9880). This also resets the view in the Toc Widget (#6675).
 	d.toc_models_.reset(documentBufferView());
 	// Navigator needs more than a simple update in this case. It needs to be
 	// rebuilt.

src/frontends/qt4/TocModel.cpp

@@ -38,6 +38,8 @@ namespace frontend {
 
 /// A QStandardItemModel that gives access to the reset methods.
 /// This is needed in order to fix http://www.lyx.org/trac/ticket/3740
+// FIXME: Better appropriately subclass QStandardItemModel and implement
+// the toc-specific reset methods there.
 class TocTypeModel : public QStandardItemModel
 {
 public:
@@ -47,27 +49,18 @@ public:
 	///
 	void reset()
 	{
-#if (QT_VERSION < 0x050000)
+		QStandardItemModel::beginResetModel();
-		QStandardItemModel::reset();
-#else
 		QStandardItemModel::endResetModel();
-#endif
 	}
 	///
 	void beginResetModel()
-	{ 
+	{
-	#if QT_VERSION >= 0x040600
+		QStandardItemModel::beginResetModel();
-		QStandardItemModel::beginResetModel(); 
-	#endif
 	}
 	///
 	void endResetModel()
-	{ 
+	{
-	#if QT_VERSION >= 0x040600
+		QStandardItemModel::endResetModel();
-		QStandardItemModel::endResetModel(); 
-	#else
-		QStandardItemModel::reset();
-	#endif
 	}
 };
 

src/frontends/qt4/TocModel.h

@@ -136,10 +136,6 @@ public:
 	TocItem const currentItem(QString const & type,
 		QModelIndex const & index) const;
 
-Q_SIGNALS:
-	/// Signal that the internal toc_models_ has been reset.
-	void modelReset();
-
 private:
 	typedef QHash<QString, TocModel *>::const_iterator const_iterator;
 	typedef QHash<QString, TocModel *>::iterator iterator;