From 8dd694b56956a1c4ec3519743904222b465e10a5 Mon Sep 17 00:00:00 2001 From: Lars Henriksen Date: Thu, 13 Dec 2018 20:29:22 +0100 Subject: Simplify day storage The function day_process_storage() is a wrapper for day_store_items(). It has an unused second argument, and is only used twice to load the selected day. It has been removed. A new function, get_slctd_day(), is the equivalant of get_today() and replaces the very awkwardly named ui_calendar_get_slctd_day_sec(). Signed-off-by: Lukas Fleischer --- src/calcurse.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) (limited to 'src/calcurse.c') diff --git a/src/calcurse.c b/src/calcurse.c index a6d54fd..d470661 100644 --- a/src/calcurse.c +++ b/src/calcurse.c @@ -48,18 +48,13 @@ int count, reg; */ static void do_storage(int day_changed) { + /* * Save the selected item before rebuilding the day vector. */ struct day_item *day = ui_day_selitem(); union aptev_ptr item; - - if (day) { - /* - * day_process_storage() rebuilds the vector of day items, so - * we need to save the reference to the actual item here. - */ + if (day) item = day->item; - } - day_process_storage(ui_calendar_get_slctd_day(), day_changed); + day_store_items(get_slctd_day(), 1); ui_day_load_items(); if (day_changed) -- cgit v1.2.3-54-g00ecf