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

core/event: Improve setup function

Browse Source 
Use g_autoptr, shuffle some code around, and fix a memory leak.
This commit is contained in:
Georges Basile Stavracas Neto
2025-11-18 14:58:08 -03:00
parent ca819aa291
commit 6bb52f53e5
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/core/gcal-event.c
+4 -4
View File
@@ -308,13 +308,13 @@ setup_component (GcalEvent *self,
GError **error) GError **error)
{ {
g_autoptr (GTimeZone) zone_start = NULL; g_autoptr (GTimeZone) zone_start = NULL;
g_autoptr (GDateTime) date_start = NULL;
g_autoptr (GTimeZone) zone_end = NULL; g_autoptr (GTimeZone) zone_end = NULL;
g_autoptr (GDateTime) date_end = NULL;
ECalComponentDateTime *start; ECalComponentDateTime *start;
ECalComponentDateTime *end; ECalComponentDateTime *end;
ECalComponentText *text; ECalComponentText *text;
ICalTime *date; ICalTime *date;
GDateTime *date_start;
GDateTime *date_end;
gboolean start_is_all_day, end_is_all_day; gboolean start_is_all_day, end_is_all_day;
gchar *description, *location; gchar *description, *location;
@@ -353,7 +353,7 @@ setup_component (GcalEvent *self,
i_cal_time_is_date (date) ? 0 : i_cal_time_get_second (date)); i_cal_time_is_date (date) ? 0 : i_cal_time_get_second (date));
start_is_all_day = gcal_date_time_is_date (date_start); start_is_all_day = gcal_date_time_is_date (date_start);
self->dt_start = date_start; self->dt_start = g_steal_pointer (&date_start);
g_clear_object (&date); g_clear_object (&date);
@@ -380,7 +380,7 @@ setup_component (GcalEvent *self,
i_cal_time_is_date (date) ? 0 : i_cal_time_get_second (date)); i_cal_time_is_date (date) ? 0 : i_cal_time_get_second (date));
end_is_all_day = gcal_date_time_is_date (date_end); end_is_all_day = gcal_date_time_is_date (date_end);
self->dt_end = g_date_time_ref (date_end); self->dt_end = g_steal_pointer (&date_end);
/* Setup all day */ /* Setup all day */
self->all_day = start_is_all_day && end_is_all_day; self->all_day = start_is_all_day && end_is_all_day;