From b703f5746d787c3efa1f42fb99b081d3bbee4c2e Mon Sep 17 00:00:00 2001 From: Georges Basile Stavracas Neto Date: Wed, 1 Apr 2020 21:01:58 -0300 Subject: [PATCH] week-header: Rename a variable "event_start" and "event_end" clearly communicate they're the start and end datetimes of the event. I'm dumb, so I need the code to tell me what's it about easily. --- src/views/gcal-week-header.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/views/gcal-week-header.c b/src/views/gcal-week-header.c index 03c24132..92f37eeb 100644 --- a/src/views/gcal-week-header.c +++ b/src/views/gcal-week-header.c @@ -1,6 +1,6 @@ /* gcal-week-header.c * - * Copyright (C) 2016 Georges Basile Stavracas Neto + * Copyright (C) 2016-2020 Georges Basile Stavracas Neto * Vamsi Krishna Gollapudi * * This program is free software: you can redistribute it and/or modify @@ -1725,8 +1725,8 @@ void gcal_week_header_add_event (GcalWeekHeader *self, GcalEvent *event) { - g_autoptr (GDateTime) start_date = NULL; - g_autoptr (GDateTime) end_date = NULL; + g_autoptr (GDateTime) event_start = NULL; + g_autoptr (GDateTime) event_end = NULL; g_autoptr (GDateTime) week_start = NULL; g_autoptr (GDateTime) week_end = NULL; gboolean all_day; @@ -1752,8 +1752,8 @@ gcal_week_header_add_event (GcalWeekHeader *self, g_date_time_get_day_of_month (week_end), 0, 0, 0); - start_date = g_date_time_ref (gcal_event_get_date_start (event)); - end_date = g_date_time_ref (gcal_event_get_date_end (event)); + event_start = g_date_time_ref (gcal_event_get_date_start (event)); + event_end = g_date_time_ref (gcal_event_get_date_end (event)); /* * Switch the week start and end by the UTC variants, in @@ -1770,19 +1770,19 @@ gcal_week_header_add_event (GcalWeekHeader *self, } else { - start_date = g_date_time_to_local (gcal_event_get_date_start (event)); - end_date = g_date_time_to_local (gcal_event_get_date_end (event)); + event_start = g_date_time_to_local (gcal_event_get_date_start (event)); + event_end = g_date_time_to_local (gcal_event_get_date_end (event)); } /* Start position */ - if (gcal_date_time_compare_date (start_date, week_start) >= 0) - start = floor (g_date_time_difference (start_date, week_start) / G_TIME_SPAN_DAY); + if (gcal_date_time_compare_date (event_start, week_start) >= 0) + start = floor (g_date_time_difference (event_start, week_start) / G_TIME_SPAN_DAY); else start = 0; /* End position */ - if (g_date_time_compare (end_date, week_end) <= 0) - end = floor (g_date_time_difference (end_date, week_start) / G_TIME_SPAN_DAY) - all_day; + if (g_date_time_compare (event_end, week_end) <= 0) + end = floor (g_date_time_difference (event_end, week_start) / G_TIME_SPAN_DAY) - all_day; else end = 6;