From 1153068407e6ef47b6bf88d40e6f7d1594e9bc9d Mon Sep 17 00:00:00 2001
From: Domenico Iezzi <domenico@iezzi.info>
Date: Tue, 13 Feb 2024 21:35:41 +0100
Subject: [PATCH] LibGUI: Remove redundant Widget::update calls from DatePicker

In DatePicker constructor, calling to Widget::update after
Calendar::update_tiles is redundant since it is already called
internally by update_tiles.

This commit also remove unused header files from DatePicker header.
---
 Userland/Libraries/LibGUI/DatePicker.cpp | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/Userland/Libraries/LibGUI/DatePicker.cpp b/Userland/Libraries/LibGUI/DatePicker.cpp
index 3e385f992b..808dab5d5e 100644
--- a/Userland/Libraries/LibGUI/DatePicker.cpp
+++ b/Userland/Libraries/LibGUI/DatePicker.cpp
@@ -8,8 +8,6 @@
 #include <LibGUI/ComboBox.h>
 #include <LibGUI/DatePicker.h>
 #include <LibGUI/DatePickerDialogGML.h>
-#include <LibGUI/ImageWidget.h>
-#include <LibGUI/Label.h>
 #include <LibGUI/SpinBox.h>
 #include <LibGUI/TextBox.h>
 
@@ -48,7 +46,6 @@ DatePicker::DatePicker(Window* parent_window, String const& title, Core::DateTim
         m_selected_date.set_time(static_cast<int>(m_selected_date.year()), index.row() + 1);
         calendar.set_selected_date(m_selected_date);
         calendar.update_tiles(m_selected_date.year(), m_selected_date.month());
-        calendar.update();
     };
 
     m_year_box = widget->find_descendant_of_type_named<GUI::SpinBox>("year_box");
@@ -57,7 +54,6 @@ DatePicker::DatePicker(Window* parent_window, String const& title, Core::DateTim
         m_selected_date.set_time(year, static_cast<int>(m_selected_date.month()));
         calendar.set_selected_date(m_selected_date);
         calendar.update_tiles(m_selected_date.year(), m_selected_date.month());
-        calendar.update();
     };
 
     auto& ok_button = *widget->find_descendant_of_type_named<GUI::Button>("ok_button");