From aeffd9024e6d9550e8846c674ca81087d8936ef3 Mon Sep 17 00:00:00 2001 From: Dawid Wolosowicz Date: Sat, 4 Sep 2021 14:57:43 +0200 Subject: [PATCH] LibGUI: Remove barely used AbstractView::is_highlighting_searching() --- Userland/Libraries/LibGUI/AbstractView.cpp | 7 +------ Userland/Libraries/LibGUI/AbstractView.h | 1 - 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/Userland/Libraries/LibGUI/AbstractView.cpp b/Userland/Libraries/LibGUI/AbstractView.cpp index d4eb73392e..88c563775d 100644 --- a/Userland/Libraries/LibGUI/AbstractView.cpp +++ b/Userland/Libraries/LibGUI/AbstractView.cpp @@ -666,11 +666,6 @@ void AbstractView::set_searchable(bool searchable) cancel_searching(); } -bool AbstractView::is_highlighting_searching(const ModelIndex& index) const -{ - return index == m_highlighted_search_index; -} - void AbstractView::draw_item_text(Gfx::Painter& painter, const ModelIndex& index, bool is_selected, const Gfx::IntRect& text_rect, const StringView& item_text, const Gfx::Font& font, Gfx::TextAlignment alignment, Gfx::TextElision elision, size_t search_highlighting_offset) { if (m_edit_index == index) @@ -681,7 +676,7 @@ void AbstractView::draw_item_text(Gfx::Painter& painter, const ModelIndex& index text_color = is_focused() ? palette().selection_text() : palette().inactive_selection_text(); else text_color = index.data(ModelRole::ForegroundColor).to_color(palette().color(foreground_role())); - if (is_highlighting_searching(index)) { + if (index == m_highlighted_search_index) { Utf8View searching_text(m_searching); auto searching_length = searching_text.length(); if (searching_length > search_highlighting_offset) diff --git a/Userland/Libraries/LibGUI/AbstractView.h b/Userland/Libraries/LibGUI/AbstractView.h index cdb40daa17..717fe77240 100644 --- a/Userland/Libraries/LibGUI/AbstractView.h +++ b/Userland/Libraries/LibGUI/AbstractView.h @@ -166,7 +166,6 @@ protected: void cancel_searching(); void start_searching_timer(); void do_search(String&&); - bool is_highlighting_searching(const ModelIndex&) const; ModelIndex drop_candidate_index() const { return m_drop_candidate_index; }