If a journey consists of n sections there are n-1 changes, not n.
Details
Details
- Reviewers
vkrause - Commits
- R1032:975d299d0720: Fix number of changes
Trip from Würzburg over Bamberg to Berlin shows 1 change instead of 2 in my app
Diff Detail
Diff Detail
- Repository
- R1032 KPublicTransport
- Branch
- changes
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 13862 Build 13880: arc lint + arc unit
Comment Actions
Basically correct, but I think this needs a std::max(0, ...) around it, to cover walk-only journeys.