From 9c48130d3717ca7e569bcf36fe753c90cc9dc3ea Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Sat, 21 Sep 2019 13:41:45 +0200 Subject: Cleanup: replace is_dive_site_used() by is_dive_site_selected() is_dive_site_used() had a "selected" parameter. If true it would return whether the given dive site had a selected dive. Turns out all callers had this parameter set to true. Therefore, replace by a simplified function without the "selected" parameter and give the function an appropriate name. Signed-off-by: Berthold Stoeger --- core/divesite.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'core/divesite.c') diff --git a/core/divesite.c b/core/divesite.c index db5d87cdf..68ac6e98a 100644 --- a/core/divesite.c +++ b/core/divesite.c @@ -191,13 +191,10 @@ int nr_of_dives_at_dive_site(struct dive_site *ds) return ds->dives.nr; } -bool is_dive_site_used(struct dive_site *ds, bool select_only) +bool is_dive_site_selected(struct dive_site *ds) { int i; - if (!select_only) - return ds->dives.nr > 0; - for (i = 0; i < ds->dives.nr; i++) { if (ds->dives.dives[i]->selected) return true; -- cgit v1.2.3-70-g09d2