mirrors/zed
1
0
Fork 0
mirror of https://github.com/zed-industries/zed.git synced 2025-01-24 11:01:54 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 12

assistant2: Fix panics when confirming nonexistent entries in the context picker (#22332)

Browse source 
This PR fixes a panic that could occur when confirming when the context
picker had no matching entries.

Release Notes:

- N/A
This commit is contained in:
Marshall Bowers 2024-12-21 11:11:29 -05:00 committed by GitHub
parent 831930aad0
commit 1449377278
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
crates/assistant2/src/context_picker/directory_context_picker.rs
View file

@ -178,7 +178,9 @@ impl PickerDelegate for DirectoryContextPickerDelegate {
}
fn confirm(&mut self, _secondary: bool, cx: &mut ViewContext<Picker<Self>>) {
let mat = &self.matches[self.selected_index];
let Some(mat) = self.matches.get(self.selected_index) else {
return;
};
let workspace = self.workspace.clone();
let Some(project) = workspace

4
crates/assistant2/src/context_picker/file_context_picker.rs
View file

@ -192,7 +192,9 @@ impl PickerDelegate for FileContextPickerDelegate {
}
fn confirm(&mut self, _secondary: bool, cx: &mut ViewContext<Picker<Self>>) {
let mat = &self.matches[self.selected_index];
let Some(mat) = self.matches.get(self.selected_index) else {
return;
};
let workspace = self.workspace.clone();
let Some(project) = workspace

4
crates/assistant2/src/context_picker/thread_context_picker.rs
View file

@ -154,7 +154,9 @@ impl PickerDelegate for ThreadContextPickerDelegate {
}
fn confirm(&mut self, _secondary: bool, cx: &mut ViewContext<Picker<Self>>) {
let entry = &self.matches[self.selected_index];
let Some(entry) = self.matches.get(self.selected_index) else {
return;
};
let Some(thread_store) = self.thread_store.upgrade() else {
return;