diff options
author | Gunnar Beutner <gbeutner@serenityos.org> | 2021-05-16 08:47:13 +0200 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-05-16 17:49:42 +0200 |
commit | c4d0b0cd6b0d0b2c773ce8f11ac9d0aaec36f9b6 (patch) | |
tree | 68d196df14316c313faa80427bae9191eb0efc8b /Userland | |
parent | 24ca645f08857612f77370c0bca14493f1de0cd8 (diff) | |
download | serenity-c4d0b0cd6b0d0b2c773ce8f11ac9d0aaec36f9b6.zip |
Userland: Don't explicitly call Vector<T>::is_null()
This method always returns false so there's no reason for calling it.
Diffstat (limited to 'Userland')
-rw-r--r-- | Userland/Applications/SoundPlayer/BarsVisualizationWidget.cpp | 2 | ||||
-rw-r--r-- | Userland/Libraries/LibLine/SuggestionManager.cpp | 40 |
2 files changed, 20 insertions, 22 deletions
diff --git a/Userland/Applications/SoundPlayer/BarsVisualizationWidget.cpp b/Userland/Applications/SoundPlayer/BarsVisualizationWidget.cpp index 834660c25e..c700611e3b 100644 --- a/Userland/Applications/SoundPlayer/BarsVisualizationWidget.cpp +++ b/Userland/Applications/SoundPlayer/BarsVisualizationWidget.cpp @@ -23,7 +23,7 @@ void BarsVisualizationWidget::paint_event(GUI::PaintEvent& event) painter.add_clip_rect(event.rect()); painter.fill_rect(frame_inner_rect(), Color::Black); - if (m_sample_buffer.is_null() || m_sample_buffer.is_empty()) + if (m_sample_buffer.is_empty()) return; fft(m_sample_buffer, false); diff --git a/Userland/Libraries/LibLine/SuggestionManager.cpp b/Userland/Libraries/LibLine/SuggestionManager.cpp index 0eb384abfa..6f0d5a5b57 100644 --- a/Userland/Libraries/LibLine/SuggestionManager.cpp +++ b/Userland/Libraries/LibLine/SuggestionManager.cpp @@ -99,30 +99,28 @@ SuggestionManager::CompletionAttemptResult SuggestionManager::attempt_completion if (m_next_suggestion_index < m_suggestions.size()) { auto can_complete = m_next_suggestion_invariant_offset <= m_largest_common_suggestion_prefix_length; - if (!m_last_shown_suggestion.text.is_null()) { - ssize_t actual_offset; - size_t shown_length = m_last_shown_suggestion_display_length; - switch (mode) { - case CompletePrefix: + ssize_t actual_offset; + size_t shown_length = m_last_shown_suggestion_display_length; + switch (mode) { + case CompletePrefix: + actual_offset = 0; + break; + case ShowSuggestions: + actual_offset = 0 - m_largest_common_suggestion_prefix_length + m_next_suggestion_invariant_offset; + if (can_complete) + shown_length = m_largest_common_suggestion_prefix_length + m_last_shown_suggestion.trivia_view.length(); + break; + default: + if (m_last_shown_suggestion_display_length == 0) actual_offset = 0; - break; - case ShowSuggestions: - actual_offset = 0 - m_largest_common_suggestion_prefix_length + m_next_suggestion_invariant_offset; - if (can_complete) - shown_length = m_largest_common_suggestion_prefix_length + m_last_shown_suggestion.trivia_view.length(); - break; - default: - if (m_last_shown_suggestion_display_length == 0) - actual_offset = 0; - else - actual_offset = 0 - m_last_shown_suggestion_display_length + m_next_suggestion_invariant_offset; - break; - } - - result.offset_region_to_remove = { m_next_suggestion_invariant_offset, shown_length }; - result.new_cursor_offset = actual_offset; + else + actual_offset = 0 - m_last_shown_suggestion_display_length + m_next_suggestion_invariant_offset; + break; } + result.offset_region_to_remove = { m_next_suggestion_invariant_offset, shown_length }; + result.new_cursor_offset = actual_offset; + auto& suggestion = suggest(); set_current_suggestion_initiation_index(initiation_start_index); |