From f57c5e3a5fbe03fcd86d72c29f00a10263347539 Mon Sep 17 00:00:00 2001
From: Johannes Altmanninger <aclopte@gmail.com>
Date: Wed, 29 Nov 2023 20:22:18 +0100
Subject: [PATCH] history: remove bogus assertion crashing on empty history
 items

In LastC++11, an empty history item means we either reached the end of history,
or the item is actually empty. The second meaning is still true.  We never
append empty history items but the history file might have been modified.

Fixes #10129
---
 fish-rust/src/history.rs | 2 --
 1 file changed, 2 deletions(-)

diff --git a/fish-rust/src/history.rs b/fish-rust/src/history.rs
index 12c7527ea..bff37a5e3 100644
--- a/fish-rust/src/history.rs
+++ b/fish-rust/src/history.rs
@@ -2032,8 +2032,6 @@ impl HistorySearch {
                 return false;
             };
 
-            assert!(!item.is_empty());
-
             // Look for an item that matches and (if deduping) that we haven't seen before.
             if !item.matches_search(&self.canon_term, self.search_type, !self.ignores_case()) {
                 continue;