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
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Basically correct, but I think this needs a std::max(0, ...) around it, to cover walk-only journeys.