devs/foxbox
1
0
Fork 0
forked from len0rd/rockbox
Code Pull requests Activity

Theme Editor: Stopped bug that caused segfault if SBS element was moved from within a WPS preview

Browse source 
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27712 a1c6a512-1295-4272-9138-f99709370657
This commit is contained in:
Robert Bieber 2010-08-05 07:38:02 +00:00
parent bfc20975b0
commit 2ea1a34f98
2 changed files with 12 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

17
utils/themeeditor/models/parsetreemodel.cpp
View file

@ -35,7 +35,7 @@
#include <iostream>
ParseTreeModel::ParseTreeModel(const char* document, QObject* parent):
QAbstractItemModel(parent)
QAbstractItemModel(parent), sbsModel(0)
{
this->tree = skin_parse(document);
@ -54,6 +54,8 @@ ParseTreeModel::~ParseTreeModel()
delete root;
if(tree)
skin_free_tree(tree);
if(sbsModel)
sbsModel->deleteLater();
}
QString ParseTreeModel::genCode()
@ -324,19 +326,22 @@ QGraphicsScene* ParseTreeModel::render(ProjectModel* project,
{
QFile sbs(sbsFile);
sbs.open(QFile::ReadOnly | QFile::Text);
ParseTreeModel sbsModel(QString(sbs.readAll()).toAscii());
if(sbsModel.root != 0)
if(sbsModel)
sbsModel->deleteLater();
sbsModel = new ParseTreeModel(QString(sbs.readAll()).toAscii());
if(sbsModel->root != 0)
{
RBRenderInfo sbsInfo(&sbsModel, project, doc, &settings, device,
RBRenderInfo sbsInfo(sbsModel, project, doc, &settings, device,
sbsScreen);
sbsScreen = new RBScreen(sbsInfo, remote);
scene->addItem(sbsScreen);
sbsInfo = RBRenderInfo(&sbsModel, project, doc, &settings,
sbsInfo = RBRenderInfo(sbsModel, project, doc, &settings,
device, sbsScreen);
sbsModel.root->render(sbsInfo);
sbsModel->root->render(sbsInfo);
}
}

1
utils/themeeditor/models/parsetreemodel.h
View file

@ -81,6 +81,7 @@ public:
private:
ParseTreeNode* root;
ParseTreeModel* sbsModel;
struct skin_element* tree;
QGraphicsScene* scene;
};