Get rid of a Q_FOREACH usage in the new ledger view
Details
Details
- Reviewers
wojnilowicz - Commits
- R261:5a3dbee4e73f: Get rid of Q_FOREACH
Compile, run tests, use new ledger view, check balances on transactions
Diff Detail
Diff Detail
- Repository
- R261 KMyMoney
- Branch
- qforeach
- Lint
No Linters Available - Unit
No Unit Test Coverage
Comment Actions
You probably want to be porting to range-based for loop for (foo : bar) instead. Why all these unrelated auto changes?
kmymoney/views/ledgerview.cpp | ||
---|---|---|
77 | Make indexes const and then do for (const auto &index : indexes) { |
kmymoney/views/ledgerview.cpp | ||
---|---|---|
77 | foreach and Q_FOREACH are essentially the same but written the other way. It's possible to do here as broulik says, because it'll save us porting effort in very distant future. |