diff options
author | Ted Gould <ted@gould.cx> | 2011-02-01 09:35:04 -0600 |
---|---|---|
committer | Ted Gould <ted@gould.cx> | 2011-02-01 09:35:04 -0600 |
commit | 004f46321f23d5967f698dd1201bc19b8a422dbf (patch) | |
tree | cccfc4cb6a75dce7b77d1e96a732c3a683e17311 | |
parent | 10fb9c744b01451f037466c941f018257a674878 (diff) | |
download | ayatana-indicator-datetime-004f46321f23d5967f698dd1201bc19b8a422dbf.tar.gz ayatana-indicator-datetime-004f46321f23d5967f698dd1201bc19b8a422dbf.tar.bz2 ayatana-indicator-datetime-004f46321f23d5967f698dd1201bc19b8a422dbf.zip |
Adding more comments
-rw-r--r-- | src/datetime-service.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/datetime-service.c b/src/datetime-service.c index 04a808a..db887a9 100644 --- a/src/datetime-service.c +++ b/src/datetime-service.c @@ -393,6 +393,9 @@ geo_create_address (GeoclueMasterClient * master, GeoclueAddress * address, GErr return; } + /* We shouldn't have created a new address if we already had one + so this is a warning. But, it really is only a mem-leak so we + don't need to error out. */ g_warn_if_fail(geo_address == NULL); g_debug("Created Geoclue Address"); @@ -422,7 +425,8 @@ geo_client_invalid (GeoclueMasterClient * client, gpointer user_data) { g_warning("Master client invalid, rebuilding."); - /* Client changes we can assume the address is invalid */ + /* Client changes we can assume the address is now invalid so we + need to unreference the one we had. */ if (geo_address != NULL) { g_object_unref(G_OBJECT(geo_address)); } @@ -453,6 +457,8 @@ geo_address_change (GeoclueMasterClient * client, gchar * a, gchar * b, gchar * { g_warning("Address provider changed. Let's change"); + /* If the address is supposed to have changed we need to drop the old + address before starting to get the new one. */ if (geo_address != NULL) { g_object_unref(G_OBJECT(geo_address)); } |