From d404aa767feb24b19da50ba833723b3dd0cfb271 Mon Sep 17 00:00:00 2001 From: Michael Werle Date: Thu, 3 Sep 2020 12:04:05 +0900 Subject: [Bug #2934] Geo Lookup - support for remote dive sites Some remote dive sites have no populated places (towns, cities) nearby. For such sites, we now fall back to looking up unpopulated place names, such as the reef or island name. Also some code refactorisation: the actual network access is now encapsulated in its own function removing some duplicated code handling in the reverseGeoLookup function and making it more readable. Furthermore, reverseGeoLookup() was completely refactored as most of its functionality was due to legacy requirements; the current code-base only calls this function from a single location and only with an empty taxonomy_data object. This makes the function more focussed and much simpler and more readable. Finally, a resource leak in reverseGeocde introduced in 4f3b26f9b6296273e37ec317bc68f32f94f546dc was fixed. Signed-off-by: Michael Werle --- core/taxonomy.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'core/taxonomy.h') diff --git a/core/taxonomy.h b/core/taxonomy.h index 5f7f0cf43..3af5160be 100644 --- a/core/taxonomy.h +++ b/core/taxonomy.h @@ -41,8 +41,8 @@ struct taxonomy_data { struct taxonomy *alloc_taxonomy(); void free_taxonomy(struct taxonomy_data *t); -void copy_taxonomy(struct taxonomy_data *orig, struct taxonomy_data *copy); -int taxonomy_index_for_category(struct taxonomy_data *t, enum taxonomy_category cat); +void copy_taxonomy(const struct taxonomy_data *orig, struct taxonomy_data *copy); +int taxonomy_index_for_category(const struct taxonomy_data *t, enum taxonomy_category cat); const char *taxonomy_get_country(struct taxonomy_data *t); void taxonomy_set_country(struct taxonomy_data *t, const char *country, enum taxonomy_origin origin); -- cgit v1.2.3-70-g09d2