diff options
author | Federico Mena Quintero <federico@ximian.com> | 2001-10-20 01:59:20 +0800 |
---|---|---|
committer | Federico Mena Quintero <federico@src.gnome.org> | 2001-10-20 01:59:20 +0800 |
commit | ee208ba079ed4d8e7073d1728cc24e2d5b7babcf (patch) | |
tree | 58335147dc935e87364da5e8476c9da834346ebf /calendar/gui | |
parent | 3e517a00308004b021cb5a02fa8ae17dfc30040f (diff) | |
download | gsoc2013-evolution-ee208ba079ed4d8e7073d1728cc24e2d5b7babcf.tar gsoc2013-evolution-ee208ba079ed4d8e7073d1728cc24e2d5b7babcf.tar.gz gsoc2013-evolution-ee208ba079ed4d8e7073d1728cc24e2d5b7babcf.tar.bz2 gsoc2013-evolution-ee208ba079ed4d8e7073d1728cc24e2d5b7babcf.tar.lz gsoc2013-evolution-ee208ba079ed4d8e7073d1728cc24e2d5b7babcf.tar.xz gsoc2013-evolution-ee208ba079ed4d8e7073d1728cc24e2d5b7babcf.tar.zst gsoc2013-evolution-ee208ba079ed4d8e7073d1728cc24e2d5b7babcf.zip |
Do not assert if we fail to load the URI list. This would of course have
2001-10-19 Federico Mena Quintero <federico@ximian.com>
* gui/alarm-notify/alarm-notify.c (add_uri_to_load): Do not assert
if we fail to load the URI list. This would of course have been a
bonobo-conf activation problem.
(remove_uri_to_load): Likewise.
* gui/alarm-notify/notify-main.c (load_calendars): Likewise.
* gui/alarm-notify/alarm-queue.c (load_missed_alarms): Make the
time range half-open so that we do not display the last alarm
twice.
svn path=/trunk/; revision=13797
Diffstat (limited to 'calendar/gui')
-rw-r--r-- | calendar/gui/alarm-notify/alarm-notify.c | 14 | ||||
-rw-r--r-- | calendar/gui/alarm-notify/alarm-queue.c | 7 | ||||
-rw-r--r-- | calendar/gui/alarm-notify/notify-main.c | 5 | ||||
-rw-r--r-- | calendar/gui/alarm-notify/save.c | 15 |
4 files changed, 29 insertions, 12 deletions
diff --git a/calendar/gui/alarm-notify/alarm-notify.c b/calendar/gui/alarm-notify/alarm-notify.c index 61b8e8ecc5..f4e493f31e 100644 --- a/calendar/gui/alarm-notify/alarm-notify.c +++ b/calendar/gui/alarm-notify/alarm-notify.c @@ -197,7 +197,12 @@ add_uri_to_load (GnomeVFSURI *uri) g_assert (str_uri != NULL); loaded_uris = get_calendars_to_load (); - g_assert (loaded_uris != NULL); + if (!loaded_uris) { + g_message ("add_uri_to_load(): Could not get the list of calendars to load; " + "will not add `%s'", str_uri); + g_free (str_uri); + return; + } /* Look for the URI in the list of calendars to load */ @@ -232,7 +237,12 @@ remove_uri_to_load (GnomeVFSURI *uri) g_assert (str_uri != NULL); loaded_uris = get_calendars_to_load (); - g_assert (loaded_uris != NULL); + if (!loaded_uris) { + g_message ("remove_uri_to_load(): Could not get the list of calendars to load; " + "will not add `%s'", str_uri); + g_free (str_uri); + return; + } /* Look for the URI in the list of calendars to load */ diff --git a/calendar/gui/alarm-notify/alarm-queue.c b/calendar/gui/alarm-notify/alarm-queue.c index b0a6aa8dda..0a57e78ad7 100644 --- a/calendar/gui/alarm-notify/alarm-queue.c +++ b/calendar/gui/alarm-notify/alarm-queue.c @@ -383,7 +383,12 @@ load_missed_alarms (ClientAlarms *ca) now = time (NULL); g_assert (saved_notification_time != -1); - load_alarms (ca, saved_notification_time, now); + + /* We add 1 to the saved_notification_time to make the time ranges + * half-open; we do not want to display the "last" displayed alarm + * twice, once when it occurs and once when the alarm daemon restarts. + */ + load_alarms (ca, saved_notification_time + 1, now); } /* Called when a calendar client finished loading; we load its alarms */ diff --git a/calendar/gui/alarm-notify/notify-main.c b/calendar/gui/alarm-notify/notify-main.c index f2e2444201..573bf0a372 100644 --- a/calendar/gui/alarm-notify/notify-main.c +++ b/calendar/gui/alarm-notify/notify-main.c @@ -117,7 +117,10 @@ load_calendars (void) int i; uris = get_calendars_to_load (); - g_assert (uris != NULL); + if (!uris) { + g_message ("load_calendars(): Could not get the list of calendars to load"); + return; + } for (i = 0; i < uris->len; i++) { char *uri; diff --git a/calendar/gui/alarm-notify/save.c b/calendar/gui/alarm-notify/save.c index b4a788e71e..2a24a7d17a 100644 --- a/calendar/gui/alarm-notify/save.c +++ b/calendar/gui/alarm-notify/save.c @@ -177,11 +177,8 @@ save_calendars_to_load (GPtrArray *uris) /* Save it */ any = bonobo_arg_new (TC_GNOME_Evolution_Calendar_StringSeq); -#if 0 - *((GNOME_Evolution_Calendar_StringSeq **) any->_value) = seq; -#else any->_value = seq; -#endif + CORBA_exception_init (&ev); bonobo_config_set_value (db, KEY_CALENDARS_TO_LOAD, any, &ev); @@ -248,11 +245,8 @@ get_calendars_to_load (void) CORBA_exception_free (&ev); /* Decode the value */ -#if 0 - seq = *((GNOME_Evolution_Calendar_StringSeq **) any->_value); -#else + seq = any->_value; -#endif len = seq->_length; uris = g_ptr_array_new (); @@ -261,7 +255,12 @@ get_calendars_to_load (void) for (i = 0; i < len; i++) uris->pdata[i] = g_strdup (seq->_buffer[i]); +#if 0 + /* FIXME: The any and sequence are leaked. If we release them this way, + * we crash inside the ORB freeing routines :( + */ bonobo_arg_release (any); +#endif return uris; } |