diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-08-14 14:09:30 -0400 |
|---|---|---|
| committer | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-08-14 14:09:30 -0400 |
| commit | 8a394b9db42a21f684c6ae85005a55c29d060c38 (patch) | |
| tree | a85c21348cd7dbc416801df09fc028917d1f3c41 /core/dive.c | |
| parent | fb47c15cd81970f19aeb9777c4c576b973889674 (diff) | |
| download | subsurface-8a394b9db42a21f684c6ae85005a55c29d060c38.tar.gz | |
Filter: constify doFilter() argument
Conceptually, the doFilter() functions shouldn't modify the dive
they test. Therefore, make the argument const. To do this, constify
the parameter of get_dive_location(), which likewise seems to be
the right thing to do.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/dive.c')
| -rw-r--r-- | core/dive.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/dive.c b/core/dive.c index da4b19346..482ba585b 100644 --- a/core/dive.c +++ b/core/dive.c @@ -4202,9 +4202,9 @@ const char *get_dive_country(struct dive *dive) return NULL; } -char *get_dive_location(struct dive *dive) +const char *get_dive_location(const struct dive *dive) { - struct dive_site *ds = get_dive_site_by_uuid(dive->dive_site_uuid); + const struct dive_site *ds = get_dive_site_by_uuid(dive->dive_site_uuid); if (ds && ds->name) return ds->name; return NULL; |