use for_each_dive instead of hand-written loop.

It's easyer to make a mistake in the loop insteaf of
using the currently correct one that's already written

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Tomaz Canabrava 2014-05-10 22:12:40 -03:00 committed by Dirk Hohndel
parent 4b8a66f489
commit 25e0fdcb16

View file

@ -246,8 +246,9 @@ bool gpsHasChanged(struct dive *dive, struct dive *master, const QString &gps_te
QList<int> getDivesInTrip(dive_trip_t *trip)
{
QList<int> ret;
for (int i = 0; i < dive_table.nr; i++) {
struct dive *d = get_dive(i);
int i;
struct dive *d;
for_each_dive (i, d) {
if (d->divetrip == trip) {
ret.push_back(get_divenr(d));
}