rooty/LibrePilot
1
0
Fork 0
mirror of https://bitbucket.org/librepilot/librepilot.git synced 2025-02-27 16:54:15 +01:00
Code Issues Releases Activity

OP-781: cleaned up; added comments; removed some debug code; fixed tabs

Browse Source 
into spaces
This commit is contained in:
filnet 2012-12-30 18:27:51 +01:00
parent 87632e6e13
commit d0447b5e06
1 changed files with 45 additions and 61 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
ground/openpilotgcs/src/plugins/coreplugin/dialogs/settingsdialog.cpp
View File

@ -126,10 +126,9 @@ SettingsDialog::SettingsDialog(QWidget *parent, const QString &categoryId,
QList<IOptionsPage*> pages = sortedOptionsPages(); QList<IOptionsPage*> pages = sortedOptionsPages();
QTreeWidgetItem *initialItem = 0;
int index = 0; int index = 0;
bool firstUavGadgetOptionsPageFound = false; bool firstUavGadgetOptionsPageFound = false;
QTreeWidgetItem *initialItem = 0;
foreach (IOptionsPage *page, pages) { foreach (IOptionsPage *page, pages) {
PageData pageData; PageData pageData;
pageData.index = index; pageData.index = index;
@ -143,9 +142,6 @@ SettingsDialog::SettingsDialog(QWidget *parent, const QString &categoryId,
QString trCategories = page->trCategory(); QString trCategories = page->trCategory();
QString currentCategory = page->category(); QString currentCategory = page->category();
//qDebug() << "currentCategory: " << currentCategory;
//qDebug() << "page id: " << page->id();
QTreeWidgetItem *categoryItem; QTreeWidgetItem *categoryItem;
if (!categories.contains(currentCategory)) { if (!categories.contains(currentCategory)) {
// Above the first gadget option we insert a separator // Above the first gadget option we insert a separator
@ -174,12 +170,10 @@ SettingsDialog::SettingsDialog(QWidget *parent, const QString &categoryId,
m_pages.append(page); m_pages.append(page);
// creating all option pages upfront is slow, so we create placeholder widgets instead // creating all option pages upfront is slow, so we create place holder widgets instead
// the real option page widget will be created later when the user selects it // the real option page widget will be created later when the user selects it
// the placeholder is a QLabel and we assume that no option page will be a QLabel... // the place holder is a QLabel and we assume that no option page will be a QLabel...
QLabel * placeholderWidget = new QLabel(stackedPages); QLabel * placeholderWidget = new QLabel(stackedPages);
placeholderWidget->setText("page placeholder");
// add widget
stackedPages->addWidget(placeholderWidget); stackedPages->addWidget(placeholderWidget);
if (page->id() == initialPage && currentCategory == initialCategory) { if (page->id() == initialPage && currentCategory == initialCategory) {
@ -199,18 +193,11 @@ SettingsDialog::SettingsDialog(QWidget *parent, const QString &categoryId,
} }
} }
qDebug() << "initialItem: " << initialItem;
if (initialItem) { if (initialItem) {
qDebug() << "initialItem text: " << initialItem->text(0);
qDebug() << "initialItem selected: " << initialItem->isSelected();
qDebug() << "initialItem parent: " << initialItem->parent();
if (!initialItem->parent()) { if (!initialItem->parent()) {
// item has no parent, meaning it is single child // item has no parent, meaning it is single child
// so select category item instead // so select category item instead as single child are not added to the tree
initialItem = categories.value(initialCategory); initialItem = categories.value(initialCategory);
qDebug() << "initialItem text: " << initialItem->text(0);
qDebug() << "initialItem selected: " << initialItem->isSelected();
qDebug() << "initialItem parent: " << initialItem->parent();
} }
pageTree->setCurrentItem(initialItem); pageTree->setCurrentItem(initialItem);
} }
@ -229,8 +216,8 @@ SettingsDialog::~SettingsDialog()
QList<QTreeWidgetItem *> *categoryItemList = m_categoryItemsMap.value(category); QList<QTreeWidgetItem *> *categoryItemList = m_categoryItemsMap.value(category);
delete categoryItemList; delete categoryItemList;
} }
// delete placeholders // delete place holders
for(int i = 0; i < stackedPages->count(); i++) { for (int i = 0; i < stackedPages->count(); i++) {
QLabel * widget = dynamic_cast<QLabel*>(stackedPages->widget(i)); QLabel * widget = dynamic_cast<QLabel*>(stackedPages->widget(i));
if (widget) { if (widget) {
delete widget; delete widget;
@ -241,7 +228,6 @@ SettingsDialog::~SettingsDialog()
void SettingsDialog::pageSelected() void SettingsDialog::pageSelected()
{ {
QTreeWidgetItem *item = pageTree->currentItem(); QTreeWidgetItem *item = pageTree->currentItem();
qDebug() << "pageSelected: " << item;
if (!item) if (!item)
return; return;
@ -249,13 +235,13 @@ void SettingsDialog::pageSelected()
int index = data.index; int index = data.index;
m_currentCategory = data.category; m_currentCategory = data.category;
m_currentPage = data.id; m_currentPage = data.id;
// check if we are looking at a placeholder or not // check if we are looking at a place holder or not
QWidget * widget = dynamic_cast<QLabel*>(stackedPages->widget(index)); QWidget *widget = dynamic_cast<QLabel*>(stackedPages->widget(index));
if (widget) { if (widget) {
// get rid of placeholder // place holder found, get rid of it...
stackedPages->removeWidget(widget); stackedPages->removeWidget(widget);
delete widget; delete widget;
// insert real page // and replace place holder with actual option page
IOptionsPage *page = m_pages.at(index); IOptionsPage *page = m_pages.at(index);
stackedPages->insertWidget(index, page->createPage(stackedPages)); stackedPages->insertWidget(index, page->createPage(stackedPages));
} }
@ -345,7 +331,7 @@ void SettingsDialog::disableApplyOk(bool disable)
void SettingsDialog::accept() void SettingsDialog::accept()
{ {
m_applied = true; m_applied = true;
for(int i = 0; i < m_pages.size(); i++) { for (int i = 0; i < m_pages.size(); i++) {
QWidget * widget = dynamic_cast<QLabel*>(stackedPages->widget(i)); QWidget * widget = dynamic_cast<QLabel*>(stackedPages->widget(i));
if (!widget) { if (!widget) {
IOptionsPage * page = m_pages.at(i); IOptionsPage * page = m_pages.at(i);
@ -358,8 +344,7 @@ void SettingsDialog::accept()
void SettingsDialog::reject() void SettingsDialog::reject()
{ {
// foreach (IOptionsPage *page, m_pages) for (int i = 0; i < m_pages.size(); i++) {
for(int i = 0; i < m_pages.size(); i++) {
QWidget * widget = dynamic_cast<QLabel*>(stackedPages->widget(i)); QWidget * widget = dynamic_cast<QLabel*>(stackedPages->widget(i));
if (!widget) { if (!widget) {
IOptionsPage * page = m_pages.at(i); IOptionsPage * page = m_pages.at(i);
@ -371,8 +356,7 @@ void SettingsDialog::reject()
void SettingsDialog::apply() void SettingsDialog::apply()
{ {
// foreach (IOptionsPage *page, m_pages) for (int i = 0; i < m_pages.size(); i++) {
for(int i = 0; i < m_pages.size(); i++) {
QWidget * widget = dynamic_cast<QLabel*>(stackedPages->widget(i)); QWidget * widget = dynamic_cast<QLabel*>(stackedPages->widget(i));
if (!widget) { if (!widget) {
IOptionsPage * page = m_pages.at(i); IOptionsPage * page = m_pages.at(i);