Itemviews: Cleanup examples

Cleanup some minor issues in the chart example:
 - remove unused members
 - use initializer list for members
 - pass a proper role to dataChanged()
 - honor roles parameter in PieView::dataChanged()
 - use nullptr instead 0
 - use new-style connect
 - fix indentation and other whitespaces

Change-Id: Idb212b07c006fe3ae31bee9cd9b1ba4d03043b5e
Reviewed-by: André Hartmann <aha_1980@gmx.de>
Reviewed-by: Paul Wicking <paul.wicking@qt.io>

1 files changed, 7 insertions, 17 deletions

diff --git a/examples/widgets/tutorials/modelview/2_formatting/mymodel.cpp b/examples/widgets/tutorials/modelview/2_formatting/mymodel.cpp
index 9511648392..938597a34e 100644
--- a/examples/widgets/tutorials/modelview/2_formatting/mymodel.cpp
+++ b/examples/widgets/tutorials/modelview/2_formatting/mymodel.cpp
@@ -48,13 +48,14 @@
 **
 ****************************************************************************/
 
+#include "mymodel.h"
+
 #include <QFont>
 #include <QBrush>
-#include "mymodel.h"
 #include <QDebug>
 
 MyModel::MyModel(QObject *parent)
-    :QAbstractTableModel(parent)
+    : QAbstractTableModel(parent)
 {
 }
 
@@ -78,7 +79,7 @@ QVariant MyModel::data(const QModelIndex &index, int role) const
     qDebug() << QString("row %1, col%2, role %3")
             .arg(row).arg(col).arg(role);
 
-    switch(role){
+    switch (role) {
     case Qt::DisplayRole:
         if (row == 0 && col == 1) return QString("<--left");
         if (row == 1 && col == 1) return QString("right-->");
@@ -86,36 +87,25 @@ QVariant MyModel::data(const QModelIndex &index, int role) const
         return QString("Row%1, Column%2")
                 .arg(row + 1)
                 .arg(col +1);
-        break;
     case Qt::FontRole:
-        if (row == 0 && col == 0) //change font only for cell(0,0)
-        {
+        if (row == 0 && col == 0) { //change font only for cell(0,0)
             QFont boldFont;
             boldFont.setBold(true);
             return boldFont;
         }
         break;
     case Qt::BackgroundRole:
-
         if (row == 1 && col == 2)  //change background only for cell(1,2)
-        {
-            QBrush redBackground(Qt::red);
-            return redBackground;
-        }
+            return QBrush(Qt::red);
         break;
     case Qt::TextAlignmentRole:
-
         if (row == 1 && col == 1) //change text alignment only for cell(1,1)
-        {
             return Qt::AlignRight + Qt::AlignVCenter;
-        }
         break;
     case Qt::CheckStateRole:
-
         if (row == 1 && col == 0) //add a checkbox to cell(1,0)
-        {
             return Qt::Checked;
-        }
+        break;
     }
     return QVariant();
 }