Federated_Computer/federated-calendar
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

event: Trivial cleanup

Browse Source 
We can avoid a lookup by simply checking if the alarm id
isn't NULL.
This commit is contained in:
Georges Basile Stavracas Neto
2020-06-17 21:24:36 -03:00
parent 4b6a559924
commit 158c65db4b
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/core/gcal-event.c
+5 -5
View File
@@ -1266,15 +1266,15 @@ void
gcal_event_remove_alarm (GcalEvent *self, gcal_event_remove_alarm (GcalEvent *self,
guint type) guint type)
{ {
const gchar *alarm_uid;
g_return_if_fail (GCAL_IS_EVENT (self)); g_return_if_fail (GCAL_IS_EVENT (self));
alarm_uid = g_hash_table_lookup (self->alarms, GINT_TO_POINTER (type));
/* Only 1 alarm per relative time */ /* Only 1 alarm per relative time */
if (g_hash_table_contains (self->alarms, GINT_TO_POINTER (type))) if (alarm_uid)
{ {
const gchar *alarm_uid;
alarm_uid = g_hash_table_lookup (self->alarms, GINT_TO_POINTER (type));
e_cal_component_remove_alarm (self->component, alarm_uid); e_cal_component_remove_alarm (self->component, alarm_uid);
g_hash_table_remove (self->alarms, GINT_TO_POINTER (type)); g_hash_table_remove (self->alarms, GINT_TO_POINTER (type));