More tweaks (am I opening a flood gate?!)

Author: nirvn

src/core/trackingmodel.cpp

@@ -57,7 +57,7 @@ QModelIndex TrackingModel::parent( const QModelIndex &index ) const
 int TrackingModel::rowCount( const QModelIndex &parent ) const
 {
   Q_UNUSED( parent )
-  return mTrackers.size();
+  return static_cast<int>( mTrackers.size() );
 }
 
 int TrackingModel::columnCount( const QModelIndex &parent ) const

src/core/valuemapmodelbase.cpp

@@ -55,7 +55,7 @@ void ValueMapModelBase::setMap( const QVariant &map )
     const QVariantMap valueMap = map.toMap();
     if ( !valueMap.empty() )
     {
-      beginInsertRows( QModelIndex(), 0, valueMap.size() - 1 );
+      beginInsertRows( QModelIndex(), 0, static_cast<int>( valueMap.size() ) - 1 );
 
       QMapIterator<QString, QVariant> i( valueMap );
       while ( i.hasNext() )
@@ -77,7 +77,7 @@ void ValueMapModelBase::setMap( const QVariant &map )
 int ValueMapModelBase::rowCount( const QModelIndex &parent ) const
 {
   Q_UNUSED( parent )
-  return mMap.size();
+  return static_cast<int>( mMap.size() );
 }
 
 QVariant ValueMapModelBase::data( const QModelIndex &index, int role ) const

src/core/vertexmodel.cpp

@@ -302,7 +302,7 @@ void VertexModel::createCandidates()
     if ( ( r == 0 || mVertices.at( r - 1 ).ring != vertex.ring ) && mGeometryType == Qgis::GeometryType::Polygon )
     {
       Vertex lastVertex;
-      for ( int i = r + 1; i < mVertices.count(); i++ )
+      for ( qsizetype i = r + 1; i < mVertices.count(); i++ )
       {
         if ( mVertices.at( i ).ring != vertex.ring )
           break;
@@ -670,7 +670,7 @@ void VertexModel::removeCurrentVertex()
 
 void VertexModel::updateGeometry( const QgsGeometry &geometry )
 {
-  int preservedIndex = mCurrentIndex;
+  qsizetype preservedIndex = mCurrentIndex;
   setGeometry( geometry );
   //since the index is shifted after reload, we decrement
   setCurrentVertex( preservedIndex - 1 );
@@ -737,7 +737,8 @@ void VertexModel::setCurrentVertex( qsizetype newVertex, bool forceUpdate )
   if ( mCurrentIndex >= 0 && mCurrentIndex < mVertices.count() )
   {
     mVertices[mCurrentIndex].currentVertex = false;
-    emit dataChanged( index( mCurrentIndex, 0, QModelIndex() ), index( mCurrentIndex, 0, QModelIndex() ) );
+    const QModelIndex changedIndex = index( static_cast<int>( mCurrentIndex ), 0, QModelIndex() );
+    emit dataChanged( changedIndex, changedIndex );
   }
 
   if ( mVertices.count() == 0 )
@@ -762,7 +763,8 @@ void VertexModel::setCurrentVertex( qsizetype newVertex, bool forceUpdate )
   if ( mCurrentIndex >= 0 && mCurrentIndex < mVertices.count() )
   {
     mVertices[mCurrentIndex].currentVertex = true;
-    emit dataChanged( index( mCurrentIndex, 0, QModelIndex() ), index( mCurrentIndex, 0, QModelIndex() ) );
+    const QModelIndex changedIndex = index( static_cast<int>( mCurrentIndex ), 0, QModelIndex() );
+    emit dataChanged( changedIndex, changedIndex );
   }
 
   emit currentVertexIndexChanged();
@@ -781,7 +783,7 @@ void VertexModel::setCurrentVertexIndex( qsizetype currentIndex )
 
 int VertexModel::currentVertexIndex() const
 {
-  return mCurrentIndex;
+  return static_cast<int>( mCurrentIndex );
 }
 
 int VertexModel::vertexCount() const