diff --git a/.gitignore b/.gitignore index cff24fb..77754fd 100644 --- a/.gitignore +++ b/.gitignore @@ -1,3 +1,6 @@ +#Build directory +build/ + # Compiled Object files *.slo *.lo diff --git a/canframemodel.cpp b/canframemodel.cpp index de09091..3edcdeb 100644 --- a/canframemodel.cpp +++ b/canframemodel.cpp @@ -11,6 +11,7 @@ CANFrameModel::~CANFrameModel() frames.clear(); filteredFrames.clear(); filters.clear(); + busFilters.clear(); } int CANFrameModel::rowCount(const QModelIndex &parent) const @@ -171,6 +172,13 @@ void CANFrameModel::setFilterState(unsigned int ID, bool state) sendRefresh(); } +void CANFrameModel::setBusFilterState(unsigned int BusID, bool state) +{ + if (!busFilters.contains(BusID)) return; + busFilters[BusID] = state; + sendRefresh(); +} + void CANFrameModel::setAllFilters(bool state) { QMap::iterator it; @@ -337,7 +345,7 @@ void CANFrameModel::recalcOverwrite() for (int i = 0; i < frames.count(); i++) { - if (filters[frames[i].frameId()]) + if (filters[frames[i].frameId()] && busFilters[frames[i].bus]) { filteredFrames.append(frames[i]); } @@ -578,6 +586,18 @@ bool CANFrameModel::any_filters_are_configured(void) return false; } +bool CANFrameModel::any_busfilters_are_configured(void) +{ + for (auto const &val : busFilters) + { + if (val == true) + continue; + else + return true; + } + return false; +} + void CANFrameModel::addFrame(const CANFrame& frame, bool autoRefresh = false) { @@ -601,10 +621,21 @@ void CANFrameModel::addFrame(const CANFrame& frame, bool autoRefresh = false) needFilterRefresh = true; } + //if this BusID isn't found in the busFilters list then add it and show it by default + if (!busFilters.contains(tempFrame.bus)) + { + // if there are any busFilters already configured, leave the new filter disabled + if (any_busfilters_are_configured()) + busFilters.insert(tempFrame.bus, false); + else + busFilters.insert(tempFrame.bus, true); + needFilterRefresh = true; + } + if (!overwriteDups) { frames.append(tempFrame); - if (filters[tempFrame.frameId()]) + if (filters[tempFrame.frameId()] && busFilters[tempFrame.bus]) { if (autoRefresh) beginInsertRows(QModelIndex(), filteredFrames.count(), filteredFrames.count()); tempFrame.frameCount = 1; @@ -629,7 +660,7 @@ void CANFrameModel::addFrame(const CANFrame& frame, bool autoRefresh = false) if (!found) { frames.append(tempFrame); - if (filters[tempFrame.frameId()]) + if (filters[tempFrame.frameId()] && busFilters[tempFrame.bus]) { if (autoRefresh) beginInsertRows(QModelIndex(), filteredFrames.count(), filteredFrames.count()); tempFrame.frameCount = 1; @@ -676,7 +707,7 @@ void CANFrameModel::sendRefresh() int count = frames.count(); for (int i = 0; i < count; i++) { - if (filters[frames[i].frameId()]) + if (filters[frames[i].frameId()] && busFilters[frames[i].bus]) { tempContainer.append(frames[i]); } @@ -727,6 +758,7 @@ void CANFrameModel::clearFrames() frames.clear(); filteredFrames.clear(); filters.clear(); + busFilters.clear(); frames.reserve(preallocSize); filteredFrames.reserve(preallocSize); this->endResetModel(); @@ -758,6 +790,11 @@ void CANFrameModel::insertFrames(const QVector &newFrames) needFilterRefresh = true; } if (filters[newFrames[i].frameId()]) + { + busFilters.insert(newFrames[i].bus, true); + needFilterRefresh = true; + } + if (filters[newFrames[i].frameId()] && busFilters[newFrames[i].bus]) { insertedFiltered++; filteredFrames.append(newFrames[i]); @@ -796,6 +833,7 @@ void CANFrameModel::loadFilterFile(QString filename) return; filters.clear(); + busFilters.clear(); while (!inFile->atEnd()) { line = inFile->readLine().simplified(); @@ -861,3 +899,8 @@ const QMap* CANFrameModel::getFiltersReference() const { return &filters; } + +const QMap* CANFrameModel::getBusFiltersReference() const +{ + return &busFilters; +} diff --git a/canframemodel.h b/canframemodel.h index 51762a1..47ceed7 100644 --- a/canframemodel.h +++ b/canframemodel.h @@ -48,6 +48,7 @@ public: void setHexMode(bool); void setSysTimeMode(bool); void setFilterState(unsigned int ID, bool state); + void setBusFilterState(unsigned int BusID, bool state); void setAllFilters(bool state); void setSecondsMode(bool); void setTimeFormat(QString); @@ -62,6 +63,7 @@ public: const QVector *getListReference() const; //thou shalt not modify these frames externally! const QVector *getFilteredListReference() const; //Thus saith the Lord, NO. const QMap *getFiltersReference() const; //this neither + const QMap *getBusFiltersReference() const; //this neither public slots: void addFrame(const CANFrame&, bool); @@ -75,10 +77,12 @@ private: void qSortCANFrameDesc(QVector* frames, Column column, int lowerBound, int upperBound); uint64_t getCANFrameVal(int row, Column col); bool any_filters_are_configured(void); + bool any_busfilters_are_configured(void); QVector frames; QVector filteredFrames; QMap filters; + QMap busFilters; DBCHandler *dbcHandler; QMutex mutex; bool interpretFrames; //should we use the dbcHandler? diff --git a/mainwindow.cpp b/mainwindow.cpp index dd5c091..eb47a56 100644 --- a/mainwindow.cpp +++ b/mainwindow.cpp @@ -124,6 +124,7 @@ MainWindow::MainWindow(QWidget *parent) : connect(ui->actionPreferences, &QAction::triggered, this, &MainWindow::showSettingsDialog); connect(model, &CANFrameModel::updatedFiltersList, this, &MainWindow::updateFilterList); connect(ui->listFilters, &QListWidget::itemChanged, this, &MainWindow::filterListItemChanged); + connect(ui->listBusFilters, &QListWidget::itemChanged, this, &MainWindow::busFilterListItemChanged); connect(ui->btnFilterAll, &QAbstractButton::clicked, this, &MainWindow::filterSetAll); connect(ui->btnFilterNone, &QAbstractButton::clicked, this, &MainWindow::filterClearAll); connect(ui->actionFirmware_Update, &QAction::triggered, this, &MainWindow::showFirmwareUploaderWindow); @@ -503,13 +504,15 @@ void MainWindow::updateFilterList() { if (model == nullptr) return; const QMap *filters = model->getFiltersReference(); - if (filters == nullptr) return; + const QMap *busFilters = model->getBusFiltersReference(); + if (filters == nullptr && busFilters == nullptr) return; qDebug() << "updateFilterList called on MainWindow"; inhibitFilterUpdate = true; ui->listFilters->clear(); + ui->listBusFilters->clear(); if (filters->isEmpty()) return; @@ -518,6 +521,13 @@ void MainWindow::updateFilterList() { /*QListWidgetItem *thisItem = */FilterUtility::createCheckableFilterItem(filterIter.key(), filterIter.value(), ui->listFilters); } + + if (busFilters->isEmpty()) return; + + for (filterIter = busFilters->begin(); filterIter != busFilters->end(); ++filterIter) + { + QListWidgetItem *thisItem = FilterUtility::createCheckableBusFilterItem(filterIter.key(), filterIter.value(), ui->listBusFilters); + } inhibitFilterUpdate = false; } @@ -534,6 +544,19 @@ void MainWindow::filterListItemChanged(QListWidgetItem *item) model->setFilterState(ID, isSet); } +void MainWindow::busFilterListItemChanged(QListWidgetItem *item) +{ + if (inhibitFilterUpdate) return; + //qDebug() << item->text(); + + // strip away possible filter label + int ID = FilterUtility::getIdAsInt(item); + bool isSet = false; + if (item->checkState() == Qt::Checked) isSet = true; + + model->setBusFilterState(ID, isSet); +} + void MainWindow::filterSetAll() { inhibitFilterUpdate = true; diff --git a/mainwindow.h b/mainwindow.h index 68b8cec..5505b51 100644 --- a/mainwindow.h +++ b/mainwindow.h @@ -94,11 +94,13 @@ private slots: void gridDoubleClicked(QModelIndex); void interpretToggled(bool); void overwriteToggled(bool); + void logReceivedFrame(CANConnection*, QVector); void tickGUIUpdate(); void toggleCapture(); void normalizeTiming(); void updateFilterList(); void filterListItemChanged(QListWidgetItem *item); + void busFilterListItemChanged(QListWidgetItem *item); void filterSetAll(); void filterClearAll(); void headerClicked (int logicalIndex);