diff --git a/Makefile b/Makefile index 53eb05d3f..9b0e2ba9c 100644 --- a/Makefile +++ b/Makefile @@ -53,7 +53,6 @@ SOURCES = \ device.c \ dive.c \ divelist.c \ - download-dialog.c \ equipment.c \ file.c \ info.c \ diff --git a/download-dialog.c b/download-dialog.c index bbd499aaf..b98562571 100644 --- a/download-dialog.c +++ b/download-dialog.c @@ -9,24 +9,7 @@ const char *default_dive_computer_vendor; const char *default_dive_computer_product; const char *default_dive_computer_device; -struct product { - const char *product; - dc_descriptor_t *descriptor; - struct product *next; -}; -struct vendor { - const char *vendor; - struct product *productlist; - struct vendor *next; -}; - -struct mydescriptor { - const char *vendor; - const char *product; - dc_family_t type; - unsigned int model; -}; struct vendor *dc_list; diff --git a/qt-gui.cpp b/qt-gui.cpp index e3a8ad5aa..dfc1df19d 100644 --- a/qt-gui.cpp +++ b/qt-gui.cpp @@ -113,8 +113,8 @@ void exit_ui(void) #endif if (existing_filename) free((void *)existing_filename); - if (default_dive_computer_device) - free((void *)default_dive_computer_device); +// if (default_dive_computer_device) +// free((void *)default_dive_computer_device); } void set_filename(const char *filename, gboolean force) diff --git a/qt-ui/downloadfromdivecomputer.cpp b/qt-ui/downloadfromdivecomputer.cpp index 385f39755..2a1d7f566 100644 --- a/qt-ui/downloadfromdivecomputer.cpp +++ b/qt-ui/downloadfromdivecomputer.cpp @@ -2,9 +2,29 @@ #include "ui_downloadfromdivecomputer.h" #include "../libdivecomputer.h" - +#include #include #include +#include + +struct product { + const char *product; + dc_descriptor_t *descriptor; + struct product *next; +}; + +struct vendor { + const char *vendor; + struct product *productlist; + struct vendor *next; +}; + +struct mydescriptor { + const char *vendor; + const char *product; + dc_family_t type; + unsigned int model; +}; namespace DownloadFromDcGlobal{ const char *err_string; @@ -17,6 +37,65 @@ DownloadFromDCWidget::DownloadFromDCWidget(QWidget* parent, Qt::WindowFlags f) : ui->progressBar->hide(); ui->progressBar->setMinimum(0); ui->progressBar->setMaximum(100); + fill_computer_list(); + + vendorModel = new QStringListModel(vendorList); + ui->vendor->setModel(vendorModel); + ui->diveComputerName->setModel(0); +} + +void DownloadFromDCWidget::on_vendor_currentIndexChanged(const QString& vendor) +{ + QAbstractItemModel *currentModel = ui->diveComputerName->model(); + if (!currentModel) + return; + + productModel = new QStringListModel(productList[vendor]); + ui->diveComputerName->setModel(productModel); + + // Memleak - but deleting gives me a crash. + //currentModel->deleteLater(); +} + +void DownloadFromDCWidget::fill_computer_list() +{ + dc_iterator_t *iterator = NULL; + dc_descriptor_t *descriptor = NULL; + struct mydescriptor *mydescriptor; + struct vendor *dcl; + struct product *pl; + + QStringList computer; + dc_descriptor_iterator(&iterator); + while (dc_iterator_next (iterator, &descriptor) == DC_STATUS_SUCCESS) { + const char *vendor = dc_descriptor_get_vendor(descriptor); + const char *product = dc_descriptor_get_product(descriptor); + + if (!vendorList.contains( vendor )) + vendorList.append( vendor ); + + if( !productList[vendor].contains( product )) + productList[vendor].push_back( product ); + } + dc_iterator_free(iterator); + + /* and add the Uemis Zurich which we are handling internally + THIS IS A HACK as we magically have a data structure here that + happens to match a data structure that is internal to libdivecomputer; + this WILL BREAK if libdivecomputer changes the dc_descriptor struct... + eventually the UEMIS code needs to move into libdivecomputer, I guess */ + + mydescriptor = (struct mydescriptor*) malloc(sizeof(struct mydescriptor)); + mydescriptor->vendor = "Uemis"; + mydescriptor->product = "Zurich"; + mydescriptor->type = DC_FAMILY_NULL; + mydescriptor->model = 0; + + if(!vendorList.contains( "Uemis")) + vendorList.append("Uemis"); + + if( !productList["Uemis"].contains( "Zurich" )) + productList["Uemis"].push_back( "Zurich" ); } void DownloadFromDCWidget::on_cancel_clicked() diff --git a/qt-ui/downloadfromdivecomputer.h b/qt-ui/downloadfromdivecomputer.h index ef17df8d4..eefcc4d29 100644 --- a/qt-ui/downloadfromdivecomputer.h +++ b/qt-ui/downloadfromdivecomputer.h @@ -1,7 +1,9 @@ #ifndef DOWNLOADFROMDIVECOMPUTER_H #define DOWNLOADFROMDIVECOMPUTER_H + #include #include +#include namespace Ui{ class DownloadFromDiveComputer; @@ -29,6 +31,7 @@ private: device_data_t *data; }; +class QStringListModel; class DownloadFromDCWidget : public QDialog{ Q_OBJECT public: @@ -37,10 +40,19 @@ public: public slots: void on_ok_clicked(); void on_cancel_clicked(); + + void on_vendor_currentIndexChanged(const QString& vendor); private: Ui::DownloadFromDiveComputer *ui; InterfaceThread *thread; bool downloading; + + QStringList vendorList; + QHash productList; + + QStringListModel *vendorModel; + QStringListModel *productModel; + void fill_computer_list(); }; #endif \ No newline at end of file