From c1d8096dd48653c9b7caae0b6e3fb153166651b0 Mon Sep 17 00:00:00 2001
From: Dax <trogu.davide@gmail.com>
Date: Thu, 29 Nov 2018 14:32:20 +0100
Subject: [PATCH] More UI polish

---
 models/disassemblermodel.cpp                  | 10 ----------
 models/disassemblermodel.h                    |  4 +---
 models/listingitemmodel.cpp                   |  4 ++--
 models/referencesmodel.cpp                    |  2 --
 models/segmentsmodel.cpp                      |  9 +++------
 models/segmentsmodel.h                        |  2 +-
 models/symboltablemodel.h                     |  2 +-
 widgets/disassemblerview/disassemblerview.cpp |  4 ++--
 8 files changed, 10 insertions(+), 27 deletions(-)

diff --git a/models/disassemblermodel.cpp b/models/disassemblermodel.cpp
index 6911548b..a3497ef7 100644
--- a/models/disassemblermodel.cpp
+++ b/models/disassemblermodel.cpp
@@ -12,13 +12,3 @@ QVariant DisassemblerModel::headerData(int section, Qt::Orientation orientation,
 
     return QVariant();
 }
-
-QVariant DisassemblerModel::data(const QModelIndex &index, int role) const
-{
-    Q_UNUSED(index)
-
-    if(role == Qt::TextAlignmentRole)
-        return Qt::AlignCenter;
-
-    return QVariant();
-}
diff --git a/models/disassemblermodel.h b/models/disassemblermodel.h
index 897a5aa1..a76ea933 100644
--- a/models/disassemblermodel.h
+++ b/models/disassemblermodel.h
@@ -12,16 +12,14 @@ class DisassemblerModel : public QAbstractListModel
     Q_OBJECT
 
     public:
-        explicit DisassemblerModel(QObject *parent = 0);
+        explicit DisassemblerModel(QObject *parent = NULL);
 
     public:
         virtual void setDisassembler(REDasm::DisassemblerAPI *disassembler);
         virtual QVariant headerData(int section, Qt::Orientation orientation, int role) const;
-        virtual QVariant data(const QModelIndex &index, int role) const;
 
     protected:
         REDasm::DisassemblerAPI* m_disassembler;
-        bool m_defaultfont;
 };
 
 #endif // DISASSEMBLERMODEL_H
diff --git a/models/listingitemmodel.cpp b/models/listingitemmodel.cpp
index 3f443967..54248837 100644
--- a/models/listingitemmodel.cpp
+++ b/models/listingitemmodel.cpp
@@ -69,7 +69,7 @@ QVariant ListingItemModel::data(const QModelIndex &index, int role) const
     REDasm::SymbolPtr symbol = m_disassembler->document()->symbol(item->address);
 
     if(!symbol)
-        return DisassemblerModel::data(index, role);
+        return QVariant();
 
     if(role == Qt::DisplayRole)
     {
@@ -113,7 +113,7 @@ QVariant ListingItemModel::data(const QModelIndex &index, int role) const
             return THEME_VALUE("string_fg");
     }
 
-    return DisassemblerModel::data(index, role);
+    return QVariant();
 }
 
 bool ListingItemModel::isItemAllowed(REDasm::ListingItem *item) const
diff --git a/models/referencesmodel.cpp b/models/referencesmodel.cpp
index f82479d6..a3c834b9 100644
--- a/models/referencesmodel.cpp
+++ b/models/referencesmodel.cpp
@@ -94,8 +94,6 @@ QVariant ReferencesModel::data(const QModelIndex &index, int role) const
             }
         }
     }
-    else if((role == Qt::TextAlignmentRole) && (index.column() > 0))
-        return Qt::AlignCenter;
 
     return QVariant();
 }
diff --git a/models/segmentsmodel.cpp b/models/segmentsmodel.cpp
index 5ddc1175..b21198d1 100644
--- a/models/segmentsmodel.cpp
+++ b/models/segmentsmodel.cpp
@@ -5,15 +5,12 @@
 
 #define ADD_SEGMENT_TYPE(s, t) { if(!s.isEmpty()) s += " | ";  s += t; }
 
-SegmentsModel::SegmentsModel(QObject *parent) : ListingItemModel(REDasm::ListingItem::SegmentItem, parent)
-{
-
-}
+SegmentsModel::SegmentsModel(QObject *parent) : ListingItemModel(REDasm::ListingItem::SegmentItem, parent) { }
 
 QVariant SegmentsModel::data(const QModelIndex &index, int role) const
 {
     if(!m_disassembler)
-        return DisassemblerModel::data(index, role);
+        return QVariant();
 
     if(role == Qt::DisplayRole)
     {
@@ -39,7 +36,7 @@ QVariant SegmentsModel::data(const QModelIndex &index, int role) const
         return THEME_VALUE("address_list_fg");
     }
 
-    return DisassemblerModel::data(index, role);
+    return QVariant();
 }
 
 QVariant SegmentsModel::headerData(int section, Qt::Orientation orientation, int role) const
diff --git a/models/segmentsmodel.h b/models/segmentsmodel.h
index 7825312f..aa3bdcdf 100644
--- a/models/segmentsmodel.h
+++ b/models/segmentsmodel.h
@@ -8,7 +8,7 @@ class SegmentsModel : public ListingItemModel
     Q_OBJECT
 
     public:
-        explicit SegmentsModel(QObject *parent = 0);
+        explicit SegmentsModel(QObject *parent = NULL);
 
     public:
         virtual QVariant data(const QModelIndex &index, int role) const;
diff --git a/models/symboltablemodel.h b/models/symboltablemodel.h
index 22ab13df..1d00d4dd 100644
--- a/models/symboltablemodel.h
+++ b/models/symboltablemodel.h
@@ -8,7 +8,7 @@ class SymbolTableModel : public ListingItemModel
     Q_OBJECT
 
     public:
-        explicit SymbolTableModel(u32 itemtype, QObject *parent = nullptr);
+        explicit SymbolTableModel(u32 itemtype, QObject *parent = NULL);
         void setSymbolFlags(u32 symbolflags);
 
     protected:
diff --git a/widgets/disassemblerview/disassemblerview.cpp b/widgets/disassemblerview/disassemblerview.cpp
index 81e242ab..f7c3fde7 100644
--- a/widgets/disassemblerview/disassemblerview.cpp
+++ b/widgets/disassemblerview/disassemblerview.cpp
@@ -17,8 +17,8 @@ DisassemblerView::DisassemblerView(QPushButton *pbstatus, QLineEdit *lefilter, Q
     ui->vSplitter->setSizes((QList<int>() << this->width() * 0.70
                                            << this->width() * 0.30));
 
-    ui->hSplitter->setSizes((QList<int>() << this->width() * 0.30
-                                          << this->width() * 0.70));
+    ui->hSplitter->setSizes((QList<int>() << this->width() * 0.25
+                                          << this->width() * 0.75));
 
     m_disassemblerlistingview = new DisassemblerListingView(ui->stackedWidget);
     m_disassemblergraphview = new DisassemblerGraphView(ui->stackedWidget);