From e618a245b97a0e1dce9524aae5efdd031cb94d83 Mon Sep 17 00:00:00 2001 From: Linus Torvalds Date: Tue, 22 Jan 2013 21:26:53 -0800 Subject: [PATCH] Use proper helper functions for dive location and for_each_dive This makes the code use the "dive_has_location()" function rather than check the longitude and latitude directly. It also uses "for_each_dive()" rather than open-coding it. Signed-off-by: Linus Torvalds Signed-off-by: Dirk Hohndel --- divelist.c | 4 ++-- gps.c | 5 ++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/divelist.c b/divelist.c index be5cd68b7..d033f93cf 100644 --- a/divelist.c +++ b/divelist.c @@ -982,7 +982,7 @@ static void get_suit(struct dive *dive, char **str) static GdkPixbuf *get_gps_icon(struct dive *dive) { - if (dive->latitude.udeg || dive->longitude.udeg) + if (dive_has_location(dive)) return gdk_pixbuf_from_pixdata(&my_pixbuf, TRUE, NULL); else return NULL; @@ -1714,7 +1714,7 @@ gboolean icon_click_cb(GtkWidget *w, GdkEventButton *event, gpointer data) gtk_tree_model_get_iter(MODEL(dive_list), &iter, path); gtk_tree_model_get(MODEL(dive_list), &iter, DIVE_INDEX, &idx, -1); dive = get_dive(idx); - if (dive && (dive->latitude.udeg || dive->longitude.udeg)) + if (dive && dive_has_location(dive)) show_gps_location(dive); } gtk_tree_path_free(path); diff --git a/gps.c b/gps.c index f68da0383..cc6eaa3a4 100644 --- a/gps.c +++ b/gps.c @@ -139,9 +139,8 @@ void show_gps_locations() if (!window || !map) map = init_map(); - for (idx = 0; idx < dive_table.nr; idx++) { - dp = dive_table.dives[idx]; - if (dp->latitude.udeg != 0 || dp->longitude.udeg != 0){ + for_each_dive(idx, dp) { + if (dive_has_location(dp)) { add_gps_point(map, dp->latitude.udeg / 1000000.0, dp->longitude.udeg / 1000000.0); }