github/SharpIDE
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

nav history max size

Browse Source
This commit is contained in:
Matt Parker
2025-10-29 22:04:44 +10:00
parent 05fa493b6b
commit 04f15094ac
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/SharpIDE.Application/Features/NavigationHistory/IdeNavigationHistoryService.cs
View File

@@ -5,7 +5,10 @@ namespace SharpIDE.Application.Features.NavigationHistory;
public class IdeNavigationHistoryService public class IdeNavigationHistoryService
{ {
private readonly Stack<IdeNavigationLocation> _backStack = new(); private const int MaxHistorySize = 100;
// Using LinkedList for back stack to efficiently remove oldest entries
private readonly LinkedList<IdeNavigationLocation> _backStack = new();
private IdeNavigationLocation? _current; private IdeNavigationLocation? _current;
private readonly Stack<IdeNavigationLocation> _forwardStack = new(); private readonly Stack<IdeNavigationLocation> _forwardStack = new();
@@ -27,7 +30,8 @@ public class IdeNavigationHistoryService
} }
if (_current is not null) if (_current is not null)
{ {
_backStack.Push(_current); _backStack.AddLast(_current);
if (_backStack.Count > MaxHistorySize) _backStack.RemoveFirst();
} }
_current = location; _current = location;
_forwardStack.Clear(); _forwardStack.Clear();
@@ -40,7 +44,8 @@ public class IdeNavigationHistoryService
{ {
_forwardStack.Push(_current); _forwardStack.Push(_current);
} }
_current = _backStack.Pop(); _current = _backStack.Last();
_backStack.RemoveLast();
} }
public void GoForward() public void GoForward()
@@ -48,7 +53,7 @@ public class IdeNavigationHistoryService
if (!CanGoForward) throw new InvalidOperationException("Cannot go forward, no history available."); if (!CanGoForward) throw new InvalidOperationException("Cannot go forward, no history available.");
if (_current is not null) if (_current is not null)
{ {
_backStack.Push(_current); _backStack.AddLast(_current);
} }
_current = _forwardStack.Pop(); _current = _forwardStack.Pop();