Changeset eec69de in subsurface


Ignore:
Timestamp:
Dec 24, 2017, 9:10:12 AM (4 weeks ago)
Author:
Dirk Hohndel <dirk@…>
Branches:
master
Children:
4c4222d
Parents:
734bd6d
git-author:
Dirk Hohndel <dirk@…> (12/24/17 08:39:21)
git-committer:
Dirk Hohndel <dirk@…> (12/24/17 09:10:12)
Message:

Small whitespace updates

We aren't really consistent. And I don't do this often enough. But based
on a few things that I saw in a recent commit, I wanted to at least fix
those. And then of course fixed everything in those two files.

Signed-off-by: Dirk Hohndel <dirk@…>

Files:
2 edited

Legend:

Unmodified
Added
Removed
  • desktop-widgets/simplewidgets.cpp

    r7451517e reec69de  
    163163                MainWindow::instance()->dive_list()->rememberSelection();
    164164                // we remember a map from dive uuid to a pair of old number / new number
    165                 QMap<int,QPair<int, int> > renumberedDives;
     165                QMap<int, QPair<int, int>> renumberedDives;
    166166                int i;
    167167                int newNr = ui.spinBox->value();
     
    170170                        if (!selectedOnly || dive->selected) {
    171171                                invalidate_dive_cache(dive);
    172                                 renumberedDives.insert(dive->id, QPair<int,int>(dive->number, newNr++));
     172                                renumberedDives.insert(dive->id, QPair<int, int>(dive->number, newNr++));
    173173                        }
    174174                }
     
    215215}
    216216
    217 SetpointDialog::SetpointDialog(QWidget *parent) :
    218         QDialog(parent),
     217SetpointDialog::SetpointDialog(QWidget *parent) : QDialog(parent),
    219218        dc(0)
    220219{
     
    282281}
    283282
    284 ShiftTimesDialog::ShiftTimesDialog(QWidget *parent) :
    285         QDialog(parent),
     283ShiftTimesDialog::ShiftTimesDialog(QWidget *parent) : QDialog(parent),
    286284        when(0)
    287285{
     
    347345}
    348346
    349 ShiftImageTimesDialog::ShiftImageTimesDialog(QWidget *parent, QStringList fileNames) :
    350         QDialog(parent),
     347ShiftImageTimesDialog::ShiftImageTimesDialog(QWidget *parent, QStringList fileNames) : QDialog(parent),
    351348        fileNames(fileNames),
    352349        m_amount(0),
     
    360357        connect(ui.matchAllImages, SIGNAL(toggled(bool)), this, SLOT(matchAllImagesToggled(bool)));
    361358        dcImageEpoch = (time_t)0;
    362        
     359
    363360        updateInvalid();
    364361}
     
    410407        }
    411408
    412         if (!allValid){
     409        if (!allValid) {
    413410                ui.warningLabel->show();
    414411                ui.invalidFilesText->show();
     
    420417        m_amount = time.hour() * 3600 + time.minute() * 60;
    421418        if (ui.backwards->isChecked())
    422                         m_amount *= -1;
     419                m_amount *= -1;
    423420        updateInvalid();
    424421}
     
    427424{
    428425        if ((m_amount > 0) == ui.backwards->isChecked())
    429                         m_amount *= -1;
    430         if (m_amount) updateInvalid();
     426                m_amount *= -1;
     427        if (m_amount)
     428                updateInvalid();
    431429}
    432430
     
    501499}
    502500
    503 FilterBase::FilterBase(FilterModelBase *model_, QWidget *parent)
    504         : QWidget(parent)
    505         , model(model_)
     501FilterBase::FilterBase(FilterModelBase *model_, QWidget *parent) : QWidget(parent),
     502        model(model_)
    506503{
    507504        ui.setupUi(this);
     
    564561        TagFilter *tagFilter = new TagFilter(this);
    565562        int minimumHeight = tagFilter->ui.filterInternalList->height() +
    566                         tagFilter->ui.verticalLayout->spacing() * tagFilter->ui.verticalLayout->count();
     563                            tagFilter->ui.verticalLayout->spacing() * tagFilter->ui.verticalLayout->count();
    567564
    568565        QListView *dummyList = new QListView();
     
    583580
    584581        ui.scrollArea->setWidget(expandedWidget);
    585         expandedWidget->resize(expandedWidget->width(), minimumHeight + dummyList->sizeHintForRow(0) * 5 );
     582        expandedWidget->resize(expandedWidget->width(), minimumHeight + dummyList->sizeHintForRow(0) * 5);
    586583        ui.scrollArea->setMinimumHeight(expandedWidget->height() + 5);
    587584
     
    739736
    740737                grownText = cursor->selectedText();
    741                 if (grownText.size() == oldSize) movedOk = false;
     738                if (grownText.size() == oldSize)
     739                        movedOk = false;
    742740                oldSize = grownText.size();
    743741                noSpaces = grownText.simplified().replace(" ", "");
  • qt-models/filtermodels.cpp

    r7451517e reec69de  
    1313#include <algorithm>
    1414
    15 #define CREATE_INSTANCE_METHOD( CLASS ) \
    16 CLASS *CLASS::instance() \
    17 { \
    18         static CLASS *self = new CLASS(); \
    19         return self; \
    20 }
     15#define CREATE_INSTANCE_METHOD(CLASS)            \
     16        CLASS *CLASS::instance()                  \
     17        {                                        \
     18                static CLASS *self = new CLASS(); \
     19                return self;                      \
     20        }
    2121
    2222CREATE_INSTANCE_METHOD(TagFilterModel)
     
    2626CREATE_INSTANCE_METHOD(MultiFilterSortModel)
    2727
    28 FilterModelBase::FilterModelBase(QObject *parent) : QStringListModel(parent)
    29                                                   , anyChecked(false)
     28FilterModelBase::FilterModelBase(QObject *parent) : QStringListModel(parent),
     29        anyChecked(false)
    3030{
    3131}
     
    9898        std::fill(checkState.begin(), checkState.end(), false);
    9999        anyChecked = false;
    100         emit dataChanged(createIndex(0,0), createIndex(rowCount()-1, 0));
     100        emit dataChanged(createIndex(0, 0), createIndex(rowCount() - 1, 0));
    101101}
    102102
     
    105105        std::fill(checkState.begin(), checkState.end(), true);
    106106        anyChecked = true;
    107         emit dataChanged(createIndex(0,0), createIndex(rowCount()-1, 0));
     107        emit dataChanged(createIndex(0, 0), createIndex(rowCount() - 1, 0));
    108108}
    109109
    110110void FilterModelBase::invertSelection()
    111111{
    112         for (char &b: checkState)
     112        for (char &b : checkState)
    113113                b = !b;
    114         anyChecked = std::any_of(checkState.begin(), checkState.end(), [](char b){return !!b;});
    115         emit dataChanged(createIndex(0,0), createIndex(rowCount()-1, 0));
     114        anyChecked = std::any_of(checkState.begin(), checkState.end(), [](char b) { return !!b; });
     115        emit dataChanged(createIndex(0, 0), createIndex(rowCount() - 1, 0));
    116116}
    117117
     
    250250        QString persons = QString(d->buddy) + "," + QString(d->divemaster);
    251251        QStringList personsList = persons.split(',', QString::SkipEmptyParts);
    252         for (QString &s: personsList)
     252        for (QString &s : personsList)
    253253                s = s.trimmed();
    254254        // only show empty buddie dives if the user checked that.
     
    374374}
    375375
    376 MultiFilterSortModel::MultiFilterSortModel(QObject *parent) :
    377         QSortFilterProxyModel(parent),
     376MultiFilterSortModel::MultiFilterSortModel(QObject *parent) : QSortFilterProxyModel(parent),
    378377        divesDisplayed(0),
    379378        justCleared(false),
     
    395394                        for (int i = 0; i < sourceModel()->rowCount(index0); i++) {
    396395                                QModelIndex child = sourceModel()->index(i, 0, index0);
    397                                 d = (struct dive *) sourceModel()->data(child, DiveTripModel::DIVE_ROLE).value<void*>();
     396                                d = (struct dive *)sourceModel()->data(child, DiveTripModel::DIVE_ROLE).value<void *>();
    398397                                ds = get_dive_site_by_uuid(d->dive_site_uuid);
    399398                                if (!ds)
    400399                                        continue;
    401                                 if ( same_string(ds->name, curr_dive_site->name) || ds->uuid == curr_dive_site->uuid) {
     400                                if (same_string(ds->name, curr_dive_site->name) || ds->uuid == curr_dive_site->uuid) {
    402401                                        if (ds->uuid != curr_dive_site->uuid) {
    403402                                                qWarning() << "Warning, two different dive sites with same name have a different id"
    404                                                         << ds->uuid << "and" << curr_dive_site->uuid;
     403                                                           << ds->uuid << "and" << curr_dive_site->uuid;
    405404                                        }
    406405                                        showTrip = true; // do not shortcircuit the loop or the counts will be wrong
     
    412411                if (!ds)
    413412                        return false;
    414                 return ( same_string(ds->name, curr_dive_site->name) || ds->uuid == curr_dive_site->uuid);
     413                return (same_string(ds->name, curr_dive_site->name) || ds->uuid == curr_dive_site->uuid);
    415414        }
    416415
     
    468467        }
    469468
    470         for_each_dive (i,d) {
     469        for_each_dive (i, d) {
    471470                if (!d->hidden_by_filter)
    472471                        divesDisplayed++;
Note: See TracChangeset for help on using the changeset viewer.