NOISSUE fix minor memory leaks
This commit is contained in:
parent
67eca08b22
commit
eb747e08b7
@ -33,6 +33,14 @@ MinecraftProfile::MinecraftProfile(ProfileStrategy *strategy)
|
|||||||
clear();
|
clear();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
MinecraftProfile::~MinecraftProfile()
|
||||||
|
{
|
||||||
|
if(m_strategy)
|
||||||
|
{
|
||||||
|
delete m_strategy;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
void MinecraftProfile::setStrategy(ProfileStrategy* strategy)
|
void MinecraftProfile::setStrategy(ProfileStrategy* strategy)
|
||||||
{
|
{
|
||||||
Q_ASSERT(strategy != nullptr);
|
Q_ASSERT(strategy != nullptr);
|
||||||
|
@ -38,6 +38,7 @@ class MULTIMC_LOGIC_EXPORT MinecraftProfile : public QAbstractListModel
|
|||||||
|
|
||||||
public:
|
public:
|
||||||
explicit MinecraftProfile(ProfileStrategy *strategy);
|
explicit MinecraftProfile(ProfileStrategy *strategy);
|
||||||
|
virtual ~MinecraftProfile();
|
||||||
|
|
||||||
void setStrategy(ProfileStrategy * strategy);
|
void setStrategy(ProfileStrategy * strategy);
|
||||||
ProfileStrategy *strategy();
|
ProfileStrategy *strategy();
|
||||||
|
@ -46,13 +46,8 @@ InstanceWindow::InstanceWindow(InstancePtr instance, QWidget *parent)
|
|||||||
|
|
||||||
// Add page container
|
// Add page container
|
||||||
{
|
{
|
||||||
auto mainLayout = new QVBoxLayout;
|
|
||||||
auto provider = std::make_shared<InstancePageProvider>(m_instance);
|
auto provider = std::make_shared<InstancePageProvider>(m_instance);
|
||||||
m_container = new PageContainer(provider, "console", this);
|
m_container = new PageContainer(provider, "console", this);
|
||||||
mainLayout->addWidget(m_container);
|
|
||||||
mainLayout->setSpacing(0);
|
|
||||||
mainLayout->setContentsMargins(0,0,0,0);
|
|
||||||
setLayout(mainLayout);
|
|
||||||
setCentralWidget(m_container);
|
setCentralWidget(m_container);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -436,7 +436,8 @@ MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent), ui(new MainWindow
|
|||||||
view = new GroupView(ui->centralWidget);
|
view = new GroupView(ui->centralWidget);
|
||||||
|
|
||||||
view->setSelectionMode(QAbstractItemView::SingleSelection);
|
view->setSelectionMode(QAbstractItemView::SingleSelection);
|
||||||
view->setItemDelegate(new ListViewDelegate());
|
// FIXME: leaks ListViewDelegate
|
||||||
|
view->setItemDelegate(new ListViewDelegate(this));
|
||||||
view->setFrameShape(QFrame::NoFrame);
|
view->setFrameShape(QFrame::NoFrame);
|
||||||
// do not show ugly blue border on the mac
|
// do not show ugly blue border on the mac
|
||||||
view->setAttribute(Qt::WA_MacShowFocusRect, false);
|
view->setAttribute(Qt::WA_MacShowFocusRect, false);
|
||||||
|
@ -10,6 +10,7 @@ LogView::LogView(QWidget* parent) : QPlainTextEdit(parent)
|
|||||||
|
|
||||||
LogView::~LogView()
|
LogView::~LogView()
|
||||||
{
|
{
|
||||||
|
delete m_defaultFormat;
|
||||||
}
|
}
|
||||||
|
|
||||||
void LogView::setWordWrap(bool wrapping)
|
void LogView::setWordWrap(bool wrapping)
|
||||||
|
@ -126,7 +126,6 @@ void PageContainer::refresh()
|
|||||||
void PageContainer::createUI()
|
void PageContainer::createUI()
|
||||||
{
|
{
|
||||||
m_pageStack = new QStackedLayout;
|
m_pageStack = new QStackedLayout;
|
||||||
m_filter = new QLineEdit;
|
|
||||||
m_pageList = new PageView;
|
m_pageList = new PageView;
|
||||||
m_header = new QLabel();
|
m_header = new QLabel();
|
||||||
m_iconHeader = new IconLabel(this, QIcon(), QSize(24, 24));
|
m_iconHeader = new IconLabel(this, QIcon(), QSize(24, 24));
|
||||||
|
@ -60,7 +60,6 @@ private:
|
|||||||
QSortFilterProxyModel *m_proxyModel;
|
QSortFilterProxyModel *m_proxyModel;
|
||||||
PageModel *m_model;
|
PageModel *m_model;
|
||||||
QStackedLayout *m_pageStack;
|
QStackedLayout *m_pageStack;
|
||||||
QLineEdit *m_filter;
|
|
||||||
QListView *m_pageList;
|
QListView *m_pageList;
|
||||||
QLabel *m_header;
|
QLabel *m_header;
|
||||||
IconLabel *m_iconHeader;
|
IconLabel *m_iconHeader;
|
||||||
|
Loading…
Reference in New Issue
Block a user