From de9bf6dfbdf3bf5d7f80772ff91e03afb900c914 Mon Sep 17 00:00:00 2001 From: Nathan Sobo Date: Thu, 6 Apr 2023 15:49:03 -0600 Subject: [PATCH] Merge MutableAppContext into AppContext There may have been a good reason for the difference at some point, or I was still learning Rust. But now it's just &mut AppContext vs &AppContext. --- .../src/activity_indicator.rs | 4 +- crates/auto_update/src/auto_update.rs | 8 +- crates/breadcrumbs/src/breadcrumbs.rs | 2 +- crates/call/src/call.rs | 6 +- crates/call/src/room.rs | 12 +- crates/client/src/client.rs | 4 +- crates/collab_ui/src/collab_titlebar_item.rs | 4 +- crates/collab_ui/src/collab_ui.rs | 10 +- crates/collab_ui/src/contact_finder.rs | 6 +- crates/collab_ui/src/contact_list.rs | 6 +- crates/collab_ui/src/contact_notification.rs | 4 +- crates/collab_ui/src/contacts_popover.rs | 4 +- .../src/incoming_call_notification.rs | 4 +- .../src/project_shared_notification.rs | 4 +- .../collab_ui/src/sharing_status_indicator.rs | 4 +- crates/command_palette/src/command_palette.rs | 4 +- crates/context_menu/src/context_menu.rs | 6 +- crates/copilot/src/copilot.rs | 7 +- crates/copilot/src/sign_in.rs | 6 +- crates/copilot_button/src/copilot_button.rs | 6 +- crates/db/src/db.rs | 4 +- crates/diagnostics/src/diagnostics.rs | 11 +- crates/diagnostics/src/items.rs | 4 +- crates/drag_and_drop/src/drag_and_drop.rs | 10 +- crates/editor/src/display_map.rs | 22 +- crates/editor/src/display_map/block_map.rs | 6 +- crates/editor/src/display_map/fold_map.rs | 14 +- .../editor/src/display_map/suggestion_map.rs | 6 +- crates/editor/src/display_map/tab_map.rs | 8 +- crates/editor/src/display_map/wrap_map.rs | 4 +- crates/editor/src/editor.rs | 18 +- crates/editor/src/editor_tests.rs | 70 +- crates/editor/src/element.rs | 12 +- crates/editor/src/hover_popover.rs | 4 +- crates/editor/src/items.rs | 12 +- crates/editor/src/link_go_to_definition.rs | 4 +- crates/editor/src/mouse_context_menu.rs | 4 +- crates/editor/src/movement.rs | 30 +- crates/editor/src/multi_buffer.rs | 31 +- crates/editor/src/scroll.rs | 4 +- crates/editor/src/scroll/actions.rs | 4 +- crates/editor/src/selections_collection.rs | 18 +- crates/editor/src/test.rs | 2 +- crates/feedback/src/feedback.rs | 4 +- crates/feedback/src/feedback_editor.rs | 6 +- crates/file_finder/src/file_finder.rs | 4 +- crates/go_to_line/src/go_to_line.rs | 8 +- crates/gpui/src/app.rs | 728 +++++++----------- crates/gpui/src/app/callback_collection.rs | 6 +- crates/gpui/src/app/menu.rs | 6 +- crates/gpui/src/app/test_app_context.rs | 27 +- crates/gpui/src/app/window_input_handler.rs | 12 +- crates/gpui/src/elements/label.rs | 2 +- crates/gpui/src/elements/list.rs | 4 +- crates/gpui/src/elements/text.rs | 6 +- crates/gpui/src/executor.rs | 4 +- crates/gpui/src/platform/mac/fonts.rs | 4 +- crates/gpui/src/presenter.rs | 38 +- crates/gpui/src/test.rs | 4 +- crates/gpui/src/text_layout.rs | 4 +- crates/gpui/src/views.rs | 2 +- crates/gpui/src/views/select.rs | 13 +- crates/journal/src/journal.rs | 6 +- crates/language/src/buffer.rs | 4 +- crates/language/src/buffer_tests.rs | 46 +- crates/language/src/language.rs | 13 +- .../src/language_selector.rs | 4 +- crates/live_kit_client/examples/test_app.rs | 2 +- crates/outline/src/outline.rs | 8 +- crates/picker/src/picker.rs | 6 +- crates/project/src/project.rs | 10 +- crates/project/src/project_tests.rs | 1 + crates/project/src/worktree.rs | 9 +- crates/project_panel/src/project_panel.rs | 6 +- crates/project_symbols/src/project_symbols.rs | 4 +- crates/recent_projects/src/recent_projects.rs | 4 +- crates/search/src/buffer_search.rs | 7 +- crates/search/src/project_search.rs | 8 +- crates/search/src/search.rs | 4 +- crates/settings/src/keymap_file.rs | 8 +- crates/settings/src/settings_file.rs | 6 +- crates/settings/src/watched_json.rs | 12 +- crates/staff_mode/src/staff_mode.rs | 10 +- crates/terminal_view/src/terminal_button.rs | 4 +- crates/terminal_view/src/terminal_view.rs | 6 +- crates/theme_selector/src/theme_selector.rs | 10 +- crates/theme_testbench/src/theme_testbench.rs | 6 +- crates/vim/src/editor_events.rs | 12 +- crates/vim/src/insert.rs | 4 +- crates/vim/src/motion.rs | 6 +- crates/vim/src/normal.rs | 12 +- crates/vim/src/normal/change.rs | 6 +- crates/vim/src/normal/delete.rs | 6 +- crates/vim/src/normal/yank.rs | 6 +- crates/vim/src/object.rs | 6 +- crates/vim/src/utils.rs | 4 +- crates/vim/src/vim.rs | 36 +- crates/vim/src/visual.rs | 12 +- crates/welcome/src/base_keymap_picker.rs | 4 +- crates/welcome/src/welcome.rs | 6 +- crates/workspace/src/dock.rs | 6 +- crates/workspace/src/item.rs | 75 +- crates/workspace/src/notifications.rs | 8 +- crates/workspace/src/pane.rs | 20 +- crates/workspace/src/searchable.rs | 45 +- crates/workspace/src/status_bar.rs | 17 +- crates/workspace/src/toolbar.rs | 14 +- crates/workspace/src/workspace.rs | 48 +- crates/zed/src/languages/json.rs | 4 +- crates/zed/src/languages/yaml.rs | 4 +- crates/zed/src/main.rs | 4 +- crates/zed/src/zed.rs | 59 +- 112 files changed, 882 insertions(+), 1041 deletions(-) diff --git a/crates/activity_indicator/src/activity_indicator.rs b/crates/activity_indicator/src/activity_indicator.rs index 2041bbc793..dfd98f6519 100644 --- a/crates/activity_indicator/src/activity_indicator.rs +++ b/crates/activity_indicator/src/activity_indicator.rs @@ -3,7 +3,7 @@ use editor::Editor; use futures::StreamExt; use gpui::{ actions, elements::*, platform::CursorStyle, Action, AppContext, Entity, ModelHandle, - MouseButton, MutableAppContext, RenderContext, View, ViewContext, ViewHandle, + MouseButton, RenderContext, View, ViewContext, ViewHandle, }; use language::{LanguageRegistry, LanguageServerBinaryStatus}; use project::{LanguageServerProgress, Project}; @@ -46,7 +46,7 @@ struct Content { action: Option>, } -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ActivityIndicator::show_error_message); cx.add_action(ActivityIndicator::dismiss_error_message); } diff --git a/crates/auto_update/src/auto_update.rs b/crates/auto_update/src/auto_update.rs index 3ad3380d26..9075e4df1a 100644 --- a/crates/auto_update/src/auto_update.rs +++ b/crates/auto_update/src/auto_update.rs @@ -5,7 +5,7 @@ use client::{ZED_APP_PATH, ZED_APP_VERSION, ZED_SECRET_CLIENT_TOKEN}; use db::kvp::KEY_VALUE_STORE; use gpui::{ actions, platform::AppVersion, AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, - MutableAppContext, Task, WeakViewHandle, + Task, WeakViewHandle, }; use serde::Deserialize; use settings::Settings; @@ -49,7 +49,7 @@ impl Entity for AutoUpdater { type Event = (); } -pub fn init(http_client: Arc, server_url: String, cx: &mut MutableAppContext) { +pub fn init(http_client: Arc, server_url: String, cx: &mut AppContext) { if let Some(version) = (*ZED_APP_VERSION).or_else(|| cx.platform().app_version().ok()) { let server_url = server_url; let auto_updater = cx.add_model(|cx| { @@ -95,7 +95,7 @@ pub fn init(http_client: Arc, server_url: String, cx: &mut Mutab pub fn notify_of_any_new_update( workspace: WeakViewHandle, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option<()> { let updater = AutoUpdater::get(cx)?; let version = updater.read(cx).current_version; @@ -124,7 +124,7 @@ pub fn notify_of_any_new_update( } impl AutoUpdater { - pub fn get(cx: &mut MutableAppContext) -> Option> { + pub fn get(cx: &mut AppContext) -> Option> { cx.default_global::>>().clone() } diff --git a/crates/breadcrumbs/src/breadcrumbs.rs b/crates/breadcrumbs/src/breadcrumbs.rs index 184dbe8468..2d8884647e 100644 --- a/crates/breadcrumbs/src/breadcrumbs.rs +++ b/crates/breadcrumbs/src/breadcrumbs.rs @@ -136,7 +136,7 @@ impl ToolbarItemView for Breadcrumbs { } } - fn pane_focus_update(&mut self, pane_focused: bool, _: &mut gpui::MutableAppContext) { + fn pane_focus_update(&mut self, pane_focused: bool, _: &mut gpui::AppContext) { self.pane_focused = pane_focused; } } diff --git a/crates/call/src/call.rs b/crates/call/src/call.rs index f9edddf374..0a8f150194 100644 --- a/crates/call/src/call.rs +++ b/crates/call/src/call.rs @@ -10,15 +10,15 @@ use futures::{future::Shared, FutureExt}; use postage::watch; use gpui::{ - AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, MutableAppContext, - Subscription, Task, WeakModelHandle, + AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, Subscription, Task, + WeakModelHandle, }; use project::Project; pub use participant::ParticipantLocation; pub use room::Room; -pub fn init(client: Arc, user_store: ModelHandle, cx: &mut MutableAppContext) { +pub fn init(client: Arc, user_store: ModelHandle, cx: &mut AppContext) { let active_call = cx.add_model(|cx| ActiveCall::new(client, user_store, cx)); cx.set_global(active_call); } diff --git a/crates/call/src/room.rs b/crates/call/src/room.rs index eeb8a6a5d8..98917eaeb6 100644 --- a/crates/call/src/room.rs +++ b/crates/call/src/room.rs @@ -11,7 +11,7 @@ use collections::{BTreeMap, HashMap, HashSet}; use fs::Fs; use futures::{FutureExt, StreamExt}; use gpui::{ - AsyncAppContext, Entity, ModelContext, ModelHandle, MutableAppContext, Task, WeakModelHandle, + AsyncAppContext, Entity, ModelContext, ModelHandle, AppContext, Task, WeakModelHandle, }; use language::LanguageRegistry; use live_kit_client::{LocalTrackPublication, LocalVideoTrack, RemoteVideoTrackUpdate}; @@ -64,7 +64,7 @@ pub struct Room { impl Entity for Room { type Event = Event; - fn release(&mut self, cx: &mut MutableAppContext) { + fn release(&mut self, cx: &mut AppContext) { if self.status.is_online() { self.leave_internal(cx).detach_and_log_err(cx); } @@ -72,7 +72,7 @@ impl Entity for Room { fn app_will_quit( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>>> { if self.status.is_online() { let leave = self.leave_internal(cx); @@ -176,7 +176,7 @@ impl Room { initial_project: Option>, client: Arc, user_store: ModelHandle, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task>> { cx.spawn(|mut cx| async move { let response = client.request(proto::CreateRoom {}).await?; @@ -219,7 +219,7 @@ impl Room { call: &IncomingCall, client: Arc, user_store: ModelHandle, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task>> { let room_id = call.room_id; cx.spawn(|mut cx| async move { @@ -257,7 +257,7 @@ impl Room { self.leave_internal(cx) } - fn leave_internal(&mut self, cx: &mut MutableAppContext) -> Task> { + fn leave_internal(&mut self, cx: &mut AppContext) -> Task> { if self.status.is_offline() { return Task::ready(Err(anyhow!("room is offline"))); } diff --git a/crates/client/src/client.rs b/crates/client/src/client.rs index 76004f14a4..6c8a5fd3af 100644 --- a/crates/client/src/client.rs +++ b/crates/client/src/client.rs @@ -15,7 +15,7 @@ use gpui::{ actions, serde_json::{self, Value}, AnyModelHandle, AnyViewHandle, AnyWeakModelHandle, AnyWeakViewHandle, AppContext, AppVersion, - AsyncAppContext, Entity, ModelHandle, MutableAppContext, Task, View, ViewContext, ViewHandle, + AsyncAppContext, Entity, ModelHandle, Task, View, ViewContext, ViewHandle, }; use lazy_static::lazy_static; use parking_lot::RwLock; @@ -67,7 +67,7 @@ pub const CONNECTION_TIMEOUT: Duration = Duration::from_secs(5); actions!(client, [SignIn, SignOut]); -pub fn init(client: Arc, cx: &mut MutableAppContext) { +pub fn init(client: Arc, cx: &mut AppContext) { cx.add_global_action({ let client = client.clone(); move |_: &SignIn, cx| { diff --git a/crates/collab_ui/src/collab_titlebar_item.rs b/crates/collab_ui/src/collab_titlebar_item.rs index b5e8696ec7..192e0bcb58 100644 --- a/crates/collab_ui/src/collab_titlebar_item.rs +++ b/crates/collab_ui/src/collab_titlebar_item.rs @@ -15,7 +15,7 @@ use gpui::{ geometry::{rect::RectF, vector::vec2f, PathBuilder}, impl_internal_actions, json::{self, ToJson}, - CursorStyle, Entity, ImageData, ModelHandle, MouseButton, MutableAppContext, RenderContext, + AppContext, CursorStyle, Entity, ImageData, ModelHandle, MouseButton, RenderContext, Subscription, View, ViewContext, ViewHandle, WeakViewHandle, }; use settings::Settings; @@ -40,7 +40,7 @@ impl_internal_actions!(collab, [LeaveCall]); #[derive(Copy, Clone, PartialEq)] pub(crate) struct LeaveCall; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(CollabTitlebarItem::toggle_collaborator_list_popover); cx.add_action(CollabTitlebarItem::toggle_contacts_popover); cx.add_action(CollabTitlebarItem::share_project); diff --git a/crates/collab_ui/src/collab_ui.rs b/crates/collab_ui/src/collab_ui.rs index 2dd2b0e6b4..7d991f8c94 100644 --- a/crates/collab_ui/src/collab_ui.rs +++ b/crates/collab_ui/src/collab_ui.rs @@ -13,13 +13,13 @@ mod sharing_status_indicator; use anyhow::anyhow; use call::ActiveCall; pub use collab_titlebar_item::{CollabTitlebarItem, ToggleContactsMenu}; -use gpui::{actions, MutableAppContext, Task}; +use gpui::{actions, AppContext, Task}; use std::sync::Arc; use workspace::{AppState, JoinProject, ToggleFollow, Workspace}; actions!(collab, [ToggleScreenSharing]); -pub fn init(app_state: Arc, cx: &mut MutableAppContext) { +pub fn init(app_state: Arc, cx: &mut AppContext) { collab_titlebar_item::init(cx); contact_notification::init(cx); contact_list::init(cx); @@ -35,7 +35,7 @@ pub fn init(app_state: Arc, cx: &mut MutableAppContext) { }); } -pub fn toggle_screen_sharing(_: &ToggleScreenSharing, cx: &mut MutableAppContext) { +pub fn toggle_screen_sharing(_: &ToggleScreenSharing, cx: &mut AppContext) { if let Some(room) = ActiveCall::global(cx).read(cx).room().cloned() { let toggle_screen_sharing = room.update(cx, |room, cx| { if room.is_screen_sharing() { @@ -48,13 +48,13 @@ pub fn toggle_screen_sharing(_: &ToggleScreenSharing, cx: &mut MutableAppContext } } -fn join_project(action: &JoinProject, app_state: Arc, cx: &mut MutableAppContext) { +fn join_project(action: &JoinProject, app_state: Arc, cx: &mut AppContext) { let project_id = action.project_id; let follow_user_id = action.follow_user_id; cx.spawn(|mut cx| async move { let existing_workspace = cx.update(|cx| { cx.window_ids() - .filter_map(|window_id| cx.root_view::(window_id)) + .filter_map(|window_id| cx.root_view(window_id)?.clone().downcast::()) .find(|workspace| { workspace.read(cx).project().read(cx).remote_id() == Some(project_id) }) diff --git a/crates/collab_ui/src/contact_finder.rs b/crates/collab_ui/src/contact_finder.rs index 5eefa60b8f..d34a344cd6 100644 --- a/crates/collab_ui/src/contact_finder.rs +++ b/crates/collab_ui/src/contact_finder.rs @@ -1,14 +1,14 @@ use client::{ContactRequestStatus, User, UserStore}; use gpui::{ - elements::*, AnyViewHandle, AppContext, Entity, ModelHandle, MouseState, MutableAppContext, - RenderContext, Task, View, ViewContext, ViewHandle, + elements::*, AnyViewHandle, AppContext, Entity, ModelHandle, MouseState, RenderContext, Task, + View, ViewContext, ViewHandle, }; use picker::{Picker, PickerDelegate}; use settings::Settings; use std::sync::Arc; use util::TryFutureExt; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { Picker::::init(cx); } diff --git a/crates/collab_ui/src/contact_list.rs b/crates/collab_ui/src/contact_list.rs index bfce7db1bc..164a742a85 100644 --- a/crates/collab_ui/src/contact_list.rs +++ b/crates/collab_ui/src/contact_list.rs @@ -10,8 +10,8 @@ use gpui::{ geometry::{rect::RectF, vector::vec2f}, impl_actions, impl_internal_actions, keymap_matcher::KeymapContext, - AppContext, CursorStyle, Entity, ModelHandle, MouseButton, MutableAppContext, PromptLevel, - RenderContext, Subscription, View, ViewContext, ViewHandle, + AppContext, CursorStyle, Entity, ModelHandle, MouseButton, PromptLevel, RenderContext, + Subscription, View, ViewContext, ViewHandle, }; use menu::{Confirm, SelectNext, SelectPrev}; use project::Project; @@ -24,7 +24,7 @@ use workspace::{JoinProject, OpenSharedScreen}; impl_actions!(contact_list, [RemoveContact, RespondToContactRequest]); impl_internal_actions!(contact_list, [ToggleExpanded, Call]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ContactList::remove_contact); cx.add_action(ContactList::respond_to_contact_request); cx.add_action(ContactList::cancel); diff --git a/crates/collab_ui/src/contact_notification.rs b/crates/collab_ui/src/contact_notification.rs index f05cca00bf..4bbba29ce9 100644 --- a/crates/collab_ui/src/contact_notification.rs +++ b/crates/collab_ui/src/contact_notification.rs @@ -3,14 +3,14 @@ use std::sync::Arc; use crate::notifications::render_user_notification; use client::{ContactEventKind, User, UserStore}; use gpui::{ - elements::*, impl_internal_actions, Entity, ModelHandle, MutableAppContext, RenderContext, + elements::*, impl_internal_actions, Entity, ModelHandle, AppContext, RenderContext, View, ViewContext, }; use workspace::notifications::Notification; impl_internal_actions!(contact_notifications, [Dismiss, RespondToContactRequest]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ContactNotification::dismiss); cx.add_action(ContactNotification::respond_to_contact_request); } diff --git a/crates/collab_ui/src/contacts_popover.rs b/crates/collab_ui/src/contacts_popover.rs index bb20cfac39..848f1205dc 100644 --- a/crates/collab_ui/src/contacts_popover.rs +++ b/crates/collab_ui/src/contacts_popover.rs @@ -1,7 +1,7 @@ use crate::{contact_finder::ContactFinder, contact_list::ContactList, ToggleContactsMenu}; use client::UserStore; use gpui::{ - actions, elements::*, Entity, ModelHandle, MouseButton, MutableAppContext, RenderContext, View, + actions, elements::*, AppContext, Entity, ModelHandle, MouseButton, RenderContext, View, ViewContext, ViewHandle, }; use project::Project; @@ -9,7 +9,7 @@ use settings::Settings; actions!(contacts_popover, [ToggleContactFinder]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ContactsPopover::toggle_contact_finder); } diff --git a/crates/collab_ui/src/incoming_call_notification.rs b/crates/collab_ui/src/incoming_call_notification.rs index 6fb0278218..6254228a95 100644 --- a/crates/collab_ui/src/incoming_call_notification.rs +++ b/crates/collab_ui/src/incoming_call_notification.rs @@ -4,7 +4,7 @@ use futures::StreamExt; use gpui::{ elements::*, geometry::{rect::RectF, vector::vec2f}, - impl_internal_actions, CursorStyle, Entity, MouseButton, MutableAppContext, RenderContext, + impl_internal_actions, CursorStyle, Entity, MouseButton, AppContext, RenderContext, View, ViewContext, WindowBounds, WindowKind, WindowOptions, }; use settings::Settings; @@ -13,7 +13,7 @@ use workspace::JoinProject; impl_internal_actions!(incoming_call_notification, [RespondToCall]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(IncomingCallNotification::respond_to_call); let mut incoming_call = ActiveCall::global(cx).read(cx).incoming(); diff --git a/crates/collab_ui/src/project_shared_notification.rs b/crates/collab_ui/src/project_shared_notification.rs index b24f3492da..769b89c2bc 100644 --- a/crates/collab_ui/src/project_shared_notification.rs +++ b/crates/collab_ui/src/project_shared_notification.rs @@ -5,7 +5,7 @@ use gpui::{ actions, elements::*, geometry::{rect::RectF, vector::vec2f}, - CursorStyle, Entity, MouseButton, MutableAppContext, RenderContext, View, ViewContext, + CursorStyle, Entity, MouseButton, AppContext, RenderContext, View, ViewContext, WindowBounds, WindowKind, WindowOptions, }; use settings::Settings; @@ -14,7 +14,7 @@ use workspace::JoinProject; actions!(project_shared_notification, [DismissProject]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ProjectSharedNotification::join); cx.add_action(ProjectSharedNotification::dismiss); diff --git a/crates/collab_ui/src/sharing_status_indicator.rs b/crates/collab_ui/src/sharing_status_indicator.rs index e16d4a2d89..cc77a59367 100644 --- a/crates/collab_ui/src/sharing_status_indicator.rs +++ b/crates/collab_ui/src/sharing_status_indicator.rs @@ -2,13 +2,13 @@ use call::ActiveCall; use gpui::{ color::Color, elements::{MouseEventHandler, Svg}, - Appearance, Element, ElementBox, Entity, MouseButton, MutableAppContext, RenderContext, View, + Appearance, Element, ElementBox, Entity, MouseButton, AppContext, RenderContext, View, }; use settings::Settings; use crate::ToggleScreenSharing; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { let active_call = ActiveCall::global(cx); let mut status_indicator = None; diff --git a/crates/command_palette/src/command_palette.rs b/crates/command_palette/src/command_palette.rs index 3a1f2e6161..a834a17ab6 100644 --- a/crates/command_palette/src/command_palette.rs +++ b/crates/command_palette/src/command_palette.rs @@ -4,7 +4,7 @@ use gpui::{ actions, elements::{ChildView, Flex, Label, ParentElement}, keymap_matcher::Keystroke, - Action, AnyViewHandle, Element, Entity, MouseState, MutableAppContext, RenderContext, View, + Action, AnyViewHandle, Element, Entity, MouseState, AppContext, RenderContext, View, ViewContext, ViewHandle, }; use picker::{Picker, PickerDelegate}; @@ -12,7 +12,7 @@ use settings::Settings; use std::cmp; use workspace::Workspace; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(CommandPalette::toggle); Picker::::init(cx); } diff --git a/crates/context_menu/src/context_menu.rs b/crates/context_menu/src/context_menu.rs index ffc121576e..06cdbcdf83 100644 --- a/crates/context_menu/src/context_menu.rs +++ b/crates/context_menu/src/context_menu.rs @@ -1,20 +1,20 @@ use gpui::{ elements::*, geometry::vector::Vector2F, impl_internal_actions, keymap_matcher::KeymapContext, platform::CursorStyle, Action, AnyViewHandle, AppContext, Axis, Entity, MouseButton, - MouseState, MutableAppContext, RenderContext, SizeConstraint, Subscription, View, ViewContext, + MouseState, RenderContext, SizeConstraint, Subscription, View, ViewContext, }; use menu::*; use settings::Settings; use std::{any::TypeId, borrow::Cow, time::Duration}; -pub type StaticItem = Box ElementBox>; +pub type StaticItem = Box ElementBox>; #[derive(Copy, Clone, PartialEq)] struct Clicked; impl_internal_actions!(context_menu, [Clicked]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ContextMenu::select_first); cx.add_action(ContextMenu::select_last); cx.add_action(ContextMenu::select_next); diff --git a/crates/copilot/src/copilot.rs b/crates/copilot/src/copilot.rs index 695ffbeed3..43fcf55e95 100644 --- a/crates/copilot/src/copilot.rs +++ b/crates/copilot/src/copilot.rs @@ -6,10 +6,7 @@ use async_compression::futures::bufread::GzipDecoder; use async_tar::Archive; use collections::HashMap; use futures::{future::Shared, Future, FutureExt, TryFutureExt}; -use gpui::{ - actions, AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, MutableAppContext, - Task, -}; +use gpui::{actions, AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, Task}; use language::{point_from_lsp, point_to_lsp, Anchor, Bias, Buffer, Language, ToPointUtf16}; use log::{debug, error}; use lsp::LanguageServer; @@ -34,7 +31,7 @@ actions!(copilot_auth, [SignIn, SignOut]); const COPILOT_NAMESPACE: &'static str = "copilot"; actions!(copilot, [NextSuggestion, PreviousSuggestion, Reinstall]); -pub fn init(http: Arc, node_runtime: Arc, cx: &mut MutableAppContext) { +pub fn init(http: Arc, node_runtime: Arc, cx: &mut AppContext) { // Disable Copilot for stable releases. if *cx.global::() == ReleaseChannel::Stable { cx.update_global::(|filter, _cx| { diff --git a/crates/copilot/src/sign_in.rs b/crates/copilot/src/sign_in.rs index 9fee3fcfaa..dba1a5e4d8 100644 --- a/crates/copilot/src/sign_in.rs +++ b/crates/copilot/src/sign_in.rs @@ -1,6 +1,6 @@ use crate::{request::PromptUserDeviceFlow, Copilot, Status}; use gpui::{ - elements::*, geometry::rect::RectF, ClipboardItem, Element, Entity, MutableAppContext, View, + elements::*, geometry::rect::RectF, ClipboardItem, Element, Entity, AppContext, View, ViewContext, ViewHandle, WindowKind, WindowOptions, }; use settings::Settings; @@ -14,7 +14,7 @@ struct OpenGithub; const COPILOT_SIGN_UP_URL: &'static str = "https://github.com/features/copilot"; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { let copilot = Copilot::global(cx).unwrap(); let mut code_verification: Option> = None; @@ -57,7 +57,7 @@ pub fn init(cx: &mut MutableAppContext) { } fn create_copilot_auth_window( - cx: &mut MutableAppContext, + cx: &mut AppContext, status: &Status, code_verification: &mut Option>, ) { diff --git a/crates/copilot_button/src/copilot_button.rs b/crates/copilot_button/src/copilot_button.rs index 589d65476f..2cbf08169b 100644 --- a/crates/copilot_button/src/copilot_button.rs +++ b/crates/copilot_button/src/copilot_button.rs @@ -3,8 +3,8 @@ use std::sync::Arc; use context_menu::{ContextMenu, ContextMenuItem}; use editor::Editor; use gpui::{ - elements::*, impl_internal_actions, CursorStyle, Element, ElementBox, Entity, MouseButton, - MouseState, MutableAppContext, RenderContext, Subscription, View, ViewContext, ViewHandle, + elements::*, impl_internal_actions, AppContext, CursorStyle, Element, ElementBox, Entity, + MouseButton, MouseState, RenderContext, Subscription, View, ViewContext, ViewHandle, }; use settings::{settings_file::SettingsFile, Settings}; use workspace::{ @@ -43,7 +43,7 @@ impl_internal_actions!( ] ); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(CopilotButton::deploy_copilot_menu); cx.add_action( |_: &mut CopilotButton, action: &ToggleCopilotForLanguage, cx| { diff --git a/crates/db/src/db.rs b/crates/db/src/db.rs index ae9325ea2e..5b8aca07e0 100644 --- a/crates/db/src/db.rs +++ b/crates/db/src/db.rs @@ -4,7 +4,7 @@ pub mod query; // Re-export pub use anyhow; use anyhow::Context; -use gpui::MutableAppContext; +use gpui::AppContext; pub use indoc::indoc; pub use lazy_static; use parking_lot::{Mutex, RwLock}; @@ -239,7 +239,7 @@ macro_rules! define_connection { }; } -pub fn write_and_log(cx: &mut MutableAppContext, db_write: impl FnOnce() -> F + Send + 'static) +pub fn write_and_log(cx: &mut AppContext, db_write: impl FnOnce() -> F + Send + 'static) where F: Future> + Send, { diff --git a/crates/diagnostics/src/diagnostics.rs b/crates/diagnostics/src/diagnostics.rs index 8abdeee2fb..0ac282d199 100644 --- a/crates/diagnostics/src/diagnostics.rs +++ b/crates/diagnostics/src/diagnostics.rs @@ -11,8 +11,8 @@ use editor::{ }; use gpui::{ actions, elements::*, fonts::TextStyle, impl_internal_actions, serde_json, AnyViewHandle, - AppContext, Entity, ModelHandle, MutableAppContext, RenderContext, Task, View, ViewContext, - ViewHandle, WeakViewHandle, + AppContext, Entity, ModelHandle, RenderContext, Task, View, ViewContext, ViewHandle, + WeakViewHandle, }; use language::{ Anchor, Bias, Buffer, Diagnostic, DiagnosticEntry, DiagnosticSeverity, Point, Selection, @@ -41,7 +41,7 @@ impl_internal_actions!(diagnostics, [Jump]); const CONTEXT_LINE_COUNT: u32 = 1; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ProjectDiagnosticsEditor::deploy); items::init(cx); } @@ -1176,10 +1176,7 @@ mod tests { }); } - fn editor_blocks( - editor: &ViewHandle, - cx: &mut MutableAppContext, - ) -> Vec<(u32, String)> { + fn editor_blocks(editor: &ViewHandle, cx: &mut AppContext) -> Vec<(u32, String)> { let mut presenter = cx.build_presenter(editor.id(), 0., Default::default()); let mut cx = presenter.build_layout_context(Default::default(), false, cx); cx.render(editor, |editor, cx| { diff --git a/crates/diagnostics/src/items.rs b/crates/diagnostics/src/items.rs index a6b0506768..d96d5fd1fb 100644 --- a/crates/diagnostics/src/items.rs +++ b/crates/diagnostics/src/items.rs @@ -2,7 +2,7 @@ use collections::HashSet; use editor::{Editor, GoToDiagnostic}; use gpui::{ elements::*, platform::CursorStyle, serde_json, Entity, ModelHandle, MouseButton, - MutableAppContext, RenderContext, Subscription, View, ViewContext, ViewHandle, WeakViewHandle, + AppContext, RenderContext, Subscription, View, ViewContext, ViewHandle, WeakViewHandle, }; use language::Diagnostic; use project::Project; @@ -17,7 +17,7 @@ pub struct DiagnosticIndicator { _observe_active_editor: Option, } -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(DiagnosticIndicator::go_to_next_diagnostic); } diff --git a/crates/drag_and_drop/src/drag_and_drop.rs b/crates/drag_and_drop/src/drag_and_drop.rs index b4881258f7..f5e5b4f138 100644 --- a/crates/drag_and_drop/src/drag_and_drop.rs +++ b/crates/drag_and_drop/src/drag_and_drop.rs @@ -5,8 +5,8 @@ use gpui::{ elements::{Empty, MouseEventHandler, Overlay}, geometry::{rect::RectF, vector::Vector2F}, scene::{MouseDown, MouseDrag}, - CursorStyle, Element, ElementBox, EventContext, MouseButton, MutableAppContext, RenderContext, - View, WeakViewHandle, + AppContext, CursorStyle, Element, ElementBox, EventContext, MouseButton, RenderContext, View, + WeakViewHandle, }; const DEAD_ZONE: f32 = 4.; @@ -261,7 +261,7 @@ impl DragAndDrop { }) } - pub fn cancel_dragging(&mut self, cx: &mut MutableAppContext) { + pub fn cancel_dragging(&mut self, cx: &mut AppContext) { if let Some(State::Dragging { payload, window_id, .. }) = &self.currently_dragged @@ -274,13 +274,13 @@ impl DragAndDrop { } } - fn finish_dragging(&mut self, cx: &mut MutableAppContext) { + fn finish_dragging(&mut self, cx: &mut AppContext) { if let Some(State::Dragging { window_id, .. }) = self.currently_dragged.take() { self.notify_containers_for_window(window_id, cx); } } - fn notify_containers_for_window(&mut self, window_id: usize, cx: &mut MutableAppContext) { + fn notify_containers_for_window(&mut self, window_id: usize, cx: &mut AppContext) { self.containers.retain(|container| { if let Some(container) = container.upgrade(cx) { if container.window_id() == window_id { diff --git a/crates/editor/src/display_map.rs b/crates/editor/src/display_map.rs index f54e2e980c..5bbd0a478a 100644 --- a/crates/editor/src/display_map.rs +++ b/crates/editor/src/display_map.rs @@ -843,7 +843,7 @@ pub fn next_rows(display_row: u32, display_map: &DisplaySnapshot) -> impl Iterat pub mod tests { use super::*; use crate::{movement, test::marked_display_snapshot}; - use gpui::{color::Color, elements::*, test::observe, MutableAppContext}; + use gpui::{color::Color, elements::*, test::observe, AppContext}; use language::{Buffer, Language, LanguageConfig, SelectionGoal}; use rand::{prelude::*, Rng}; use smol::stream::StreamExt; @@ -1117,7 +1117,7 @@ pub mod tests { } #[gpui::test(retries = 5)] - fn test_soft_wraps(cx: &mut MutableAppContext) { + fn test_soft_wraps(cx: &mut AppContext) { cx.foreground().set_block_on_ticks(usize::MAX..=usize::MAX); cx.foreground().forbid_parking(); @@ -1210,7 +1210,7 @@ pub mod tests { } #[gpui::test] - fn test_text_chunks(cx: &mut gpui::MutableAppContext) { + fn test_text_chunks(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let text = sample_text(6, 6, 'a'); let buffer = MultiBuffer::build_simple(&text, cx); @@ -1509,9 +1509,9 @@ pub mod tests { } #[gpui::test] - fn test_clip_point(cx: &mut gpui::MutableAppContext) { + fn test_clip_point(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); - fn assert(text: &str, shift_right: bool, bias: Bias, cx: &mut gpui::MutableAppContext) { + fn assert(text: &str, shift_right: bool, bias: Bias, cx: &mut gpui::AppContext) { let (unmarked_snapshot, mut markers) = marked_display_snapshot(text, cx); match bias { @@ -1558,10 +1558,10 @@ pub mod tests { } #[gpui::test] - fn test_clip_at_line_ends(cx: &mut gpui::MutableAppContext) { + fn test_clip_at_line_ends(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); - fn assert(text: &str, cx: &mut gpui::MutableAppContext) { + fn assert(text: &str, cx: &mut gpui::AppContext) { let (mut unmarked_snapshot, markers) = marked_display_snapshot(text, cx); unmarked_snapshot.clip_at_line_ends = true; assert_eq!( @@ -1577,7 +1577,7 @@ pub mod tests { } #[gpui::test] - fn test_tabs_with_multibyte_chars(cx: &mut gpui::MutableAppContext) { + fn test_tabs_with_multibyte_chars(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let text = "✅\t\tα\nβ\t\n🏀β\t\tγ"; let buffer = MultiBuffer::build_simple(text, cx); @@ -1638,7 +1638,7 @@ pub mod tests { } #[gpui::test] - fn test_max_point(cx: &mut gpui::MutableAppContext) { + fn test_max_point(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("aaa\n\t\tbbb", cx); let font_cache = cx.font_cache(); @@ -1687,7 +1687,7 @@ pub mod tests { rows: Range, map: &ModelHandle, theme: &'a SyntaxTheme, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Vec<(String, Option)> { chunks(rows, map, theme, cx) .into_iter() @@ -1699,7 +1699,7 @@ pub mod tests { rows: Range, map: &ModelHandle, theme: &'a SyntaxTheme, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Vec<(String, Option, Option)> { let snapshot = map.update(cx, |map, cx| map.snapshot(cx)); let mut chunks: Vec<(String, Option, Option)> = Vec::new(); diff --git a/crates/editor/src/display_map/block_map.rs b/crates/editor/src/display_map/block_map.rs index c4af03e703..20af1535a0 100644 --- a/crates/editor/src/display_map/block_map.rs +++ b/crates/editor/src/display_map/block_map.rs @@ -1015,7 +1015,7 @@ mod tests { } #[gpui::test] - fn test_basic_blocks(cx: &mut gpui::MutableAppContext) { + fn test_basic_blocks(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let family_id = cx @@ -1191,7 +1191,7 @@ mod tests { } #[gpui::test] - fn test_blocks_on_wrapped_lines(cx: &mut gpui::MutableAppContext) { + fn test_blocks_on_wrapped_lines(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let family_id = cx @@ -1241,7 +1241,7 @@ mod tests { } #[gpui::test(iterations = 100)] - fn test_random_blocks(cx: &mut gpui::MutableAppContext, mut rng: StdRng) { + fn test_random_blocks(cx: &mut gpui::AppContext, mut rng: StdRng) { cx.set_global(Settings::test(cx)); let operations = env::var("OPERATIONS") diff --git a/crates/editor/src/display_map/fold_map.rs b/crates/editor/src/display_map/fold_map.rs index fa1de81e0d..b8802e3c90 100644 --- a/crates/editor/src/display_map/fold_map.rs +++ b/crates/editor/src/display_map/fold_map.rs @@ -1214,7 +1214,7 @@ mod tests { use Bias::{Left, Right}; #[gpui::test] - fn test_basic_folds(cx: &mut gpui::MutableAppContext) { + fn test_basic_folds(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple(&sample_text(5, 6, 'a'), cx); let subscription = buffer.update(cx, |buffer, _| buffer.subscribe()); @@ -1287,7 +1287,7 @@ mod tests { } #[gpui::test] - fn test_adjacent_folds(cx: &mut gpui::MutableAppContext) { + fn test_adjacent_folds(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("abcdefghijkl", cx); let subscription = buffer.update(cx, |buffer, _| buffer.subscribe()); @@ -1334,7 +1334,7 @@ mod tests { } #[gpui::test] - fn test_overlapping_folds(cx: &mut gpui::MutableAppContext) { + fn test_overlapping_folds(cx: &mut gpui::AppContext) { let buffer = MultiBuffer::build_simple(&sample_text(5, 6, 'a'), cx); let buffer_snapshot = buffer.read(cx).snapshot(cx); let mut map = FoldMap::new(buffer_snapshot.clone()).0; @@ -1350,7 +1350,7 @@ mod tests { } #[gpui::test] - fn test_merging_folds_via_edit(cx: &mut gpui::MutableAppContext) { + fn test_merging_folds_via_edit(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple(&sample_text(5, 6, 'a'), cx); let subscription = buffer.update(cx, |buffer, _| buffer.subscribe()); @@ -1374,7 +1374,7 @@ mod tests { } #[gpui::test] - fn test_folds_in_range(cx: &mut gpui::MutableAppContext) { + fn test_folds_in_range(cx: &mut gpui::AppContext) { let buffer = MultiBuffer::build_simple(&sample_text(5, 6, 'a'), cx); let buffer_snapshot = buffer.read(cx).snapshot(cx); let mut map = FoldMap::new(buffer_snapshot.clone()).0; @@ -1401,7 +1401,7 @@ mod tests { } #[gpui::test(iterations = 100)] - fn test_random_folds(cx: &mut gpui::MutableAppContext, mut rng: StdRng) { + fn test_random_folds(cx: &mut gpui::AppContext, mut rng: StdRng) { cx.set_global(Settings::test(cx)); let operations = env::var("OPERATIONS") .map(|i| i.parse().expect("invalid `OPERATIONS` variable")) @@ -1656,7 +1656,7 @@ mod tests { } #[gpui::test] - fn test_buffer_rows(cx: &mut gpui::MutableAppContext) { + fn test_buffer_rows(cx: &mut gpui::AppContext) { let text = sample_text(6, 6, 'a') + "\n"; let buffer = MultiBuffer::build_simple(&text, cx); diff --git a/crates/editor/src/display_map/suggestion_map.rs b/crates/editor/src/display_map/suggestion_map.rs index 17dbae6031..7be377a69b 100644 --- a/crates/editor/src/display_map/suggestion_map.rs +++ b/crates/editor/src/display_map/suggestion_map.rs @@ -578,7 +578,7 @@ impl<'a> Iterator for SuggestionBufferRows<'a> { mod tests { use super::*; use crate::{display_map::fold_map::FoldMap, MultiBuffer}; - use gpui::MutableAppContext; + use gpui::AppContext; use rand::{prelude::StdRng, Rng}; use settings::Settings; use std::{ @@ -587,7 +587,7 @@ mod tests { }; #[gpui::test] - fn test_basic(cx: &mut MutableAppContext) { + fn test_basic(cx: &mut AppContext) { let buffer = MultiBuffer::build_simple("abcdefghi", cx); let buffer_edits = buffer.update(cx, |buffer, _| buffer.subscribe()); let (mut fold_map, fold_snapshot) = FoldMap::new(buffer.read(cx).snapshot(cx)); @@ -632,7 +632,7 @@ mod tests { } #[gpui::test(iterations = 100)] - fn test_random_suggestions(cx: &mut MutableAppContext, mut rng: StdRng) { + fn test_random_suggestions(cx: &mut AppContext, mut rng: StdRng) { cx.set_global(Settings::test(cx)); let operations = env::var("OPERATIONS") .map(|i| i.parse().expect("invalid `OPERATIONS` variable")) diff --git a/crates/editor/src/display_map/tab_map.rs b/crates/editor/src/display_map/tab_map.rs index 2530186bce..e44e345def 100644 --- a/crates/editor/src/display_map/tab_map.rs +++ b/crates/editor/src/display_map/tab_map.rs @@ -578,7 +578,7 @@ mod tests { use rand::{prelude::StdRng, Rng}; #[gpui::test] - fn test_expand_tabs(cx: &mut gpui::MutableAppContext) { + fn test_expand_tabs(cx: &mut gpui::AppContext) { let buffer = MultiBuffer::build_simple("", cx); let buffer_snapshot = buffer.read(cx).snapshot(cx); let (_, fold_snapshot) = FoldMap::new(buffer_snapshot.clone()); @@ -591,7 +591,7 @@ mod tests { } #[gpui::test] - fn test_long_lines(cx: &mut gpui::MutableAppContext) { + fn test_long_lines(cx: &mut gpui::AppContext) { let max_expansion_column = 12; let input = "A\tBC\tDEF\tG\tHI\tJ\tK\tL\tM"; let output = "A BC DEF G HI J K L M"; @@ -641,7 +641,7 @@ mod tests { #[gpui::test] fn test_long_lines_with_character_spanning_max_expansion_column( - cx: &mut gpui::MutableAppContext, + cx: &mut gpui::AppContext, ) { let max_expansion_column = 8; let input = "abcdefg⋯hij"; @@ -657,7 +657,7 @@ mod tests { } #[gpui::test(iterations = 100)] - fn test_random_tabs(cx: &mut gpui::MutableAppContext, mut rng: StdRng) { + fn test_random_tabs(cx: &mut gpui::AppContext, mut rng: StdRng) { let tab_size = NonZeroU32::new(rng.gen_range(1..=4)).unwrap(); let len = rng.gen_range(0..30); let buffer = if rng.gen() { diff --git a/crates/editor/src/display_map/wrap_map.rs b/crates/editor/src/display_map/wrap_map.rs index 160b61650d..e1de4fce77 100644 --- a/crates/editor/src/display_map/wrap_map.rs +++ b/crates/editor/src/display_map/wrap_map.rs @@ -7,7 +7,7 @@ use crate::MultiBufferSnapshot; use gpui::{ fonts::{FontId, HighlightStyle}, text_layout::LineWrapper, - Entity, ModelContext, ModelHandle, MutableAppContext, Task, + AppContext, Entity, ModelContext, ModelHandle, Task, }; use language::{Chunk, Point}; use lazy_static::lazy_static; @@ -79,7 +79,7 @@ impl WrapMap { font_id: FontId, font_size: f32, wrap_width: Option, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> (ModelHandle, WrapSnapshot) { let handle = cx.add_model(|cx| { let mut this = Self { diff --git a/crates/editor/src/editor.rs b/crates/editor/src/editor.rs index 5058a4353f..933d76613b 100644 --- a/crates/editor/src/editor.rs +++ b/crates/editor/src/editor.rs @@ -42,8 +42,8 @@ use gpui::{ platform::CursorStyle, serde_json::{self, json}, AnyViewHandle, AppContext, AsyncAppContext, ClipboardItem, Element, ElementBox, Entity, - ModelHandle, MouseButton, MutableAppContext, RenderContext, Subscription, Task, View, - ViewContext, ViewHandle, WeakViewHandle, + ModelHandle, MouseButton, RenderContext, Subscription, Task, View, ViewContext, ViewHandle, + WeakViewHandle, }; use highlight_matching_bracket::refresh_matching_bracket_highlights; use hover_popover::{hide_hover, HideHover, HoverState}; @@ -295,7 +295,7 @@ pub enum Direction { Next, } -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(Editor::new_file); cx.add_action(Editor::select); cx.add_action(Editor::cancel); @@ -1314,7 +1314,7 @@ impl Editor { self.buffer().read(cx).title(cx) } - pub fn snapshot(&mut self, cx: &mut MutableAppContext) -> EditorSnapshot { + pub fn snapshot(&mut self, cx: &mut AppContext) -> EditorSnapshot { EditorSnapshot { mode: self.mode, display_snapshot: self.display_map.update(cx, |map, cx| map.snapshot(cx)), @@ -6168,13 +6168,13 @@ impl Editor { }); } - pub fn longest_row(&self, cx: &mut MutableAppContext) -> u32 { + pub fn longest_row(&self, cx: &mut AppContext) -> u32 { self.display_map .update(cx, |map, cx| map.snapshot(cx)) .longest_row() } - pub fn max_point(&self, cx: &mut MutableAppContext) -> DisplayPoint { + pub fn max_point(&self, cx: &mut AppContext) -> DisplayPoint { self.display_map .update(cx, |map, cx| map.snapshot(cx)) .max_point() @@ -6194,7 +6194,7 @@ impl Editor { }); } - pub fn display_text(&self, cx: &mut MutableAppContext) -> String { + pub fn display_text(&self, cx: &mut AppContext) -> String { self.display_map .update(cx, |map, cx| map.snapshot(cx)) .text() @@ -6226,7 +6226,7 @@ impl Editor { cx.notify(); } - pub fn set_wrap_width(&self, width: Option, cx: &mut MutableAppContext) -> bool { + pub fn set_wrap_width(&self, width: Option, cx: &mut AppContext) -> bool { self.display_map .update(cx, |map, cx| map.set_wrap_width(width, cx)) } @@ -6766,7 +6766,7 @@ pub struct EditorReleased(pub WeakViewHandle); impl Entity for Editor { type Event = Event; - fn release(&mut self, cx: &mut MutableAppContext) { + fn release(&mut self, cx: &mut AppContext) { cx.emit_global(EditorReleased(self.handle.clone())); } } diff --git a/crates/editor/src/editor_tests.rs b/crates/editor/src/editor_tests.rs index 07326cb7dd..f9f3e1c4f0 100644 --- a/crates/editor/src/editor_tests.rs +++ b/crates/editor/src/editor_tests.rs @@ -28,7 +28,7 @@ use workspace::{ }; #[gpui::test] -fn test_edit_events(cx: &mut MutableAppContext) { +fn test_edit_events(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); let buffer = cx.add_model(|cx| { let mut buffer = language::Buffer::new(0, "123456", cx); @@ -155,7 +155,7 @@ fn test_edit_events(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_undo_redo_with_selection_restoration(cx: &mut MutableAppContext) { +fn test_undo_redo_with_selection_restoration(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); let mut now = Instant::now(); let buffer = cx.add_model(|cx| language::Buffer::new(0, "123456", cx)); @@ -225,7 +225,7 @@ fn test_undo_redo_with_selection_restoration(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_ime_composition(cx: &mut MutableAppContext) { +fn test_ime_composition(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); let buffer = cx.add_model(|cx| { let mut buffer = language::Buffer::new(0, "abcde", cx); @@ -327,7 +327,7 @@ fn test_ime_composition(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_selection_with_mouse(cx: &mut gpui::MutableAppContext) { +fn test_selection_with_mouse(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("aaaaaa\nbbbbbb\ncccccc\nddddddd\n", cx); @@ -392,7 +392,7 @@ fn test_selection_with_mouse(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_canceling_pending_selection(cx: &mut gpui::MutableAppContext) { +fn test_canceling_pending_selection(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("aaaaaa\nbbbbbb\ncccccc\ndddddd\n", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -424,7 +424,7 @@ fn test_canceling_pending_selection(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_clone(cx: &mut gpui::MutableAppContext) { +fn test_clone(cx: &mut gpui::AppContext) { let (text, selection_ranges) = marked_text_ranges( indoc! {" one @@ -480,7 +480,7 @@ fn test_clone(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_navigation_history(cx: &mut gpui::MutableAppContext) { +fn test_navigation_history(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); cx.set_global(DragAndDrop::::default()); use workspace::item::Item; @@ -492,7 +492,7 @@ fn test_navigation_history(cx: &mut gpui::MutableAppContext) { let handle = cx.handle(); editor.set_nav_history(Some(pane.read(cx).nav_history_for_item(&handle))); - fn pop_history(editor: &mut Editor, cx: &mut MutableAppContext) -> Option { + fn pop_history(editor: &mut Editor, cx: &mut AppContext) -> Option { editor.nav_history.as_mut().unwrap().pop_backward(cx) } @@ -590,7 +590,7 @@ fn test_navigation_history(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_cancel(cx: &mut gpui::MutableAppContext) { +fn test_cancel(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("aaaaaa\nbbbbbb\ncccccc\ndddddd\n", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -630,7 +630,7 @@ fn test_cancel(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_fold_action(cx: &mut gpui::MutableAppContext) { +fn test_fold_action(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple( &" @@ -717,7 +717,7 @@ fn test_fold_action(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_move_cursor(cx: &mut gpui::MutableAppContext) { +fn test_move_cursor(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple(&sample_text(6, 6, 'a'), cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer.clone(), cx)); @@ -793,7 +793,7 @@ fn test_move_cursor(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_move_cursor_multibyte(cx: &mut gpui::MutableAppContext) { +fn test_move_cursor_multibyte(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("ⓐⓑⓒⓓⓔ\nabcde\nαβγδε\n", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer.clone(), cx)); @@ -895,7 +895,7 @@ fn test_move_cursor_multibyte(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_move_cursor_different_line_lengths(cx: &mut gpui::MutableAppContext) { +fn test_move_cursor_different_line_lengths(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("ⓐⓑⓒⓓⓔ\nabcd\nαβγ\nabcd\nⓐⓑⓒⓓⓔ\n", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer.clone(), cx)); @@ -942,7 +942,7 @@ fn test_move_cursor_different_line_lengths(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_beginning_end_of_line(cx: &mut gpui::MutableAppContext) { +fn test_beginning_end_of_line(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("abc\n def", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -1102,7 +1102,7 @@ fn test_beginning_end_of_line(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_prev_next_word_boundary(cx: &mut gpui::MutableAppContext) { +fn test_prev_next_word_boundary(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("use std::str::{foo, bar}\n\n {baz.qux()}", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -1151,7 +1151,7 @@ fn test_prev_next_word_boundary(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_prev_next_word_bounds_with_soft_wrap(cx: &mut gpui::MutableAppContext) { +fn test_prev_next_word_bounds_with_soft_wrap(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("use one::{\n two::three::four::five\n};", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -1330,7 +1330,7 @@ async fn test_delete_to_beginning_of_line(cx: &mut gpui::TestAppContext) { } #[gpui::test] -fn test_delete_to_word_boundary(cx: &mut gpui::MutableAppContext) { +fn test_delete_to_word_boundary(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("one two three four", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer.clone(), cx)); @@ -1365,7 +1365,7 @@ fn test_delete_to_word_boundary(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_newline(cx: &mut gpui::MutableAppContext) { +fn test_newline(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("aaaa\n bbbb\n", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer.clone(), cx)); @@ -1385,7 +1385,7 @@ fn test_newline(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_newline_with_old_selections(cx: &mut gpui::MutableAppContext) { +fn test_newline_with_old_selections(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple( " @@ -1517,7 +1517,7 @@ async fn test_newline_below(cx: &mut gpui::TestAppContext) { } #[gpui::test] -fn test_insert_with_old_selections(cx: &mut gpui::MutableAppContext) { +fn test_insert_with_old_selections(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("a( X ), b( Y ), c( Z )", cx); let (_, editor) = cx.add_window(Default::default(), |cx| { @@ -1836,7 +1836,7 @@ async fn test_indent_outdent_with_hard_tabs(cx: &mut gpui::TestAppContext) { } #[gpui::test] -fn test_indent_outdent_with_excerpts(cx: &mut gpui::MutableAppContext) { +fn test_indent_outdent_with_excerpts(cx: &mut gpui::AppContext) { cx.set_global( Settings::test(cx) .with_language_defaults( @@ -2022,7 +2022,7 @@ async fn test_delete(cx: &mut gpui::TestAppContext) { } #[gpui::test] -fn test_delete_line(cx: &mut gpui::MutableAppContext) { +fn test_delete_line(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("abc\ndef\nghi\n", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -2062,7 +2062,7 @@ fn test_delete_line(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_duplicate_line(cx: &mut gpui::MutableAppContext) { +fn test_duplicate_line(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("abc\ndef\nghi\n", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -2110,7 +2110,7 @@ fn test_duplicate_line(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_move_line_up_down(cx: &mut gpui::MutableAppContext) { +fn test_move_line_up_down(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple(&sample_text(10, 5, 'a'), cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -2206,7 +2206,7 @@ fn test_move_line_up_down(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_move_line_up_down_with_blocks(cx: &mut gpui::MutableAppContext) { +fn test_move_line_up_down_with_blocks(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple(&sample_text(10, 5, 'a'), cx); let snapshot = buffer.read(cx).snapshot(cx); @@ -2230,7 +2230,7 @@ fn test_move_line_up_down_with_blocks(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_transpose(cx: &mut gpui::MutableAppContext) { +fn test_transpose(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); _ = cx @@ -2524,7 +2524,7 @@ async fn test_paste_multiline(cx: &mut gpui::TestAppContext) { } #[gpui::test] -fn test_select_all(cx: &mut gpui::MutableAppContext) { +fn test_select_all(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("abc\nde\nfgh", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -2538,7 +2538,7 @@ fn test_select_all(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_select_line(cx: &mut gpui::MutableAppContext) { +fn test_select_line(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple(&sample_text(6, 5, 'a'), cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -2582,7 +2582,7 @@ fn test_select_line(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_split_selection_into_lines(cx: &mut gpui::MutableAppContext) { +fn test_split_selection_into_lines(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple(&sample_text(9, 5, 'a'), cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -2650,7 +2650,7 @@ fn test_split_selection_into_lines(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_add_selection_above_below(cx: &mut gpui::MutableAppContext) { +fn test_add_selection_above_below(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("abc\ndefghi\n\njk\nlmno\n", cx); let (_, view) = cx.add_window(Default::default(), |cx| build_editor(buffer, cx)); @@ -4928,7 +4928,7 @@ async fn test_toggle_block_comment(cx: &mut gpui::TestAppContext) { } #[gpui::test] -fn test_editing_disjoint_excerpts(cx: &mut gpui::MutableAppContext) { +fn test_editing_disjoint_excerpts(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = cx.add_model(|cx| Buffer::new(0, sample_text(3, 4, 'a'), cx)); let multibuffer = cx.add_model(|cx| { @@ -4975,7 +4975,7 @@ fn test_editing_disjoint_excerpts(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_editing_overlapping_excerpts(cx: &mut gpui::MutableAppContext) { +fn test_editing_overlapping_excerpts(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let markers = vec![('[', ']').into(), ('(', ')').into()]; let (initial_text, mut excerpt_ranges) = marked_text_ranges_by( @@ -5048,7 +5048,7 @@ fn test_editing_overlapping_excerpts(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_refresh_selections(cx: &mut gpui::MutableAppContext) { +fn test_refresh_selections(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = cx.add_model(|cx| Buffer::new(0, sample_text(3, 4, 'a'), cx)); let mut excerpt1_id = None; @@ -5134,7 +5134,7 @@ fn test_refresh_selections(cx: &mut gpui::MutableAppContext) { } #[gpui::test] -fn test_refresh_selections_while_selecting_with_mouse(cx: &mut gpui::MutableAppContext) { +fn test_refresh_selections_while_selecting_with_mouse(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = cx.add_model(|cx| Buffer::new(0, sample_text(3, 4, 'a'), cx)); let mut excerpt1_id = None; @@ -5267,7 +5267,7 @@ async fn test_extra_newline_insertion(cx: &mut gpui::TestAppContext) { } #[gpui::test] -fn test_highlighted_ranges(cx: &mut gpui::MutableAppContext) { +fn test_highlighted_ranges(cx: &mut gpui::AppContext) { let buffer = MultiBuffer::build_simple(&sample_text(16, 8, 'a'), cx); cx.set_global(Settings::test(cx)); diff --git a/crates/editor/src/element.rs b/crates/editor/src/element.rs index c349559d7d..7e33055945 100644 --- a/crates/editor/src/element.rs +++ b/crates/editor/src/element.rs @@ -32,8 +32,8 @@ use gpui::{ platform::CursorStyle, text_layout::{self, Line, RunStyle, TextLayoutCache}, AppContext, Axis, Border, CursorRegion, Element, ElementBox, EventContext, LayoutContext, - Modifiers, MouseButton, MouseButtonEvent, MouseMovedEvent, MouseRegion, MutableAppContext, - PaintContext, Quad, SceneBuilder, SizeConstraint, ViewContext, WeakViewHandle, + Modifiers, MouseButton, MouseButtonEvent, MouseMovedEvent, MouseRegion, PaintContext, Quad, + SceneBuilder, SizeConstraint, ViewContext, WeakViewHandle, }; use itertools::Itertools; use json::json; @@ -103,14 +103,14 @@ impl EditorElement { self.view.upgrade(cx).unwrap().read(cx) } - fn update_view(&self, cx: &mut MutableAppContext, f: F) -> T + fn update_view(&self, cx: &mut AppContext, f: F) -> T where F: FnOnce(&mut Editor, &mut ViewContext) -> T, { self.view.upgrade(cx).unwrap().update(cx, f) } - fn snapshot(&self, cx: &mut MutableAppContext) -> EditorSnapshot { + fn snapshot(&self, cx: &mut AppContext) -> EditorSnapshot { self.update_view(cx, |view, cx| view.snapshot(cx)) } @@ -2522,7 +2522,7 @@ mod tests { use util::test::sample_text; #[gpui::test] - fn test_layout_line_numbers(cx: &mut gpui::MutableAppContext) { + fn test_layout_line_numbers(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple(&sample_text(6, 6, 'a'), cx); let (window_id, editor) = cx.add_window(Default::default(), |cx| { @@ -2542,7 +2542,7 @@ mod tests { } #[gpui::test] - fn test_layout_with_placeholder_text_and_blocks(cx: &mut gpui::MutableAppContext) { + fn test_layout_with_placeholder_text_and_blocks(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let buffer = MultiBuffer::build_simple("", cx); let (window_id, editor) = cx.add_window(Default::default(), |cx| { diff --git a/crates/editor/src/hover_popover.rs b/crates/editor/src/hover_popover.rs index 1a23a02efc..01cfeeb246 100644 --- a/crates/editor/src/hover_popover.rs +++ b/crates/editor/src/hover_popover.rs @@ -4,7 +4,7 @@ use gpui::{ elements::{Flex, MouseEventHandler, Padding, Text}, impl_internal_actions, platform::CursorStyle, - Axis, Element, ElementBox, ModelHandle, MouseButton, MutableAppContext, RenderContext, Task, + AppContext, Axis, Element, ElementBox, ModelHandle, MouseButton, RenderContext, Task, ViewContext, }; use language::{Bias, DiagnosticEntry, DiagnosticSeverity}; @@ -36,7 +36,7 @@ pub struct HideHover; actions!(editor, [Hover]); impl_internal_actions!(editor, [HoverAt, HideHover]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(hover); cx.add_action(hover_at); cx.add_action(hide_hover); diff --git a/crates/editor/src/items.rs b/crates/editor/src/items.rs index fa18c747f8..0f2a98dfe9 100644 --- a/crates/editor/src/items.rs +++ b/crates/editor/src/items.rs @@ -7,8 +7,8 @@ use anyhow::{anyhow, Context, Result}; use collections::HashSet; use futures::future::try_join_all; use gpui::{ - elements::*, geometry::vector::vec2f, AppContext, Entity, ModelHandle, MutableAppContext, - RenderContext, Subscription, Task, View, ViewContext, ViewHandle, WeakViewHandle, + elements::*, geometry::vector::vec2f, AppContext, Entity, ModelHandle, RenderContext, + Subscription, Task, View, ViewContext, ViewHandle, WeakViewHandle, }; use language::{ proto::serialize_anchor as serialize_text_anchor, Bias, Buffer, OffsetRangeExt, Point, @@ -48,7 +48,7 @@ impl FollowableItem for Editor { project: ModelHandle, remote_id: ViewId, state: &mut Option, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>>> { let Some(proto::view::Variant::Editor(_)) = state else { return None }; let Some(proto::view::Variant::Editor(state)) = state.take() else { unreachable!() }; @@ -769,7 +769,7 @@ impl Item for Editor { buffer: ModelHandle, workspace_id: WorkspaceId, item_id: ItemId, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { if let Some(file) = buffer.read(cx).file().and_then(|file| file.as_local()) { let path = file.abs_path(cx); @@ -1161,7 +1161,7 @@ fn path_for_file<'a>( #[cfg(test)] mod tests { use super::*; - use gpui::MutableAppContext; + use gpui::AppContext; use std::{ path::{Path, PathBuf}, sync::Arc, @@ -1169,7 +1169,7 @@ mod tests { }; #[gpui::test] - fn test_path_for_file(cx: &mut MutableAppContext) { + fn test_path_for_file(cx: &mut AppContext) { let file = TestFile { path: Path::new("").into(), full_path: PathBuf::from(""), diff --git a/crates/editor/src/link_go_to_definition.rs b/crates/editor/src/link_go_to_definition.rs index 214f1902a6..f405176d24 100644 --- a/crates/editor/src/link_go_to_definition.rs +++ b/crates/editor/src/link_go_to_definition.rs @@ -1,6 +1,6 @@ use std::ops::Range; -use gpui::{impl_internal_actions, MutableAppContext, Task, ViewContext}; +use gpui::{impl_internal_actions, AppContext, Task, ViewContext}; use language::{Bias, ToOffset}; use project::LocationLink; use settings::Settings; @@ -38,7 +38,7 @@ impl_internal_actions!( ] ); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(update_go_to_definition_link); cx.add_action(go_to_fetched_definition); cx.add_action(go_to_fetched_type_definition); diff --git a/crates/editor/src/mouse_context_menu.rs b/crates/editor/src/mouse_context_menu.rs index ba48bb2b8d..a7a37dd5a6 100644 --- a/crates/editor/src/mouse_context_menu.rs +++ b/crates/editor/src/mouse_context_menu.rs @@ -1,6 +1,6 @@ use context_menu::ContextMenuItem; use gpui::{ - elements::AnchorCorner, geometry::vector::Vector2F, impl_internal_actions, MutableAppContext, + elements::AnchorCorner, geometry::vector::Vector2F, impl_internal_actions, AppContext, ViewContext, }; @@ -17,7 +17,7 @@ pub struct DeployMouseContextMenu { impl_internal_actions!(editor, [DeployMouseContextMenu]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(deploy_context_menu); } diff --git a/crates/editor/src/movement.rs b/crates/editor/src/movement.rs index 8685f07099..284f0c94bc 100644 --- a/crates/editor/src/movement.rs +++ b/crates/editor/src/movement.rs @@ -372,9 +372,9 @@ mod tests { use settings::Settings; #[gpui::test] - fn test_previous_word_start(cx: &mut gpui::MutableAppContext) { + fn test_previous_word_start(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); - fn assert(marked_text: &str, cx: &mut gpui::MutableAppContext) { + fn assert(marked_text: &str, cx: &mut gpui::AppContext) { let (snapshot, display_points) = marked_display_snapshot(marked_text, cx); assert_eq!( previous_word_start(&snapshot, display_points[1]), @@ -399,9 +399,9 @@ mod tests { } #[gpui::test] - fn test_previous_subword_start(cx: &mut gpui::MutableAppContext) { + fn test_previous_subword_start(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); - fn assert(marked_text: &str, cx: &mut gpui::MutableAppContext) { + fn assert(marked_text: &str, cx: &mut gpui::AppContext) { let (snapshot, display_points) = marked_display_snapshot(marked_text, cx); assert_eq!( previous_subword_start(&snapshot, display_points[1]), @@ -433,11 +433,11 @@ mod tests { } #[gpui::test] - fn test_find_preceding_boundary(cx: &mut gpui::MutableAppContext) { + fn test_find_preceding_boundary(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); fn assert( marked_text: &str, - cx: &mut gpui::MutableAppContext, + cx: &mut gpui::AppContext, is_boundary: impl FnMut(char, char) -> bool, ) { let (snapshot, display_points) = marked_display_snapshot(marked_text, cx); @@ -465,9 +465,9 @@ mod tests { } #[gpui::test] - fn test_next_word_end(cx: &mut gpui::MutableAppContext) { + fn test_next_word_end(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); - fn assert(marked_text: &str, cx: &mut gpui::MutableAppContext) { + fn assert(marked_text: &str, cx: &mut gpui::AppContext) { let (snapshot, display_points) = marked_display_snapshot(marked_text, cx); assert_eq!( next_word_end(&snapshot, display_points[0]), @@ -489,9 +489,9 @@ mod tests { } #[gpui::test] - fn test_next_subword_end(cx: &mut gpui::MutableAppContext) { + fn test_next_subword_end(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); - fn assert(marked_text: &str, cx: &mut gpui::MutableAppContext) { + fn assert(marked_text: &str, cx: &mut gpui::AppContext) { let (snapshot, display_points) = marked_display_snapshot(marked_text, cx); assert_eq!( next_subword_end(&snapshot, display_points[0]), @@ -522,11 +522,11 @@ mod tests { } #[gpui::test] - fn test_find_boundary(cx: &mut gpui::MutableAppContext) { + fn test_find_boundary(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); fn assert( marked_text: &str, - cx: &mut gpui::MutableAppContext, + cx: &mut gpui::AppContext, is_boundary: impl FnMut(char, char) -> bool, ) { let (snapshot, display_points) = marked_display_snapshot(marked_text, cx); @@ -554,9 +554,9 @@ mod tests { } #[gpui::test] - fn test_surrounding_word(cx: &mut gpui::MutableAppContext) { + fn test_surrounding_word(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); - fn assert(marked_text: &str, cx: &mut gpui::MutableAppContext) { + fn assert(marked_text: &str, cx: &mut gpui::AppContext) { let (snapshot, display_points) = marked_display_snapshot(marked_text, cx); assert_eq!( surrounding_word(&snapshot, display_points[1]), @@ -575,7 +575,7 @@ mod tests { } #[gpui::test] - fn test_move_up_and_down_with_excerpts(cx: &mut gpui::MutableAppContext) { + fn test_move_up_and_down_with_excerpts(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); let family_id = cx .font_cache() diff --git a/crates/editor/src/multi_buffer.rs b/crates/editor/src/multi_buffer.rs index ed4d297c51..bec9d5967c 100644 --- a/crates/editor/src/multi_buffer.rs +++ b/crates/editor/src/multi_buffer.rs @@ -1489,15 +1489,12 @@ impl MultiBuffer { #[cfg(any(test, feature = "test-support"))] impl MultiBuffer { - pub fn build_simple(text: &str, cx: &mut gpui::MutableAppContext) -> ModelHandle { + pub fn build_simple(text: &str, cx: &mut gpui::AppContext) -> ModelHandle { let buffer = cx.add_model(|cx| Buffer::new(0, text, cx)); cx.add_model(|cx| Self::singleton(buffer, cx)) } - pub fn build_random( - rng: &mut impl rand::Rng, - cx: &mut gpui::MutableAppContext, - ) -> ModelHandle { + pub fn build_random(rng: &mut impl rand::Rng, cx: &mut gpui::AppContext) -> ModelHandle { cx.add_model(|cx| { let mut multibuffer = MultiBuffer::new(0); let mutation_count = rng.gen_range(1..=5); @@ -3729,7 +3726,7 @@ where mod tests { use super::*; use futures::StreamExt; - use gpui::{MutableAppContext, TestAppContext}; + use gpui::{AppContext, TestAppContext}; use language::{Buffer, Rope}; use rand::prelude::*; use settings::Settings; @@ -3739,7 +3736,7 @@ mod tests { use util::test::sample_text; #[gpui::test] - fn test_singleton(cx: &mut MutableAppContext) { + fn test_singleton(cx: &mut AppContext) { let buffer = cx.add_model(|cx| Buffer::new(0, sample_text(6, 6, 'a'), cx)); let multibuffer = cx.add_model(|cx| MultiBuffer::singleton(buffer.clone(), cx)); @@ -3766,7 +3763,7 @@ mod tests { } #[gpui::test] - fn test_remote(cx: &mut MutableAppContext) { + fn test_remote(cx: &mut AppContext) { let host_buffer = cx.add_model(|cx| Buffer::new(0, "a", cx)); let guest_buffer = cx.add_model(|cx| { let state = host_buffer.read(cx).to_proto(); @@ -3797,7 +3794,7 @@ mod tests { } #[gpui::test] - fn test_excerpt_boundaries_and_clipping(cx: &mut MutableAppContext) { + fn test_excerpt_boundaries_and_clipping(cx: &mut AppContext) { let buffer_1 = cx.add_model(|cx| Buffer::new(0, sample_text(6, 6, 'a'), cx)); let buffer_2 = cx.add_model(|cx| Buffer::new(0, sample_text(6, 6, 'g'), cx)); let multibuffer = cx.add_model(|_| MultiBuffer::new(0)); @@ -4021,7 +4018,7 @@ mod tests { } #[gpui::test] - fn test_excerpt_events(cx: &mut MutableAppContext) { + fn test_excerpt_events(cx: &mut AppContext) { let buffer_1 = cx.add_model(|cx| Buffer::new(0, sample_text(10, 3, 'a'), cx)); let buffer_2 = cx.add_model(|cx| Buffer::new(0, sample_text(10, 3, 'm'), cx)); @@ -4098,7 +4095,7 @@ mod tests { } #[gpui::test] - fn test_push_excerpts_with_context_lines(cx: &mut MutableAppContext) { + fn test_push_excerpts_with_context_lines(cx: &mut AppContext) { let buffer = cx.add_model(|cx| Buffer::new(0, sample_text(20, 3, 'a'), cx)); let multibuffer = cx.add_model(|_| MultiBuffer::new(0)); let anchor_ranges = multibuffer.update(cx, |multibuffer, cx| { @@ -4172,7 +4169,7 @@ mod tests { } #[gpui::test] - fn test_empty_multibuffer(cx: &mut MutableAppContext) { + fn test_empty_multibuffer(cx: &mut AppContext) { let multibuffer = cx.add_model(|_| MultiBuffer::new(0)); let snapshot = multibuffer.read(cx).snapshot(cx); @@ -4182,7 +4179,7 @@ mod tests { } #[gpui::test] - fn test_singleton_multibuffer_anchors(cx: &mut MutableAppContext) { + fn test_singleton_multibuffer_anchors(cx: &mut AppContext) { let buffer = cx.add_model(|cx| Buffer::new(0, "abcd", cx)); let multibuffer = cx.add_model(|cx| MultiBuffer::singleton(buffer.clone(), cx)); let old_snapshot = multibuffer.read(cx).snapshot(cx); @@ -4202,7 +4199,7 @@ mod tests { } #[gpui::test] - fn test_multibuffer_anchors(cx: &mut MutableAppContext) { + fn test_multibuffer_anchors(cx: &mut AppContext) { let buffer_1 = cx.add_model(|cx| Buffer::new(0, "abcd", cx)); let buffer_2 = cx.add_model(|cx| Buffer::new(0, "efghi", cx)); let multibuffer = cx.add_model(|cx| { @@ -4260,7 +4257,7 @@ mod tests { } #[gpui::test] - fn test_resolving_anchors_after_replacing_their_excerpts(cx: &mut MutableAppContext) { + fn test_resolving_anchors_after_replacing_their_excerpts(cx: &mut AppContext) { let buffer_1 = cx.add_model(|cx| Buffer::new(0, "abcd", cx)); let buffer_2 = cx.add_model(|cx| Buffer::new(0, "ABCDEFGHIJKLMNOP", cx)); let multibuffer = cx.add_model(|_| MultiBuffer::new(0)); @@ -4563,7 +4560,7 @@ mod tests { } #[gpui::test(iterations = 100)] - fn test_random_multibuffer(cx: &mut MutableAppContext, mut rng: StdRng) { + fn test_random_multibuffer(cx: &mut AppContext, mut rng: StdRng) { let operations = env::var("OPERATIONS") .map(|i| i.parse().expect("invalid `OPERATIONS` variable")) .unwrap_or(10); @@ -4980,7 +4977,7 @@ mod tests { } #[gpui::test] - fn test_history(cx: &mut MutableAppContext) { + fn test_history(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); let buffer_1 = cx.add_model(|cx| Buffer::new(0, "1234", cx)); let buffer_2 = cx.add_model(|cx| Buffer::new(0, "5678", cx)); diff --git a/crates/editor/src/scroll.rs b/crates/editor/src/scroll.rs index 1b3fd96dbd..df6188d58e 100644 --- a/crates/editor/src/scroll.rs +++ b/crates/editor/src/scroll.rs @@ -9,7 +9,7 @@ use std::{ use gpui::{ geometry::vector::{vec2f, Vector2F}, - Axis, MutableAppContext, Task, ViewContext, + Axis, AppContext, Task, ViewContext, }; use language::{Bias, Point}; use util::ResultExt; @@ -369,7 +369,7 @@ impl Editor { /// Ordering::Equal => on screen /// Ordering::Less => above the screen /// Ordering::Greater => below the screen - pub fn newest_selection_on_screen(&self, cx: &mut MutableAppContext) -> Ordering { + pub fn newest_selection_on_screen(&self, cx: &mut AppContext) -> Ordering { let snapshot = self.display_map.update(cx, |map, cx| map.snapshot(cx)); let newest_head = self .selections diff --git a/crates/editor/src/scroll/actions.rs b/crates/editor/src/scroll/actions.rs index fb3dec0129..80b8c1f261 100644 --- a/crates/editor/src/scroll/actions.rs +++ b/crates/editor/src/scroll/actions.rs @@ -1,5 +1,5 @@ use gpui::{ - actions, geometry::vector::Vector2F, impl_internal_actions, Axis, MutableAppContext, + actions, geometry::vector::Vector2F, impl_internal_actions, Axis, AppContext, ViewContext, }; use language::Bias; @@ -32,7 +32,7 @@ pub struct Scroll { impl_internal_actions!(editor, [Scroll]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(Editor::next_screen); cx.add_action(Editor::scroll); cx.add_action(Editor::scroll_cursor_top); diff --git a/crates/editor/src/selections_collection.rs b/crates/editor/src/selections_collection.rs index f3ce89adc5..4c32b543b2 100644 --- a/crates/editor/src/selections_collection.rs +++ b/crates/editor/src/selections_collection.rs @@ -6,7 +6,7 @@ use std::{ }; use collections::HashMap; -use gpui::{AppContext, ModelHandle, MutableAppContext}; +use gpui::{AppContext, ModelHandle}; use itertools::Itertools; use language::{Bias, Point, Selection, SelectionGoal, TextDimension, ToPoint}; use util::post_inc; @@ -53,7 +53,7 @@ impl SelectionsCollection { } } - fn display_map(&self, cx: &mut MutableAppContext) -> DisplaySnapshot { + fn display_map(&self, cx: &mut AppContext) -> DisplaySnapshot { self.display_map.update(cx, |map, cx| map.snapshot(cx)) } @@ -136,7 +136,7 @@ impl SelectionsCollection { } // Returns all of the selections, adjusted to take into account the selection line_mode - pub fn all_adjusted(&self, cx: &mut MutableAppContext) -> Vec> { + pub fn all_adjusted(&self, cx: &mut AppContext) -> Vec> { let mut selections = self.all::(cx); if self.line_mode { let map = self.display_map(cx); @@ -151,7 +151,7 @@ impl SelectionsCollection { pub fn all_adjusted_display( &self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> (DisplaySnapshot, Vec>) { if self.line_mode { let selections = self.all::(cx); @@ -198,7 +198,7 @@ impl SelectionsCollection { pub fn all_display( &self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> (DisplaySnapshot, Vec>) { let display_map = self.display_map(cx); let selections = self @@ -224,7 +224,7 @@ impl SelectionsCollection { resolve(self.newest_anchor(), &self.buffer(cx)) } - pub fn newest_display(&self, cx: &mut MutableAppContext) -> Selection { + pub fn newest_display(&self, cx: &mut AppContext) -> Selection { let display_map = self.display_map(cx); let selection = self .newest_anchor() @@ -279,7 +279,7 @@ impl SelectionsCollection { } #[cfg(any(test, feature = "test-support"))] - pub fn display_ranges(&self, cx: &mut MutableAppContext) -> Vec> { + pub fn display_ranges(&self, cx: &mut AppContext) -> Vec> { let display_map = self.display_map(cx); self.disjoint_anchors() .iter() @@ -324,7 +324,7 @@ impl SelectionsCollection { pub(crate) fn change_with( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, change: impl FnOnce(&mut MutableSelectionsCollection) -> R, ) -> (bool, R) { let mut mutable_collection = MutableSelectionsCollection { @@ -345,7 +345,7 @@ impl SelectionsCollection { pub struct MutableSelectionsCollection<'a> { collection: &'a mut SelectionsCollection, selections_changed: bool, - cx: &'a mut MutableAppContext, + cx: &'a mut AppContext, } impl<'a> MutableSelectionsCollection<'a> { diff --git a/crates/editor/src/test.rs b/crates/editor/src/test.rs index 9d0c1c7f1a..8a3e08bea5 100644 --- a/crates/editor/src/test.rs +++ b/crates/editor/src/test.rs @@ -21,7 +21,7 @@ fn init_logger() { // Returns a snapshot from text containing '|' character markers with the markers removed, and DisplayPoints for each one. pub fn marked_display_snapshot( text: &str, - cx: &mut gpui::MutableAppContext, + cx: &mut gpui::AppContext, ) -> (DisplaySnapshot, Vec) { let (unmarked_text, markers) = marked_text_offsets(text); diff --git a/crates/feedback/src/feedback.rs b/crates/feedback/src/feedback.rs index cd72cdf950..40bebb0170 100644 --- a/crates/feedback/src/feedback.rs +++ b/crates/feedback/src/feedback.rs @@ -6,7 +6,7 @@ pub mod submit_feedback_button; use std::sync::Arc; mod system_specs; -use gpui::{actions, impl_actions, ClipboardItem, MutableAppContext, PromptLevel, ViewContext}; +use gpui::{actions, impl_actions, ClipboardItem, AppContext, PromptLevel, ViewContext}; use serde::Deserialize; use system_specs::SystemSpecs; use workspace::{AppState, Workspace}; @@ -28,7 +28,7 @@ actions!( ] ); -pub fn init(app_state: Arc, cx: &mut MutableAppContext) { +pub fn init(app_state: Arc, cx: &mut AppContext) { let system_specs = SystemSpecs::new(&cx); let system_specs_text = system_specs.to_string(); diff --git a/crates/feedback/src/feedback_editor.rs b/crates/feedback/src/feedback_editor.rs index 62ce0caa7d..217a69eedf 100644 --- a/crates/feedback/src/feedback_editor.rs +++ b/crates/feedback/src/feedback_editor.rs @@ -11,8 +11,8 @@ use futures::AsyncReadExt; use gpui::{ actions, elements::{ChildView, Flex, Label, ParentElement, Svg}, - serde_json, AnyViewHandle, AppContext, Element, ElementBox, Entity, ModelHandle, - MutableAppContext, PromptLevel, RenderContext, Task, View, ViewContext, ViewHandle, + serde_json, AnyViewHandle, AppContext, Element, ElementBox, Entity, ModelHandle, PromptLevel, + RenderContext, Task, View, ViewContext, ViewHandle, }; use isahc::Request; use language::Buffer; @@ -35,7 +35,7 @@ const FEEDBACK_SUBMISSION_ERROR_TEXT: &str = actions!(feedback, [GiveFeedback, SubmitFeedback]); -pub fn init(system_specs: SystemSpecs, app_state: Arc, cx: &mut MutableAppContext) { +pub fn init(system_specs: SystemSpecs, app_state: Arc, cx: &mut AppContext) { cx.add_action({ move |workspace: &mut Workspace, _: &GiveFeedback, cx: &mut ViewContext| { FeedbackEditor::deploy(system_specs.clone(), workspace, app_state.clone(), cx); diff --git a/crates/file_finder/src/file_finder.rs b/crates/file_finder/src/file_finder.rs index 1160cb6174..8e3490902e 100644 --- a/crates/file_finder/src/file_finder.rs +++ b/crates/file_finder/src/file_finder.rs @@ -1,7 +1,7 @@ use fuzzy::PathMatch; use gpui::{ actions, elements::*, AnyViewHandle, AppContext, Entity, ModelHandle, MouseState, - MutableAppContext, RenderContext, Task, View, ViewContext, ViewHandle, + RenderContext, Task, View, ViewContext, ViewHandle, }; use picker::{Picker, PickerDelegate}; use project::{PathMatchCandidateSet, Project, ProjectPath, WorktreeId}; @@ -31,7 +31,7 @@ pub struct FileFinder { actions!(file_finder, [Toggle]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(FileFinder::toggle); Picker::::init(cx); } diff --git a/crates/go_to_line/src/go_to_line.rs b/crates/go_to_line/src/go_to_line.rs index 32c7d3c810..981f222744 100644 --- a/crates/go_to_line/src/go_to_line.rs +++ b/crates/go_to_line/src/go_to_line.rs @@ -2,8 +2,8 @@ use std::sync::Arc; use editor::{display_map::ToDisplayPoint, scroll::autoscroll::Autoscroll, DisplayPoint, Editor}; use gpui::{ - actions, elements::*, geometry::vector::Vector2F, AnyViewHandle, Axis, Entity, - MutableAppContext, RenderContext, View, ViewContext, ViewHandle, + actions, elements::*, geometry::vector::Vector2F, AnyViewHandle, AppContext, Axis, Entity, + RenderContext, View, ViewContext, ViewHandle, }; use menu::{Cancel, Confirm}; use settings::Settings; @@ -12,7 +12,7 @@ use workspace::Workspace; actions!(go_to_line, [Toggle]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(GoToLine::toggle); cx.add_action(GoToLine::confirm); cx.add_action(GoToLine::cancel); @@ -140,7 +140,7 @@ impl GoToLine { impl Entity for GoToLine { type Event = Event; - fn release(&mut self, cx: &mut MutableAppContext) { + fn release(&mut self, cx: &mut AppContext) { let scroll_position = self.prev_scroll_position.take(); self.active_editor.update(cx, |editor, cx| { editor.highlight_rows(None); diff --git a/crates/gpui/src/app.rs b/crates/gpui/src/app.rs index 0923c6993e..a70a9a5985 100644 --- a/crates/gpui/src/app.rs +++ b/crates/gpui/src/app.rs @@ -57,10 +57,10 @@ use self::ref_counts::RefCounts; pub trait Entity: 'static { type Event; - fn release(&mut self, _: &mut MutableAppContext) {} + fn release(&mut self, _: &mut AppContext) {} fn app_will_quit( &mut self, - _: &mut MutableAppContext, + _: &mut AppContext, ) -> Option>>> { None } @@ -182,17 +182,17 @@ pub trait UpdateView { } #[derive(Clone)] -pub struct App(Rc>); +pub struct App(Rc>); #[derive(Clone)] -pub struct AsyncAppContext(Rc>); +pub struct AsyncAppContext(Rc>); impl App { pub fn new(asset_source: impl AssetSource) -> Result { let platform = platform::current::platform(); let foreground = Rc::new(executor::Foreground::platform(platform.dispatcher())?); let foreground_platform = platform::current::foreground_platform(foreground.clone()); - let app = Self(Rc::new(RefCell::new(MutableAppContext::new( + let app = Self(Rc::new(RefCell::new(AppContext::new( foreground, Arc::new(executor::Background::new()), platform.clone(), @@ -220,7 +220,7 @@ impl App { pub fn on_become_active(self, mut callback: F) -> Self where - F: 'static + FnMut(&mut MutableAppContext), + F: 'static + FnMut(&mut AppContext), { let cx = self.0.clone(); self.0 @@ -232,7 +232,7 @@ impl App { pub fn on_resign_active(self, mut callback: F) -> Self where - F: 'static + FnMut(&mut MutableAppContext), + F: 'static + FnMut(&mut AppContext), { let cx = self.0.clone(); self.0 @@ -244,7 +244,7 @@ impl App { pub fn on_quit(&mut self, mut callback: F) -> &mut Self where - F: 'static + FnMut(&mut MutableAppContext), + F: 'static + FnMut(&mut AppContext), { let cx = self.0.clone(); self.0 @@ -257,7 +257,7 @@ impl App { /// Handle the application being re-activated when no windows are open. pub fn on_reopen(&mut self, mut callback: F) -> &mut Self where - F: 'static + FnMut(&mut MutableAppContext), + F: 'static + FnMut(&mut AppContext), { let cx = self.0.clone(); self.0 @@ -269,7 +269,7 @@ impl App { pub fn on_event(&mut self, mut callback: F) -> &mut Self where - F: 'static + FnMut(Event, &mut MutableAppContext) -> bool, + F: 'static + FnMut(Event, &mut AppContext) -> bool, { let cx = self.0.clone(); self.0 @@ -283,7 +283,7 @@ impl App { pub fn on_open_urls(&mut self, mut callback: F) -> &mut Self where - F: 'static + FnMut(Vec, &mut MutableAppContext), + F: 'static + FnMut(Vec, &mut AppContext), { let cx = self.0.clone(); self.0 @@ -295,7 +295,7 @@ impl App { pub fn run(self, on_finish_launching: F) where - F: 'static + FnOnce(&mut MutableAppContext), + F: 'static + FnOnce(&mut AppContext), { let platform = self.0.borrow().foreground_platform.clone(); platform.run(Box::new(move || { @@ -307,14 +307,14 @@ impl App { } pub fn platform(&self) -> Arc { - self.0.borrow().platform() + self.0.borrow().platform.clone() } pub fn font_cache(&self) -> Arc { - self.0.borrow().cx.font_cache.clone() + self.0.borrow().font_cache.clone() } - fn update T>(&mut self, callback: F) -> T { + fn update T>(&mut self, callback: F) -> T { let mut state = self.0.borrow_mut(); let result = state.update(callback); state.pending_notifications.clear(); @@ -333,10 +333,10 @@ impl AsyncAppContext { } pub fn read T>(&self, callback: F) -> T { - callback(self.0.borrow().as_ref()) + callback(&*self.0.borrow()) } - pub fn update T>(&mut self, callback: F) -> T { + pub fn update T>(&mut self, callback: F) -> T { self.0.borrow_mut().update(callback) } @@ -379,7 +379,7 @@ impl AsyncAppContext { } pub fn platform(&self) -> Arc { - self.0.borrow().platform() + self.0.borrow().platform().clone() } pub fn foreground(&self) -> Rc { @@ -387,7 +387,7 @@ impl AsyncAppContext { } pub fn background(&self) -> Arc { - self.0.borrow().cx.background.clone() + self.0.borrow().background.clone() } } @@ -435,7 +435,7 @@ impl ReadModelWith for AsyncAppContext { read: &mut dyn FnMut(&E, &AppContext) -> T, ) -> T { let cx = self.0.borrow(); - let cx = cx.as_ref(); + let cx = &*cx; read(handle.read(cx), cx) } } @@ -463,37 +463,45 @@ impl ReadViewWith for AsyncAppContext { V: View, { let cx = self.0.borrow(); - let cx = cx.as_ref(); + let cx = &*cx; read(handle.read(cx), cx) } } -type ActionCallback = - dyn FnMut(&mut dyn AnyView, &dyn Action, &mut MutableAppContext, usize, usize); -type GlobalActionCallback = dyn FnMut(&dyn Action, &mut MutableAppContext); +type ActionCallback = dyn FnMut(&mut dyn AnyView, &dyn Action, &mut AppContext, usize, usize); +type GlobalActionCallback = dyn FnMut(&dyn Action, &mut AppContext); -type SubscriptionCallback = Box bool>; -type GlobalSubscriptionCallback = Box; -type ObservationCallback = Box bool>; -type GlobalObservationCallback = Box; -type FocusObservationCallback = Box bool>; -type ReleaseObservationCallback = Box; -type ActionObservationCallback = Box; -type WindowActivationCallback = Box bool>; -type WindowFullscreenCallback = Box bool>; -type WindowBoundsCallback = Box bool>; -type KeystrokeCallback = Box< - dyn FnMut(&Keystroke, &MatchResult, Option<&Box>, &mut MutableAppContext) -> bool, ->; -type ActiveLabeledTasksCallback = Box bool>; +type SubscriptionCallback = Box bool>; +type GlobalSubscriptionCallback = Box; +type ObservationCallback = Box bool>; +type GlobalObservationCallback = Box; +type FocusObservationCallback = Box bool>; +type ReleaseObservationCallback = Box; +type ActionObservationCallback = Box; +type WindowActivationCallback = Box bool>; +type WindowFullscreenCallback = Box bool>; +type WindowBoundsCallback = Box bool>; +type KeystrokeCallback = + Box>, &mut AppContext) -> bool>; +type ActiveLabeledTasksCallback = Box bool>; type DeserializeActionCallback = fn(json: &str) -> anyhow::Result>; -type WindowShouldCloseSubscriptionCallback = Box bool>; +type WindowShouldCloseSubscriptionCallback = Box bool>; + +pub struct AppContext { + models: HashMap>, + views: HashMap<(usize, usize), Box>, + pub(crate) parents: HashMap<(usize, usize), ParentId>, + windows: HashMap, + globals: HashMap>, + element_states: HashMap>, + background: Arc, + ref_counts: Arc>, + font_cache: Arc, + platform: Arc, -pub struct MutableAppContext { weak_self: Option>>, foreground_platform: Rc, assets: Arc, - cx: AppContext, action_deserializers: HashMap<&'static str, (TypeId, DeserializeActionCallback)>, capture_actions: HashMap>>>, // Entity Types -> { Action Types -> Action Handlers } @@ -533,7 +541,7 @@ pub struct MutableAppContext { active_labeled_tasks: BTreeMap, } -impl MutableAppContext { +impl AppContext { fn new( foreground: Rc, background: Arc, @@ -544,21 +552,20 @@ impl MutableAppContext { asset_source: impl AssetSource, ) -> Self { Self { + models: Default::default(), + views: Default::default(), + parents: Default::default(), + windows: Default::default(), + globals: Default::default(), + element_states: Default::default(), + ref_counts: Arc::new(Mutex::new(ref_counts)), + background, + font_cache, + platform, + weak_self: None, foreground_platform, assets: Arc::new(AssetCache::new(asset_source)), - cx: AppContext { - models: Default::default(), - views: Default::default(), - parents: Default::default(), - windows: Default::default(), - globals: Default::default(), - element_states: Default::default(), - ref_counts: Arc::new(Mutex::new(ref_counts)), - background, - font_cache, - platform, - }, action_deserializers: Default::default(), capture_actions: Default::default(), actions: Default::default(), @@ -602,15 +609,15 @@ impl MutableAppContext { self.update(|cx| { for model_id in cx.models.keys().copied().collect::>() { - let mut model = cx.cx.models.remove(&model_id).unwrap(); + let mut model = cx.models.remove(&model_id).unwrap(); futures.extend(model.app_will_quit(cx)); - cx.cx.models.insert(model_id, model); + cx.models.insert(model_id, model); } for view_id in cx.views.keys().copied().collect::>() { - let mut view = cx.cx.views.remove(&view_id).unwrap(); + let mut view = cx.views.remove(&view_id).unwrap(); futures.extend(view.app_will_quit(cx)); - cx.cx.views.insert(view_id, view); + cx.views.insert(view_id, view); } }); @@ -627,28 +634,16 @@ impl MutableAppContext { } pub fn remove_all_windows(&mut self) { - for (window_id, _) in self.cx.windows.drain() { + for (window_id, _) in self.windows.drain() { self.presenters_and_platform_windows.remove(&window_id); } self.flush_effects(); } - pub fn platform(&self) -> Arc { - self.cx.platform.clone() - } - - pub fn font_cache(&self) -> &Arc { - &self.cx.font_cache - } - pub fn foreground(&self) -> &Rc { &self.foreground } - pub fn background(&self) -> &Arc { - &self.cx.background - } - pub fn debug_elements(&self, window_id: usize) -> Option { self.presenters_and_platform_windows .get(&window_id) @@ -696,7 +691,7 @@ impl MutableAppContext { let handler = Box::new( move |view: &mut dyn AnyView, action: &dyn Action, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize| { let action = action.as_any().downcast_ref().unwrap(); @@ -745,9 +740,9 @@ impl MutableAppContext { pub fn add_global_action(&mut self, mut handler: F) where A: Action, - F: 'static + FnMut(&A, &mut MutableAppContext), + F: 'static + FnMut(&A, &mut AppContext), { - let handler = Box::new(move |action: &dyn Action, cx: &mut MutableAppContext| { + let handler = Box::new(move |action: &dyn Action, cx: &mut AppContext| { let action = action.as_any().downcast_ref().unwrap(); handler(action, cx); }); @@ -783,7 +778,7 @@ impl MutableAppContext { } pub fn window_ids(&self) -> impl Iterator + '_ { - self.cx.windows.keys().copied() + self.windows.keys().copied() } pub fn activate_window(&self, window_id: usize) { @@ -792,23 +787,14 @@ impl MutableAppContext { } } - pub fn root_view(&self, window_id: usize) -> Option> { - self.cx - .windows - .get(&window_id) - .and_then(|window| window.root_view.clone().downcast::()) - } - pub fn window_is_active(&self, window_id: usize) -> bool { - self.cx - .windows + self.windows .get(&window_id) .map_or(false, |window| window.is_active) } pub fn window_is_fullscreen(&self, window_id: usize) -> bool { - self.cx - .windows + self.windows .get(&window_id) .map_or(false, |window| window.is_fullscreen) } @@ -833,12 +819,11 @@ impl MutableAppContext { let window_id = params.window_id; let view_id = params.view_id; let mut view = self - .cx .views .remove(&(window_id, view_id)) .ok_or_else(|| anyhow!("view not found"))?; let element = view.render(params, self); - self.cx.views.insert((window_id, view_id), view); + self.views.insert((window_id, view_id), view); Ok(element) } @@ -1005,7 +990,7 @@ impl MutableAppContext { entity_id: handle.id(), subscription_id, callback: Box::new(move |payload, cx| { - if let Some(emitter) = H::upgrade_from(&emitter, cx.as_ref()) { + if let Some(emitter) = H::upgrade_from(&emitter, cx) { let payload = payload.downcast_ref().expect("downcast is type safe"); callback(emitter, payload, cx) } else { @@ -1042,7 +1027,7 @@ impl MutableAppContext { fn observe_focus(&mut self, handle: &ViewHandle, mut callback: F) -> Subscription where - F: 'static + FnMut(ViewHandle, bool, &mut MutableAppContext) -> bool, + F: 'static + FnMut(ViewHandle, bool, &mut AppContext) -> bool, V: View, { let subscription_id = post_inc(&mut self.next_subscription_id); @@ -1066,7 +1051,7 @@ impl MutableAppContext { pub fn observe_global(&mut self, mut observe: F) -> Subscription where G: Any, - F: 'static + FnMut(&mut MutableAppContext), + F: 'static + FnMut(&mut AppContext), { let type_id = TypeId::of::(); let id = post_inc(&mut self.next_subscription_id); @@ -1074,7 +1059,7 @@ impl MutableAppContext { self.global_observations.add_callback( type_id, id, - Box::new(move |cx: &mut MutableAppContext| observe(cx)), + Box::new(move |cx: &mut AppContext| observe(cx)), ); Subscription::GlobalObservation(self.global_observations.subscribe(type_id, id)) } @@ -1082,7 +1067,7 @@ impl MutableAppContext { pub fn observe_default_global(&mut self, observe: F) -> Subscription where G: Any + Default, - F: 'static + FnMut(&mut MutableAppContext), + F: 'static + FnMut(&mut AppContext), { if !self.has_global::() { self.set_global(G::default()); @@ -1114,7 +1099,7 @@ impl MutableAppContext { pub fn observe_actions(&mut self, callback: F) -> Subscription where - F: 'static + FnMut(TypeId, &mut MutableAppContext), + F: 'static + FnMut(TypeId, &mut AppContext), { let subscription_id = post_inc(&mut self.next_subscription_id); self.action_dispatch_observations @@ -1127,7 +1112,7 @@ impl MutableAppContext { fn observe_window_activation(&mut self, window_id: usize, callback: F) -> Subscription where - F: 'static + FnMut(bool, &mut MutableAppContext) -> bool, + F: 'static + FnMut(bool, &mut AppContext) -> bool, { let subscription_id = post_inc(&mut self.next_subscription_id); self.pending_effects @@ -1144,7 +1129,7 @@ impl MutableAppContext { fn observe_fullscreen(&mut self, window_id: usize, callback: F) -> Subscription where - F: 'static + FnMut(bool, &mut MutableAppContext) -> bool, + F: 'static + FnMut(bool, &mut AppContext) -> bool, { let subscription_id = post_inc(&mut self.next_subscription_id); self.pending_effects @@ -1161,7 +1146,7 @@ impl MutableAppContext { fn observe_window_bounds(&mut self, window_id: usize, callback: F) -> Subscription where - F: 'static + FnMut(WindowBounds, Uuid, &mut MutableAppContext) -> bool, + F: 'static + FnMut(WindowBounds, Uuid, &mut AppContext) -> bool, { let subscription_id = post_inc(&mut self.next_subscription_id); self.pending_effects @@ -1179,12 +1164,7 @@ impl MutableAppContext { pub fn observe_keystrokes(&mut self, window_id: usize, callback: F) -> Subscription where F: 'static - + FnMut( - &Keystroke, - &MatchResult, - Option<&Box>, - &mut MutableAppContext, - ) -> bool, + + FnMut(&Keystroke, &MatchResult, Option<&Box>, &mut AppContext) -> bool, { let subscription_id = post_inc(&mut self.next_subscription_id); self.keystroke_observations @@ -1197,7 +1177,7 @@ impl MutableAppContext { pub fn observe_active_labeled_tasks(&mut self, callback: F) -> Subscription where - F: 'static + FnMut(&mut MutableAppContext) -> bool, + F: 'static + FnMut(&mut AppContext) -> bool, { let subscription_id = post_inc(&mut self.next_subscription_id); self.active_labeled_task_observations @@ -1208,14 +1188,14 @@ impl MutableAppContext { ) } - pub fn defer(&mut self, callback: impl 'static + FnOnce(&mut MutableAppContext)) { + pub fn defer(&mut self, callback: impl 'static + FnOnce(&mut AppContext)) { self.pending_effects.push_back(Effect::Deferred { callback: Box::new(callback), after_window_update: false, }) } - pub fn after_window_update(&mut self, callback: impl 'static + FnOnce(&mut MutableAppContext)) { + pub fn after_window_update(&mut self, callback: impl 'static + FnOnce(&mut AppContext)) { self.pending_effects.push_back(Effect::Deferred { callback: Box::new(callback), after_window_update: true, @@ -1343,7 +1323,7 @@ impl MutableAppContext { pub fn is_action_available(&self, action: &dyn Action) -> bool { let action_type = action.as_any().type_id(); - if let Some(window_id) = self.cx.platform.main_window_id() { + if let Some(window_id) = self.platform.main_window_id() { if let Some(focused_view_id) = self.focused_view_id(window_id) { for view_id in self.ancestors(window_id, focused_view_id) { if let Some(view) = self.views.get(&(window_id, view_id)) { @@ -1368,7 +1348,7 @@ impl MutableAppContext { &mut self, window_id: usize, view_id: usize, - mut visit: impl FnMut(usize, bool, &mut MutableAppContext) -> bool, + mut visit: impl FnMut(usize, bool, &mut AppContext) -> bool, ) -> bool { // List of view ids from the leaf to the root of the window let path = self.ancestors(window_id, view_id).collect::>(); @@ -1431,9 +1411,9 @@ impl MutableAppContext { .ancestors(window_id, focused_view_id) .collect::>() { - if let Some(mut view) = self.cx.views.remove(&(window_id, view_id)) { + if let Some(mut view) = self.views.remove(&(window_id, view_id)) { let handled = view.key_down(event, self, window_id, view_id); - self.cx.views.insert((window_id, view_id), view); + self.views.insert((window_id, view_id), view); if handled { return true; } @@ -1452,9 +1432,9 @@ impl MutableAppContext { .ancestors(window_id, focused_view_id) .collect::>() { - if let Some(mut view) = self.cx.views.remove(&(window_id, view_id)) { + if let Some(mut view) = self.views.remove(&(window_id, view_id)) { let handled = view.key_up(event, self, window_id, view_id); - self.cx.views.insert((window_id, view_id), view); + self.views.insert((window_id, view_id), view); if handled { return true; } @@ -1477,9 +1457,9 @@ impl MutableAppContext { .ancestors(window_id, focused_view_id) .collect::>() { - if let Some(mut view) = self.cx.views.remove(&(window_id, view_id)) { + if let Some(mut view) = self.views.remove(&(window_id, view_id)) { let handled = view.modifiers_changed(event, self, window_id, view_id); - self.cx.views.insert((window_id, view_id), view); + self.views.insert((window_id, view_id), view); if handled { return true; } @@ -1497,10 +1477,9 @@ impl MutableAppContext { let dispatch_path = self .ancestors(window_id, focused_view_id) .filter_map(|view_id| { - self.cx - .views + self.views .get(&(window_id, view_id)) - .map(|view| (view_id, view.keymap_context(self.as_ref()))) + .map(|view| (view_id, view.keymap_context(self))) }) .collect(); @@ -1544,7 +1523,7 @@ impl MutableAppContext { pub fn default_global(&mut self) -> &T { let type_id = TypeId::of::(); self.update(|this| { - if let Entry::Vacant(entry) = this.cx.globals.entry(type_id) { + if let Entry::Vacant(entry) = this.globals.entry(type_id) { entry.insert(Box::new(T::default())); this.notify_global(type_id); } @@ -1555,7 +1534,7 @@ impl MutableAppContext { pub fn set_global(&mut self, state: T) { self.update(|this| { let type_id = TypeId::of::(); - this.cx.globals.insert(type_id, Box::new(state)); + this.globals.insert(type_id, Box::new(state)); this.notify_global(type_id); }); } @@ -1563,17 +1542,16 @@ impl MutableAppContext { pub fn update_default_global(&mut self, update: F) -> U where T: 'static + Default, - F: FnOnce(&mut T, &mut MutableAppContext) -> U, + F: FnOnce(&mut T, &mut AppContext) -> U, { self.update(|this| { let type_id = TypeId::of::(); let mut state = this - .cx .globals .remove(&type_id) .unwrap_or_else(|| Box::new(T::default())); let result = update(state.downcast_mut().unwrap(), this); - this.cx.globals.insert(type_id, state); + this.globals.insert(type_id, state); this.notify_global(type_id); result }) @@ -1582,13 +1560,13 @@ impl MutableAppContext { pub fn update_global(&mut self, update: F) -> U where T: 'static, - F: FnOnce(&mut T, &mut MutableAppContext) -> U, + F: FnOnce(&mut T, &mut AppContext) -> U, { self.update(|this| { let type_id = TypeId::of::(); - if let Some(mut state) = this.cx.globals.remove(&type_id) { + if let Some(mut state) = this.globals.remove(&type_id) { let result = update(state.downcast_mut().unwrap(), this); - this.cx.globals.insert(type_id, state); + this.globals.insert(type_id, state); this.notify_global(type_id); result } else { @@ -1598,7 +1576,7 @@ impl MutableAppContext { } pub fn clear_globals(&mut self) { - self.cx.globals.clear(); + self.globals.clear(); } pub fn add_model(&mut self, build_model: F) -> ModelHandle @@ -1608,10 +1586,10 @@ impl MutableAppContext { { self.update(|this| { let model_id = post_inc(&mut this.next_entity_id); - let handle = ModelHandle::new(model_id, &this.cx.ref_counts); + let handle = ModelHandle::new(model_id, &this.ref_counts); let mut cx = ModelContext::new(this, model_id); let model = build_model(&mut cx); - this.cx.models.insert(model_id, Box::new(model)); + this.models.insert(model_id, Box::new(model)); handle }) } @@ -1630,7 +1608,7 @@ impl MutableAppContext { let root_view = this .build_and_insert_view(window_id, ParentId::Root, |cx| Some(build_root_view(cx))) .unwrap(); - this.cx.windows.insert( + this.windows.insert( window_id, Window { root_view: root_view.clone().into_any(), @@ -1643,8 +1621,7 @@ impl MutableAppContext { root_view.update(this, |view, cx| view.focus_in(cx.handle().into_any(), cx)); let window = - this.cx - .platform + this.platform .open_window(window_id, window_options, this.foreground.clone()); this.register_platform_window(window_id, window); @@ -1663,7 +1640,7 @@ impl MutableAppContext { .build_and_insert_view(window_id, ParentId::Root, |cx| Some(build_root_view(cx))) .unwrap(); let focused_view_id = root_view.id(); - this.cx.windows.insert( + this.windows.insert( window_id, Window { root_view: root_view.clone().into_any(), @@ -1675,7 +1652,7 @@ impl MutableAppContext { ); root_view.update(this, |view, cx| view.focus_in(cx.handle().into_any(), cx)); - let status_item = this.cx.platform.add_status_item(); + let status_item = this.platform.add_status_item(); this.register_platform_window(window_id, status_item); (window_id, root_view) @@ -1783,7 +1760,7 @@ impl MutableAppContext { let root_view = this .build_and_insert_view(window_id, ParentId::Root, |cx| Some(build_root_view(cx))) .unwrap(); - let window = this.cx.windows.get_mut(&window_id).unwrap(); + let window = this.windows.get_mut(&window_id).unwrap(); window.root_view = root_view.clone().into_any(); window.focused_view_id = Some(root_view.id()); root_view @@ -1791,7 +1768,7 @@ impl MutableAppContext { } pub fn remove_window(&mut self, window_id: usize) { - self.cx.windows.remove(&window_id); + self.windows.remove(&window_id); self.presenters_and_platform_windows.remove(&window_id); self.flush_effects(); } @@ -1806,8 +1783,8 @@ impl MutableAppContext { window_id, titlebar_height, appearance, - self.cx.font_cache.clone(), - TextLayoutCache::new(self.cx.platform.fonts()), + self.font_cache.clone(), + TextLayoutCache::new(self.platform.fonts()), self.assets.clone(), self, ) @@ -1856,20 +1833,20 @@ impl MutableAppContext { self.update(|this| { let view_id = post_inc(&mut this.next_entity_id); // Make sure we can tell child views about their parent - this.cx.parents.insert((window_id, view_id), parent_id); + this.parents.insert((window_id, view_id), parent_id); let mut cx = ViewContext::new(this, window_id, view_id); let handle = if let Some(view) = build_view(&mut cx) { - this.cx.views.insert((window_id, view_id), Box::new(view)); - if let Some(window) = this.cx.windows.get_mut(&window_id) { + this.views.insert((window_id, view_id), Box::new(view)); + if let Some(window) = this.windows.get_mut(&window_id) { window .invalidation .get_or_insert_with(Default::default) .updated .insert(view_id); } - Some(ViewHandle::new(window_id, view_id, &this.cx.ref_counts)) + Some(ViewHandle::new(window_id, view_id, &this.ref_counts)) } else { - this.cx.parents.remove(&(window_id, view_id)); + this.parents.remove(&(window_id, view_id)); None }; handle @@ -1879,7 +1856,7 @@ impl MutableAppContext { fn remove_dropped_entities(&mut self) { loop { let (dropped_models, dropped_views, dropped_element_states) = - self.cx.ref_counts.lock().take_dropped(); + self.ref_counts.lock().take_dropped(); if dropped_models.is_empty() && dropped_views.is_empty() && dropped_element_states.is_empty() @@ -1890,7 +1867,7 @@ impl MutableAppContext { for model_id in dropped_models { self.subscriptions.remove(model_id); self.observations.remove(model_id); - let mut model = self.cx.models.remove(&model_id).unwrap(); + let mut model = self.models.remove(&model_id).unwrap(); model.release(self); self.pending_effects .push_back(Effect::ModelRelease { model_id, model }); @@ -1899,9 +1876,9 @@ impl MutableAppContext { for (window_id, view_id) in dropped_views { self.subscriptions.remove(view_id); self.observations.remove(view_id); - let mut view = self.cx.views.remove(&(window_id, view_id)).unwrap(); + let mut view = self.views.remove(&(window_id, view_id)).unwrap(); view.release(self); - let change_focus_to = self.cx.windows.get_mut(&window_id).and_then(|window| { + let change_focus_to = self.windows.get_mut(&window_id).and_then(|window| { window .invalidation .get_or_insert_with(Default::default) @@ -1913,7 +1890,7 @@ impl MutableAppContext { None } }); - self.cx.parents.remove(&(window_id, view_id)); + self.parents.remove(&(window_id, view_id)); if let Some(view_id) = change_focus_to { self.handle_focus_effect(window_id, Some(view_id)); @@ -1924,7 +1901,7 @@ impl MutableAppContext { } for key in dropped_element_states { - self.cx.element_states.remove(&key); + self.element_states.remove(&key); } } } @@ -2028,7 +2005,7 @@ impl MutableAppContext { } Effect::ResizeWindow { window_id } => { - if let Some(window) = self.cx.windows.get_mut(&window_id) { + if let Some(window) = self.windows.get_mut(&window_id) { window .invalidation .get_or_insert(WindowInvalidation::default()); @@ -2153,7 +2130,7 @@ impl MutableAppContext { fn update_windows(&mut self) { let mut invalidations: HashMap<_, _> = Default::default(); - for (window_id, window) in &mut self.cx.windows { + for (window_id, window) in &mut self.windows { if let Some(invalidation) = window.invalidation.take() { invalidations.insert(*window_id, invalidation); } @@ -2243,13 +2220,7 @@ impl MutableAppContext { fn perform_window_refresh(&mut self) { let mut presenters = mem::take(&mut self.presenters_and_platform_windows); for (window_id, (presenter, window)) in &mut presenters { - let mut invalidation = self - .cx - .windows - .get_mut(window_id) - .unwrap() - .invalidation - .take(); + let mut invalidation = self.windows.get_mut(window_id).unwrap().invalidation.take(); let mut presenter = presenter.borrow_mut(); presenter.refresh( invalidation.as_mut().unwrap_or(&mut Default::default()), @@ -2279,11 +2250,10 @@ impl MutableAppContext { observed_view_id: usize, ) { if self - .cx .views .contains_key(&(observed_window_id, observed_view_id)) { - if let Some(window) = self.cx.windows.get_mut(&observed_window_id) { + if let Some(window) = self.windows.get_mut(&observed_window_id) { window .invalidation .get_or_insert_with(Default::default) @@ -2309,7 +2279,6 @@ impl MutableAppContext { fn handle_fullscreen_effect(&mut self, window_id: usize, is_fullscreen: bool) { //Short circuit evaluation if we're already g2g if self - .cx .windows .get(&window_id) .map(|w| w.is_fullscreen == is_fullscreen) @@ -2319,7 +2288,7 @@ impl MutableAppContext { } self.update(|this| { - let window = this.cx.windows.get_mut(&window_id)?; + let window = this.windows.get_mut(&window_id)?; window.is_fullscreen = is_fullscreen; let mut fullscreen_observations = this.window_fullscreen_observations.clone(); @@ -2359,7 +2328,6 @@ impl MutableAppContext { fn handle_window_activation_effect(&mut self, window_id: usize, active: bool) { //Short circuit evaluation if we're already g2g if self - .cx .windows .get(&window_id) .map(|w| w.is_active == active) @@ -2369,19 +2337,19 @@ impl MutableAppContext { } self.update(|this| { - let window = this.cx.windows.get_mut(&window_id)?; + let window = this.windows.get_mut(&window_id)?; window.is_active = active; //Handle focus let focused_id = window.focused_view_id?; for view_id in this.ancestors(window_id, focused_id).collect::>() { - if let Some(mut view) = this.cx.views.remove(&(window_id, view_id)) { + if let Some(mut view) = this.views.remove(&(window_id, view_id)) { if active { view.focus_in(this, window_id, view_id, focused_id); } else { view.focus_out(this, window_id, view_id, focused_id); } - this.cx.views.insert((window_id, view_id), view); + this.views.insert((window_id, view_id), view); } } @@ -2394,7 +2362,6 @@ impl MutableAppContext { fn handle_focus_effect(&mut self, window_id: usize, focused_id: Option) { if self - .cx .windows .get(&window_id) .map(|w| w.focused_view_id) @@ -2404,7 +2371,7 @@ impl MutableAppContext { } self.update(|this| { - let blurred_id = this.cx.windows.get_mut(&window_id).and_then(|window| { + let blurred_id = this.windows.get_mut(&window_id).and_then(|window| { let blurred_id = window.focused_view_id; window.focused_view_id = focused_id; blurred_id @@ -2419,9 +2386,9 @@ impl MutableAppContext { if let Some(blurred_id) = blurred_id { for view_id in blurred_parents.iter().copied() { - if let Some(mut view) = this.cx.views.remove(&(window_id, view_id)) { + if let Some(mut view) = this.views.remove(&(window_id, view_id)) { view.focus_out(this, window_id, view_id, blurred_id); - this.cx.views.insert((window_id, view_id), view); + this.views.insert((window_id, view_id), view); } } @@ -2431,9 +2398,9 @@ impl MutableAppContext { if let Some(focused_id) = focused_id { for view_id in focused_parents { - if let Some(mut view) = this.cx.views.remove(&(window_id, view_id)) { + if let Some(mut view) = this.views.remove(&(window_id, view_id)) { view.focus_in(this, window_id, view_id, focused_id); - this.cx.views.insert((window_id, view_id), view); + this.views.insert((window_id, view_id), view); } } @@ -2453,7 +2420,7 @@ impl MutableAppContext { if let Some(view_id) = view_id { this.halt_action_dispatch = false; this.visit_dispatch_path(window_id, view_id, |view_id, capture_phase, this| { - if let Some(mut view) = this.cx.views.remove(&(window_id, view_id)) { + if let Some(mut view) = this.views.remove(&(window_id, view_id)) { let type_id = view.as_any().type_id(); if let Some((name, mut handlers)) = this @@ -2474,7 +2441,7 @@ impl MutableAppContext { .insert(name, handlers); } - this.cx.views.insert((window_id, view_id), view); + this.views.insert((window_id, view_id), view); } !this.halt_action_dispatch @@ -2594,22 +2561,22 @@ impl MutableAppContext { } pub fn write_to_clipboard(&self, item: ClipboardItem) { - self.cx.platform.write_to_clipboard(item); + self.platform.write_to_clipboard(item); } pub fn read_from_clipboard(&self) -> Option { - self.cx.platform.read_from_clipboard() + self.platform.read_from_clipboard() } #[cfg(any(test, feature = "test-support"))] pub fn leak_detector(&self) -> Arc> { - self.cx.ref_counts.lock().leak_detector.clone() + self.ref_counts.lock().leak_detector.clone() } } -impl ReadModel for MutableAppContext { +impl ReadModel for AppContext { fn read_model(&self, handle: &ModelHandle) -> &T { - if let Some(model) = self.cx.models.get(&handle.model_id) { + if let Some(model) = self.models.get(&handle.model_id) { model .as_any() .downcast_ref() @@ -2620,13 +2587,13 @@ impl ReadModel for MutableAppContext { } } -impl UpdateModel for MutableAppContext { +impl UpdateModel for AppContext { fn update_model( &mut self, handle: &ModelHandle, update: &mut dyn FnMut(&mut T, &mut ModelContext) -> V, ) -> V { - if let Some(mut model) = self.cx.models.remove(&handle.model_id) { + if let Some(mut model) = self.models.remove(&handle.model_id) { self.update(|this| { let mut cx = ModelContext::new(this, handle.model_id); let result = update( @@ -2636,7 +2603,7 @@ impl UpdateModel for MutableAppContext { .expect("downcast is type safe"), &mut cx, ); - this.cx.models.insert(handle.model_id, model); + this.models.insert(handle.model_id, model); result }) } else { @@ -2645,36 +2612,65 @@ impl UpdateModel for MutableAppContext { } } -impl UpgradeModelHandle for MutableAppContext { +impl UpgradeModelHandle for AppContext { fn upgrade_model_handle( &self, handle: &WeakModelHandle, ) -> Option> { - self.cx.upgrade_model_handle(handle) + if self.models.contains_key(&handle.model_id) { + Some(ModelHandle::new(handle.model_id, &self.ref_counts)) + } else { + None + } } fn model_handle_is_upgradable(&self, handle: &WeakModelHandle) -> bool { - self.cx.model_handle_is_upgradable(handle) + self.models.contains_key(&handle.model_id) } fn upgrade_any_model_handle(&self, handle: &AnyWeakModelHandle) -> Option { - self.cx.upgrade_any_model_handle(handle) + if self.models.contains_key(&handle.model_id) { + Some(AnyModelHandle::new( + handle.model_id, + handle.model_type, + self.ref_counts.clone(), + )) + } else { + None + } } } -impl UpgradeViewHandle for MutableAppContext { +impl UpgradeViewHandle for AppContext { fn upgrade_view_handle(&self, handle: &WeakViewHandle) -> Option> { - self.cx.upgrade_view_handle(handle) + if self.ref_counts.lock().is_entity_alive(handle.view_id) { + Some(ViewHandle::new( + handle.window_id, + handle.view_id, + &self.ref_counts, + )) + } else { + None + } } fn upgrade_any_view_handle(&self, handle: &AnyWeakViewHandle) -> Option { - self.cx.upgrade_any_view_handle(handle) + if self.ref_counts.lock().is_entity_alive(handle.view_id) { + Some(AnyViewHandle::new( + handle.window_id, + handle.view_id, + handle.view_type, + self.ref_counts.clone(), + )) + } else { + None + } } } -impl ReadView for MutableAppContext { +impl ReadView for AppContext { fn read_view(&self, handle: &ViewHandle) -> &T { - if let Some(view) = self.cx.views.get(&(handle.window_id, handle.view_id)) { + if let Some(view) = self.views.get(&(handle.window_id, handle.view_id)) { view.as_any().downcast_ref().expect("downcast is type safe") } else { panic!("circular view reference for type {}", type_name::()); @@ -2682,7 +2678,7 @@ impl ReadView for MutableAppContext { } } -impl UpdateView for MutableAppContext { +impl UpdateView for AppContext { fn update_view( &mut self, handle: &ViewHandle, @@ -2693,7 +2689,6 @@ impl UpdateView for MutableAppContext { { self.update(|this| { let mut view = this - .cx .views .remove(&(handle.window_id, handle.view_id)) .expect("circular view update"); @@ -2705,49 +2700,20 @@ impl UpdateView for MutableAppContext { .expect("downcast is type safe"), &mut cx, ); - this.cx - .views - .insert((handle.window_id, handle.view_id), view); + this.views.insert((handle.window_id, handle.view_id), view); result }) } } -impl AsRef for MutableAppContext { - fn as_ref(&self) -> &AppContext { - &self.cx - } -} - -impl Deref for MutableAppContext { - type Target = AppContext; - - fn deref(&self) -> &Self::Target { - &self.cx - } -} - #[derive(Debug)] pub enum ParentId { View(usize), Root, } -pub struct AppContext { - models: HashMap>, - views: HashMap<(usize, usize), Box>, - pub(crate) parents: HashMap<(usize, usize), ParentId>, - windows: HashMap, - globals: HashMap>, - element_states: HashMap>, - background: Arc, - ref_counts: Arc>, - font_cache: Arc, - platform: Arc, -} - impl AppContext { - pub(crate) fn root_view(&self, window_id: usize) -> Option { + pub fn root_view(&self, window_id: usize) -> Option { self.windows .get(&window_id) .map(|window| window.root_view.clone()) @@ -2816,7 +2782,7 @@ impl AppContext { /// Returns the id of the parent of the given view, or none if the given /// view is the root. - fn parent(&self, window_id: usize, view_id: usize) -> Option { + pub fn parent(&self, window_id: usize, view_id: usize) -> Option { if let Some(ParentId::View(view_id)) = self.parents.get(&(window_id, view_id)) { Some(*view_id) } else { @@ -2835,87 +2801,6 @@ impl AppContext { } } -impl ReadModel for AppContext { - fn read_model(&self, handle: &ModelHandle) -> &T { - if let Some(model) = self.models.get(&handle.model_id) { - model - .as_any() - .downcast_ref() - .expect("downcast should be type safe") - } else { - panic!("circular model reference"); - } - } -} - -impl UpgradeModelHandle for AppContext { - fn upgrade_model_handle( - &self, - handle: &WeakModelHandle, - ) -> Option> { - if self.models.contains_key(&handle.model_id) { - Some(ModelHandle::new(handle.model_id, &self.ref_counts)) - } else { - None - } - } - - fn model_handle_is_upgradable(&self, handle: &WeakModelHandle) -> bool { - self.models.contains_key(&handle.model_id) - } - - fn upgrade_any_model_handle(&self, handle: &AnyWeakModelHandle) -> Option { - if self.models.contains_key(&handle.model_id) { - Some(AnyModelHandle::new( - handle.model_id, - handle.model_type, - self.ref_counts.clone(), - )) - } else { - None - } - } -} - -impl UpgradeViewHandle for AppContext { - fn upgrade_view_handle(&self, handle: &WeakViewHandle) -> Option> { - if self.ref_counts.lock().is_entity_alive(handle.view_id) { - Some(ViewHandle::new( - handle.window_id, - handle.view_id, - &self.ref_counts, - )) - } else { - None - } - } - - fn upgrade_any_view_handle(&self, handle: &AnyWeakViewHandle) -> Option { - if self.ref_counts.lock().is_entity_alive(handle.view_id) { - Some(AnyViewHandle::new( - handle.window_id, - handle.view_id, - handle.view_type, - self.ref_counts.clone(), - )) - } else { - None - } - } -} - -impl ReadView for AppContext { - fn read_view(&self, handle: &ViewHandle) -> &T { - if let Some(view) = self.views.get(&(handle.window_id, handle.view_id)) { - view.as_any() - .downcast_ref() - .expect("downcast should be type safe") - } else { - panic!("circular view reference"); - } - } -} - struct Window { root_view: AnyViewHandle, focused_view_id: Option, @@ -2961,7 +2846,7 @@ pub enum Effect { view_id: usize, }, Deferred { - callback: Box, + callback: Box, after_window_update: bool, }, GlobalNotification { @@ -3213,10 +3098,10 @@ impl Debug for Effect { pub trait AnyModel { fn as_any(&self) -> &dyn Any; fn as_any_mut(&mut self) -> &mut dyn Any; - fn release(&mut self, cx: &mut MutableAppContext); + fn release(&mut self, cx: &mut AppContext); fn app_will_quit( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>>>; } @@ -3232,13 +3117,13 @@ where self } - fn release(&mut self, cx: &mut MutableAppContext) { + fn release(&mut self, cx: &mut AppContext) { self.release(cx); } fn app_will_quit( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>>> { self.app_will_quit(cx) } @@ -3247,23 +3132,23 @@ where pub trait AnyView { fn as_any(&self) -> &dyn Any; fn as_any_mut(&mut self) -> &mut dyn Any; - fn release(&mut self, cx: &mut MutableAppContext); + fn release(&mut self, cx: &mut AppContext); fn app_will_quit( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>>>; fn ui_name(&self) -> &'static str; - fn render(&mut self, params: RenderParams, cx: &mut MutableAppContext) -> ElementBox; + fn render(&mut self, params: RenderParams, cx: &mut AppContext) -> ElementBox; fn focus_in( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, focused_id: usize, ); fn focus_out( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, focused_id: usize, @@ -3271,21 +3156,21 @@ pub trait AnyView { fn key_down( &mut self, event: &KeyDownEvent, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ) -> bool; fn key_up( &mut self, event: &KeyUpEvent, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ) -> bool; fn modifiers_changed( &mut self, event: &ModifiersChangedEvent, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ) -> bool; @@ -3295,12 +3180,12 @@ pub trait AnyView { fn text_for_range(&self, range: Range, cx: &AppContext) -> Option; fn selected_text_range(&self, cx: &AppContext) -> Option>; fn marked_text_range(&self, cx: &AppContext) -> Option>; - fn unmark_text(&mut self, cx: &mut MutableAppContext, window_id: usize, view_id: usize); + fn unmark_text(&mut self, cx: &mut AppContext, window_id: usize, view_id: usize); fn replace_text_in_range( &mut self, range: Option>, text: &str, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ); @@ -3309,7 +3194,7 @@ pub trait AnyView { range: Option>, new_text: &str, new_selected_range: Option>, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ); @@ -3335,13 +3220,13 @@ where self } - fn release(&mut self, cx: &mut MutableAppContext) { + fn release(&mut self, cx: &mut AppContext) { self.release(cx); } fn app_will_quit( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>>> { self.app_will_quit(cx) } @@ -3350,13 +3235,13 @@ where T::ui_name() } - fn render(&mut self, params: RenderParams, cx: &mut MutableAppContext) -> ElementBox { + fn render(&mut self, params: RenderParams, cx: &mut AppContext) -> ElementBox { View::render(self, &mut RenderContext::new(params, cx)) } fn focus_in( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, focused_id: usize, @@ -3378,7 +3263,7 @@ where fn focus_out( &mut self, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, blurred_id: usize, @@ -3401,7 +3286,7 @@ where fn key_down( &mut self, event: &KeyDownEvent, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ) -> bool { @@ -3412,7 +3297,7 @@ where fn key_up( &mut self, event: &KeyUpEvent, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ) -> bool { @@ -3423,7 +3308,7 @@ where fn modifiers_changed( &mut self, event: &ModifiersChangedEvent, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ) -> bool { @@ -3451,7 +3336,7 @@ where View::marked_text_range(self, cx) } - fn unmark_text(&mut self, cx: &mut MutableAppContext, window_id: usize, view_id: usize) { + fn unmark_text(&mut self, cx: &mut AppContext, window_id: usize, view_id: usize) { let mut cx = ViewContext::new(cx, window_id, view_id); View::unmark_text(self, &mut cx) } @@ -3460,7 +3345,7 @@ where &mut self, range: Option>, text: &str, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ) { @@ -3473,7 +3358,7 @@ where range: Option>, new_text: &str, new_selected_range: Option>, - cx: &mut MutableAppContext, + cx: &mut AppContext, window_id: usize, view_id: usize, ) { @@ -3483,14 +3368,14 @@ where } pub struct ModelContext<'a, T: ?Sized> { - app: &'a mut MutableAppContext, + app: &'a mut AppContext, model_id: usize, model_type: PhantomData, halt_stream: bool, } impl<'a, T: Entity> ModelContext<'a, T> { - fn new(app: &'a mut MutableAppContext, model_id: usize) -> Self { + fn new(app: &'a mut AppContext, model_id: usize) -> Self { Self { app, model_id, @@ -3500,7 +3385,7 @@ impl<'a, T: Entity> ModelContext<'a, T> { } pub fn background(&self) -> &Arc { - &self.app.cx.background + &self.app.background } pub fn halt_stream(&mut self) { @@ -3613,7 +3498,7 @@ impl<'a, T: Entity> ModelContext<'a, T> { } pub fn handle(&self) -> ModelHandle { - ModelHandle::new(self.model_id, &self.app.cx.ref_counts) + ModelHandle::new(self.model_id, &self.app.ref_counts) } pub fn weak_handle(&self) -> WeakModelHandle { @@ -3643,12 +3528,12 @@ impl<'a, T: Entity> ModelContext<'a, T> { impl AsRef for ModelContext<'_, M> { fn as_ref(&self) -> &AppContext { - &self.app.cx + &self.app } } -impl AsMut for ModelContext<'_, M> { - fn as_mut(&mut self) -> &mut MutableAppContext { +impl AsMut for ModelContext<'_, M> { + fn as_mut(&mut self) -> &mut AppContext { self.app } } @@ -3674,20 +3559,20 @@ impl UpgradeModelHandle for ModelContext<'_, M> { &self, handle: &WeakModelHandle, ) -> Option> { - self.cx.upgrade_model_handle(handle) + self.app.upgrade_model_handle(handle) } fn model_handle_is_upgradable(&self, handle: &WeakModelHandle) -> bool { - self.cx.model_handle_is_upgradable(handle) + self.app.model_handle_is_upgradable(handle) } fn upgrade_any_model_handle(&self, handle: &AnyWeakModelHandle) -> Option { - self.cx.upgrade_any_model_handle(handle) + self.app.upgrade_any_model_handle(handle) } } impl Deref for ModelContext<'_, M> { - type Target = MutableAppContext; + type Target = AppContext; fn deref(&self) -> &Self::Target { self.app @@ -3701,14 +3586,14 @@ impl DerefMut for ModelContext<'_, M> { } pub struct ViewContext<'a, T: ?Sized> { - app: &'a mut MutableAppContext, + app: &'a mut AppContext, window_id: usize, view_id: usize, view_type: PhantomData, } impl<'a, T: View> ViewContext<'a, T> { - fn new(app: &'a mut MutableAppContext, window_id: usize, view_id: usize) -> Self { + fn new(app: &'a mut AppContext, window_id: usize, view_id: usize) -> Self { Self { app, window_id, @@ -3718,13 +3603,17 @@ impl<'a, T: View> ViewContext<'a, T> { } pub fn handle(&self) -> ViewHandle { - ViewHandle::new(self.window_id, self.view_id, &self.app.cx.ref_counts) + ViewHandle::new(self.window_id, self.view_id, &self.app.ref_counts) } pub fn weak_handle(&self) -> WeakViewHandle { WeakViewHandle::new(self.window_id, self.view_id) } + pub fn parent(&self) -> Option { + self.app.parent(self.window_id, self.view_id) + } + pub fn window_id(&self) -> usize { self.window_id } @@ -3738,10 +3627,10 @@ impl<'a, T: View> ViewContext<'a, T> { } pub fn background_executor(&self) -> &Arc { - &self.app.cx.background + &self.app.background } - pub fn platform(&self) -> Arc { + pub fn platform(&self) -> &Arc { self.app.platform() } @@ -3877,19 +3766,14 @@ impl<'a, T: View> ViewContext<'a, T> { .build_and_insert_view(self.window_id, ParentId::View(self.view_id), build_view) } - pub fn parent(&mut self) -> Option { - self.cx.parent(self.window_id, self.view_id) - } - pub fn reparent(&mut self, view_handle: &AnyViewHandle) { if self.window_id != view_handle.window_id { panic!("Can't reparent view to a view from a different window"); } - self.cx - .parents + self.parents .remove(&(view_handle.window_id, view_handle.view_id)); let new_parent_id = self.view_id; - self.cx.parents.insert( + self.parents.insert( (view_handle.window_id, view_handle.view_id), ParentId::View(new_parent_id), ); @@ -3905,7 +3789,7 @@ impl<'a, T: View> ViewContext<'a, T> { let root_view = this .build_and_insert_view(window_id, ParentId::Root, |cx| Some(build_root_view(cx))) .unwrap(); - let window = this.cx.windows.get_mut(&window_id).unwrap(); + let window = this.windows.get_mut(&window_id).unwrap(); window.root_view = root_view.clone().into_any(); window.focused_view_id = Some(root_view.id()); root_view @@ -4205,7 +4089,7 @@ pub struct RenderContext<'a, T: View> { pub(crate) view_type: PhantomData, pub(crate) hovered_region_ids: HashSet, pub(crate) clicked_region_ids: Option<(HashSet, MouseButton)>, - pub app: &'a mut MutableAppContext, + pub app: &'a mut AppContext, pub titlebar_height: f32, pub appearance: Appearance, pub refreshing: bool, @@ -4240,7 +4124,7 @@ impl MouseState { } impl<'a, V: View> RenderContext<'a, V> { - fn new(params: RenderParams, app: &'a mut MutableAppContext) -> Self { + fn new(params: RenderParams, app: &'a mut AppContext) -> Self { Self { app, window_id: params.window_id, @@ -4292,11 +4176,10 @@ impl<'a, V: View> RenderContext<'a, V> { element_id, tag: TypeId::of::(), }; - self.cx - .element_states + self.element_states .entry(id) .or_insert_with(|| Box::new(initial)); - ElementStateHandle::new(id, self.frame_count, &self.cx.ref_counts) + ElementStateHandle::new(id, self.frame_count, &self.ref_counts) } pub fn default_element_state( @@ -4307,14 +4190,8 @@ impl<'a, V: View> RenderContext<'a, V> { } } -impl AsRef for &AppContext { - fn as_ref(&self) -> &AppContext { - self - } -} - impl Deref for RenderContext<'_, V> { - type Target = MutableAppContext; + type Target = AppContext; fn deref(&self) -> &Self::Target { self.app @@ -4349,14 +4226,8 @@ impl ReadView for RenderContext<'_, V> { } } -impl AsRef for ViewContext<'_, M> { - fn as_ref(&self) -> &AppContext { - &self.app.cx - } -} - impl Deref for ViewContext<'_, M> { - type Target = MutableAppContext; + type Target = AppContext; fn deref(&self) -> &Self::Target { self.app @@ -4369,8 +4240,8 @@ impl DerefMut for ViewContext<'_, M> { } } -impl AsMut for ViewContext<'_, M> { - fn as_mut(&mut self) -> &mut MutableAppContext { +impl AsMut for ViewContext<'_, M> { + fn as_mut(&mut self) -> &mut AppContext { self.app } } @@ -4386,35 +4257,35 @@ impl UpgradeModelHandle for ViewContext<'_, V> { &self, handle: &WeakModelHandle, ) -> Option> { - self.cx.upgrade_model_handle(handle) + self.app.upgrade_model_handle(handle) } fn model_handle_is_upgradable(&self, handle: &WeakModelHandle) -> bool { - self.cx.model_handle_is_upgradable(handle) + self.app.model_handle_is_upgradable(handle) } fn upgrade_any_model_handle(&self, handle: &AnyWeakModelHandle) -> Option { - self.cx.upgrade_any_model_handle(handle) + self.app.upgrade_any_model_handle(handle) } } impl UpgradeViewHandle for ViewContext<'_, V> { fn upgrade_view_handle(&self, handle: &WeakViewHandle) -> Option> { - self.cx.upgrade_view_handle(handle) + self.app.upgrade_view_handle(handle) } fn upgrade_any_view_handle(&self, handle: &AnyWeakViewHandle) -> Option { - self.cx.upgrade_any_view_handle(handle) + self.app.upgrade_any_view_handle(handle) } } impl UpgradeViewHandle for RenderContext<'_, V> { fn upgrade_view_handle(&self, handle: &WeakViewHandle) -> Option> { - self.cx.upgrade_view_handle(handle) + self.app.upgrade_view_handle(handle) } fn upgrade_any_view_handle(&self, handle: &AnyWeakViewHandle) -> Option { - self.cx.upgrade_any_view_handle(handle) + self.app.upgrade_any_view_handle(handle) } } @@ -4747,7 +4618,7 @@ impl ViewHandle { pub fn defer(&self, cx: &mut C, update: F) where - C: AsMut, + C: AsMut, F: 'static + FnOnce(&mut T, &mut ViewContext), { let this = self.clone(); @@ -5191,14 +5062,11 @@ impl ElementStateHandle { pub fn update(&self, cx: &mut C, f: impl FnOnce(&mut T, &mut C) -> R) -> R where - C: DerefMut, + C: DerefMut, { - let mut element_state = cx.deref_mut().cx.element_states.remove(&self.id).unwrap(); + let mut element_state = cx.deref_mut().element_states.remove(&self.id).unwrap(); let result = f(element_state.downcast_mut().unwrap(), cx); - cx.deref_mut() - .cx - .element_states - .insert(self.id, element_state); + cx.deref_mut().element_states.insert(self.id, element_state); result } } @@ -5279,7 +5147,7 @@ mod tests { }; #[crate::test(self)] - fn test_model_handles(cx: &mut MutableAppContext) { + fn test_model_handles(cx: &mut AppContext) { struct Model { other: Option>, events: Vec, @@ -5311,7 +5179,7 @@ mod tests { let handle_1 = cx.add_model(|cx| Model::new(None, cx)); let handle_2 = cx.add_model(|cx| Model::new(Some(handle_1.clone()), cx)); - assert_eq!(cx.cx.models.len(), 2); + assert_eq!(cx.models.len(), 2); handle_1.update(cx, |model, cx| { model.events.push("updated".into()); @@ -5334,13 +5202,13 @@ mod tests { model.other.take(); }); - assert_eq!(cx.cx.models.len(), 1); + assert_eq!(cx.models.len(), 1); assert!(cx.subscriptions.is_empty()); assert!(cx.observations.is_empty()); } #[crate::test(self)] - fn test_model_events(cx: &mut MutableAppContext) { + fn test_model_events(cx: &mut AppContext) { #[derive(Default)] struct Model { events: Vec, @@ -5373,7 +5241,7 @@ mod tests { } #[crate::test(self)] - fn test_model_emit_before_subscribe_in_same_update_cycle(cx: &mut MutableAppContext) { + fn test_model_emit_before_subscribe_in_same_update_cycle(cx: &mut AppContext) { #[derive(Default)] struct Model; @@ -5404,7 +5272,7 @@ mod tests { } #[crate::test(self)] - fn test_observe_and_notify_from_model(cx: &mut MutableAppContext) { + fn test_observe_and_notify_from_model(cx: &mut AppContext) { #[derive(Default)] struct Model { count: usize, @@ -5443,7 +5311,7 @@ mod tests { } #[crate::test(self)] - fn test_model_notify_before_observe_in_same_update_cycle(cx: &mut MutableAppContext) { + fn test_model_notify_before_observe_in_same_update_cycle(cx: &mut AppContext) { #[derive(Default)] struct Model; @@ -5474,7 +5342,7 @@ mod tests { } #[crate::test(self)] - fn test_defer_and_after_window_update(cx: &mut MutableAppContext) { + fn test_defer_and_after_window_update(cx: &mut AppContext) { struct View { render_count: usize, } @@ -5526,7 +5394,7 @@ mod tests { } #[crate::test(self)] - fn test_view_handles(cx: &mut MutableAppContext) { + fn test_view_handles(cx: &mut AppContext) { struct View { other: Option>, events: Vec, @@ -5564,7 +5432,7 @@ mod tests { let (_, root_view) = cx.add_window(Default::default(), |cx| View::new(None, cx)); let handle_1 = cx.add_view(&root_view, |cx| View::new(None, cx)); let handle_2 = cx.add_view(&root_view, |cx| View::new(Some(handle_1.clone()), cx)); - assert_eq!(cx.cx.views.len(), 3); + assert_eq!(cx.views.len(), 3); handle_1.update(cx, |view, cx| { view.events.push("updated".into()); @@ -5585,13 +5453,13 @@ mod tests { view.other.take(); }); - assert_eq!(cx.cx.views.len(), 2); + assert_eq!(cx.views.len(), 2); assert!(cx.subscriptions.is_empty()); assert!(cx.observations.is_empty()); } #[crate::test(self)] - fn test_add_window(cx: &mut MutableAppContext) { + fn test_add_window(cx: &mut AppContext) { struct View { mouse_down_count: Arc, } @@ -5636,7 +5504,7 @@ mod tests { } #[crate::test(self)] - fn test_entity_release_hooks(cx: &mut MutableAppContext) { + fn test_entity_release_hooks(cx: &mut AppContext) { struct Model { released: Rc>, } @@ -5648,7 +5516,7 @@ mod tests { impl Entity for Model { type Event = (); - fn release(&mut self, _: &mut MutableAppContext) { + fn release(&mut self, _: &mut AppContext) { self.released.set(true); } } @@ -5656,7 +5524,7 @@ mod tests { impl Entity for View { type Event = (); - fn release(&mut self, _: &mut MutableAppContext) { + fn release(&mut self, _: &mut AppContext) { self.released.set(true); } } @@ -5709,7 +5577,7 @@ mod tests { } #[crate::test(self)] - fn test_view_events(cx: &mut MutableAppContext) { + fn test_view_events(cx: &mut AppContext) { struct Model; impl Entity for Model { @@ -5751,7 +5619,7 @@ mod tests { } #[crate::test(self)] - fn test_global_events(cx: &mut MutableAppContext) { + fn test_global_events(cx: &mut AppContext) { #[derive(Clone, Debug, Eq, PartialEq)] struct GlobalEvent(u64); @@ -5803,9 +5671,7 @@ mod tests { } #[crate::test(self)] - fn test_global_events_emitted_before_subscription_in_same_update_cycle( - cx: &mut MutableAppContext, - ) { + fn test_global_events_emitted_before_subscription_in_same_update_cycle(cx: &mut AppContext) { let events = Rc::new(RefCell::new(Vec::new())); cx.update(|cx| { { @@ -5838,7 +5704,7 @@ mod tests { } #[crate::test(self)] - fn test_global_nested_events(cx: &mut MutableAppContext) { + fn test_global_nested_events(cx: &mut AppContext) { #[derive(Clone, Debug, Eq, PartialEq)] struct GlobalEvent(u64); @@ -5882,7 +5748,7 @@ mod tests { } #[crate::test(self)] - fn test_global(cx: &mut MutableAppContext) { + fn test_global(cx: &mut AppContext) { type Global = usize; let observation_count = Rc::new(RefCell::new(0)); @@ -5937,7 +5803,7 @@ mod tests { } #[crate::test(self)] - fn test_dropping_subscribers(cx: &mut MutableAppContext) { + fn test_dropping_subscribers(cx: &mut AppContext) { struct Model; impl Entity for Model { @@ -5968,7 +5834,7 @@ mod tests { } #[crate::test(self)] - fn test_view_emit_before_subscribe_in_same_update_cycle(cx: &mut MutableAppContext) { + fn test_view_emit_before_subscribe_in_same_update_cycle(cx: &mut AppContext) { let (_, view) = cx.add_window::(Default::default(), |cx| { drop(cx.subscribe(&cx.handle(), { move |this, _, _, _| this.events.push("dropped before flush".into()) @@ -5989,7 +5855,7 @@ mod tests { } #[crate::test(self)] - fn test_observe_and_notify_from_view(cx: &mut MutableAppContext) { + fn test_observe_and_notify_from_view(cx: &mut AppContext) { #[derive(Default)] struct Model { state: String, @@ -6019,7 +5885,7 @@ mod tests { } #[crate::test(self)] - fn test_view_notify_before_observe_in_same_update_cycle(cx: &mut MutableAppContext) { + fn test_view_notify_before_observe_in_same_update_cycle(cx: &mut AppContext) { let (_, view) = cx.add_window::(Default::default(), |cx| { drop(cx.observe(&cx.handle(), { move |this, _, _| this.events.push("dropped before flush".into()) @@ -6040,7 +5906,7 @@ mod tests { } #[crate::test(self)] - fn test_notify_and_drop_observe_subscription_in_same_update_cycle(cx: &mut MutableAppContext) { + fn test_notify_and_drop_observe_subscription_in_same_update_cycle(cx: &mut AppContext) { struct Model; impl Entity for Model { type Event = (); @@ -6065,7 +5931,7 @@ mod tests { } #[crate::test(self)] - fn test_dropping_observers(cx: &mut MutableAppContext) { + fn test_dropping_observers(cx: &mut AppContext) { struct Model; impl Entity for Model { @@ -6093,7 +5959,7 @@ mod tests { } #[crate::test(self)] - fn test_dropping_subscriptions_during_callback(cx: &mut MutableAppContext) { + fn test_dropping_subscriptions_during_callback(cx: &mut AppContext) { struct Model; impl Entity for Model { @@ -6240,7 +6106,7 @@ mod tests { } #[crate::test(self)] - fn test_focus(cx: &mut MutableAppContext) { + fn test_focus(cx: &mut AppContext) { struct View { name: String, events: Arc>>, @@ -6381,7 +6247,7 @@ mod tests { } #[crate::test(self)] - fn test_deserialize_actions(cx: &mut MutableAppContext) { + fn test_deserialize_actions(cx: &mut AppContext) { #[derive(Clone, Debug, Deserialize, PartialEq, Eq)] pub struct ComplexAction { arg: String, @@ -6391,8 +6257,8 @@ mod tests { actions!(test::something, [SimpleAction]); impl_actions!(test::something, [ComplexAction]); - cx.add_global_action(move |_: &SimpleAction, _: &mut MutableAppContext| {}); - cx.add_global_action(move |_: &ComplexAction, _: &mut MutableAppContext| {}); + cx.add_global_action(move |_: &SimpleAction, _: &mut AppContext| {}); + cx.add_global_action(move |_: &ComplexAction, _: &mut AppContext| {}); let action1 = cx .deserialize_action( @@ -6417,7 +6283,7 @@ mod tests { } #[crate::test(self)] - fn test_dispatch_action(cx: &mut MutableAppContext) { + fn test_dispatch_action(cx: &mut AppContext) { struct ViewA { id: usize, } @@ -6463,7 +6329,7 @@ mod tests { cx.add_global_action({ let actions = actions.clone(); - move |_: &Action, _: &mut MutableAppContext| { + move |_: &Action, _: &mut AppContext| { actions.borrow_mut().push("global".to_string()); } }); @@ -6581,7 +6447,7 @@ mod tests { } #[crate::test(self)] - fn test_dispatch_keystroke(cx: &mut MutableAppContext) { + fn test_dispatch_keystroke(cx: &mut AppContext) { #[derive(Clone, Deserialize, PartialEq)] pub struct Action(String); @@ -6701,7 +6567,7 @@ mod tests { } #[crate::test(self)] - fn test_keystrokes_for_action(cx: &mut MutableAppContext) { + fn test_keystrokes_for_action(cx: &mut AppContext) { actions!(test, [Action1, Action2, GlobalAction]); struct View1 {} @@ -6786,7 +6652,7 @@ mod tests { fn available_actions( window_id: usize, view_id: usize, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Vec<(&'static str, Vec)> { cx.available_actions(window_id, view_id) .map(|(action_name, _, bindings)| { @@ -6948,7 +6814,7 @@ mod tests { } #[crate::test(self)] - fn test_refresh_windows(cx: &mut MutableAppContext) { + fn test_refresh_windows(cx: &mut AppContext) { struct View(usize); impl super::Entity for View { @@ -7106,7 +6972,7 @@ mod tests { } #[crate::test(self)] - fn test_child_view(cx: &mut MutableAppContext) { + fn test_child_view(cx: &mut AppContext) { struct Child { rendered: Rc>, dropped: Rc>, diff --git a/crates/gpui/src/app/callback_collection.rs b/crates/gpui/src/app/callback_collection.rs index 7496d71501..e693008b29 100644 --- a/crates/gpui/src/app/callback_collection.rs +++ b/crates/gpui/src/app/callback_collection.rs @@ -1,4 +1,4 @@ -use crate::MutableAppContext; +use crate::AppContext; use collections::{BTreeMap, HashMap, HashSet}; use parking_lot::Mutex; use std::sync::Arc; @@ -93,10 +93,10 @@ impl CallbackCollection { drop(callbacks); } - pub fn emit bool>( + pub fn emit bool>( &mut self, key: K, - cx: &mut MutableAppContext, + cx: &mut AppContext, mut call_callback: C, ) { let callbacks = self.internal.lock().callbacks.remove(&key); diff --git a/crates/gpui/src/app/menu.rs b/crates/gpui/src/app/menu.rs index b0965539a5..a87d476f16 100644 --- a/crates/gpui/src/app/menu.rs +++ b/crates/gpui/src/app/menu.rs @@ -1,4 +1,4 @@ -use crate::{Action, App, ForegroundPlatform, MutableAppContext}; +use crate::{Action, App, AppContext, ForegroundPlatform}; pub struct Menu<'a> { pub name: &'a str, @@ -51,7 +51,7 @@ pub enum OsAction { Redo, } -impl MutableAppContext { +impl AppContext { pub fn set_menus(&mut self, menus: Vec) { self.foreground_platform .set_menus(menus, &self.keystroke_matcher); @@ -77,7 +77,7 @@ pub(crate) fn setup_menu_handlers(foreground_platform: &dyn ForegroundPlatform, let cx = app.0.clone(); move |action| { let mut cx = cx.borrow_mut(); - if let Some(main_window_id) = cx.cx.platform.main_window_id() { + if let Some(main_window_id) = cx.platform.main_window_id() { if let Some(view_id) = cx.focused_view_id(main_window_id) { cx.handle_dispatch_action_from_effect(main_window_id, Some(view_id), action); return; diff --git a/crates/gpui/src/app/test_app_context.rs b/crates/gpui/src/app/test_app_context.rs index 21059da181..cd38a9059c 100644 --- a/crates/gpui/src/app/test_app_context.rs +++ b/crates/gpui/src/app/test_app_context.rs @@ -19,9 +19,8 @@ use smol::stream::StreamExt; use crate::{ executor, geometry::vector::Vector2F, keymap_matcher::Keystroke, platform, Action, AnyViewHandle, AppContext, Appearance, Entity, Event, FontCache, Handle, InputHandler, - KeyDownEvent, ModelContext, ModelHandle, MutableAppContext, Platform, ReadModelWith, - ReadViewWith, RenderContext, Task, UpdateModel, UpdateView, View, ViewContext, ViewHandle, - WeakHandle, + KeyDownEvent, ModelContext, ModelHandle, Platform, ReadModelWith, ReadViewWith, RenderContext, + Task, UpdateModel, UpdateView, View, ViewContext, ViewHandle, WeakHandle, }; use collections::BTreeMap; @@ -30,7 +29,7 @@ use super::{ }; pub struct TestAppContext { - cx: Rc>, + cx: Rc>, foreground_platform: Rc, condition_duration: Option, pub function_name: String, @@ -48,7 +47,7 @@ impl TestAppContext { first_entity_id: usize, function_name: String, ) -> Self { - let mut cx = MutableAppContext::new( + let mut cx = AppContext::new( foreground, background, platform, @@ -149,15 +148,15 @@ impl TestAppContext { self.cx.borrow().window_ids().collect() } - pub fn root_view(&self, window_id: usize) -> Option> { + pub fn root_view(&self, window_id: usize) -> Option { self.cx.borrow().root_view(window_id) } pub fn read T>(&self, callback: F) -> T { - callback(self.cx.borrow().as_ref()) + callback(&*self.cx.borrow()) } - pub fn update T>(&mut self, callback: F) -> T { + pub fn update T>(&mut self, callback: F) -> T { let mut state = self.cx.borrow_mut(); // Don't increment pending flushes in order for effects to be flushed before the callback // completes, which is helpful in tests. @@ -194,7 +193,7 @@ impl TestAppContext { } pub fn font_cache(&self) -> Arc { - self.cx.borrow().cx.font_cache.clone() + self.cx.borrow().font_cache.clone() } pub fn foreground_platform(&self) -> Rc { @@ -202,7 +201,7 @@ impl TestAppContext { } pub fn platform(&self) -> Arc { - self.cx.borrow().cx.platform.clone() + self.cx.borrow().platform.clone() } pub fn foreground(&self) -> Rc { @@ -396,7 +395,7 @@ impl ReadModelWith for TestAppContext { read: &mut dyn FnMut(&E, &AppContext) -> T, ) -> T { let cx = self.cx.borrow(); - let cx = cx.as_ref(); + let cx = &*cx; read(handle.read(cx), cx) } } @@ -424,7 +423,7 @@ impl ReadViewWith for TestAppContext { V: View, { let cx = self.cx.borrow(); - let cx = cx.as_ref(); + let cx = &*cx; read(handle.read(cx), cx) } } @@ -513,7 +512,7 @@ impl ModelHandle { loop { { let cx = cx.borrow(); - let cx = cx.as_ref(); + let cx = &*cx; if predicate( handle .upgrade(cx) @@ -600,7 +599,7 @@ impl ViewHandle { loop { { let cx = cx.borrow(); - let cx = cx.as_ref(); + let cx = &*cx; if predicate( handle .upgrade(cx) diff --git a/crates/gpui/src/app/window_input_handler.rs b/crates/gpui/src/app/window_input_handler.rs index 855f0e3041..30ca3d6e02 100644 --- a/crates/gpui/src/app/window_input_handler.rs +++ b/crates/gpui/src/app/window_input_handler.rs @@ -2,10 +2,10 @@ use std::{cell::RefCell, ops::Range, rc::Rc}; use pathfinder_geometry::rect::RectF; -use crate::{AnyView, AppContext, InputHandler, MutableAppContext}; +use crate::{AnyView, AppContext, InputHandler}; pub struct WindowInputHandler { - pub app: Rc>, + pub app: Rc>, pub window_id: usize, } @@ -23,21 +23,21 @@ impl WindowInputHandler { let app = self.app.try_borrow().ok()?; let view_id = app.focused_view_id(self.window_id)?; - let view = app.cx.views.get(&(self.window_id, view_id))?; + let view = app.views.get(&(self.window_id, view_id))?; let result = f(view.as_ref(), &app); Some(result) } fn update_focused_view(&mut self, f: F) -> Option where - F: FnOnce(usize, usize, &mut dyn AnyView, &mut MutableAppContext) -> T, + F: FnOnce(usize, usize, &mut dyn AnyView, &mut AppContext) -> T, { let mut app = self.app.try_borrow_mut().ok()?; app.update(|app| { let view_id = app.focused_view_id(self.window_id)?; - let mut view = app.cx.views.remove(&(self.window_id, view_id))?; + let mut view = app.views.remove(&(self.window_id, view_id))?; let result = f(self.window_id, view_id, view.as_mut(), &mut *app); - app.cx.views.insert((self.window_id, view_id), view); + app.views.insert((self.window_id, view_id), view); Some(result) }) } diff --git a/crates/gpui/src/elements/label.rs b/crates/gpui/src/elements/label.rs index aae00b07a8..41eea05957 100644 --- a/crates/gpui/src/elements/label.rs +++ b/crates/gpui/src/elements/label.rs @@ -210,7 +210,7 @@ mod tests { use crate::fonts::{Properties as FontProperties, Weight}; #[crate::test(self)] - fn test_layout_label_with_highlights(cx: &mut crate::MutableAppContext) { + fn test_layout_label_with_highlights(cx: &mut crate::AppContext) { let default_style = TextStyle::new( "Menlo", 12., diff --git a/crates/gpui/src/elements/list.rs b/crates/gpui/src/elements/list.rs index 53a0b70b35..df06f05009 100644 --- a/crates/gpui/src/elements/list.rs +++ b/crates/gpui/src/elements/list.rs @@ -630,7 +630,7 @@ mod tests { use std::env; #[crate::test(self)] - fn test_layout(cx: &mut crate::MutableAppContext) { + fn test_layout(cx: &mut crate::AppContext) { let mut presenter = cx.build_presenter(0, 0., Default::default()); let (_, view) = cx.add_window(Default::default(), |_| TestView); let constraint = SizeConstraint::new(vec2f(0., 0.), vec2f(100., 40.)); @@ -725,7 +725,7 @@ mod tests { } #[crate::test(self, iterations = 10, seed = 0)] - fn test_random(cx: &mut crate::MutableAppContext, mut rng: StdRng) { + fn test_random(cx: &mut crate::AppContext, mut rng: StdRng) { let operations = env::var("OPERATIONS") .map(|i| i.parse().expect("invalid `OPERATIONS` variable")) .unwrap_or(10); diff --git a/crates/gpui/src/elements/text.rs b/crates/gpui/src/elements/text.rs index d18382052c..f5ece8ba2c 100644 --- a/crates/gpui/src/elements/text.rs +++ b/crates/gpui/src/elements/text.rs @@ -271,12 +271,10 @@ pub fn layout_highlighted_chunks<'a>( #[cfg(test)] mod tests { use super::*; - use crate::{ - elements::Empty, fonts, ElementBox, Entity, MutableAppContext, RenderContext, View, - }; + use crate::{elements::Empty, fonts, AppContext, ElementBox, Entity, RenderContext, View}; #[crate::test(self)] - fn test_soft_wrapping_with_carriage_returns(cx: &mut MutableAppContext) { + fn test_soft_wrapping_with_carriage_returns(cx: &mut AppContext) { let (window_id, _) = cx.add_window(Default::default(), |_| TestView); let mut presenter = cx.build_presenter(window_id, Default::default(), Default::default()); fonts::with_font_cache(cx.font_cache().clone(), || { diff --git a/crates/gpui/src/executor.rs b/crates/gpui/src/executor.rs index 16afa987e9..0629a411f8 100644 --- a/crates/gpui/src/executor.rs +++ b/crates/gpui/src/executor.rs @@ -17,7 +17,7 @@ use std::{ use crate::{ platform::{self, Dispatcher}, - util, MutableAppContext, + util, AppContext, }; pub enum Foreground { @@ -931,7 +931,7 @@ impl Task { } impl Task> { - pub fn detach_and_log_err(self, cx: &mut MutableAppContext) { + pub fn detach_and_log_err(self, cx: &mut AppContext) { cx.spawn(|_| async move { if let Err(err) = self.await { log::error!("{}", err); diff --git a/crates/gpui/src/platform/mac/fonts.rs b/crates/gpui/src/platform/mac/fonts.rs index de1661b339..96916698fc 100644 --- a/crates/gpui/src/platform/mac/fonts.rs +++ b/crates/gpui/src/platform/mac/fonts.rs @@ -498,12 +498,12 @@ extern "C" { #[cfg(test)] mod tests { use super::*; - use crate::MutableAppContext; + use crate::AppContext; use font_kit::properties::{Style, Weight}; use platform::FontSystem as _; #[crate::test(self, retries = 5)] - fn test_layout_str(_: &mut MutableAppContext) { + fn test_layout_str(_: &mut AppContext) { // This is failing intermittently on CI and we don't have time to figure it out let fonts = FontSystem::new(); let menlo = fonts.load_family("Menlo", &Default::default()).unwrap(); diff --git a/crates/gpui/src/presenter.rs b/crates/gpui/src/presenter.rs index 71b79f9def..b68f8b9dd9 100644 --- a/crates/gpui/src/presenter.rs +++ b/crates/gpui/src/presenter.rs @@ -1,5 +1,5 @@ use crate::{ - app::{AppContext, MutableAppContext, WindowInvalidation}, + app::WindowInvalidation, elements::Element, font_cache::FontCache, geometry::rect::RectF, @@ -10,11 +10,11 @@ use crate::{ MouseMove, MouseMoveOut, MouseScrollWheel, MouseUp, MouseUpOut, Scene, }, text_layout::TextLayoutCache, - Action, AnyModelHandle, AnyViewHandle, AnyWeakModelHandle, AnyWeakViewHandle, Appearance, - AssetCache, ElementBox, Entity, FontSystem, ModelHandle, MouseButton, MouseMovedEvent, - MouseRegion, MouseRegionId, MouseState, ParentId, ReadModel, ReadView, RenderContext, - RenderParams, SceneBuilder, UpgradeModelHandle, UpgradeViewHandle, View, ViewHandle, - WeakModelHandle, WeakViewHandle, + Action, AnyModelHandle, AnyViewHandle, AnyWeakModelHandle, AnyWeakViewHandle, AppContext, + Appearance, AssetCache, ElementBox, Entity, FontSystem, ModelHandle, MouseButton, + MouseMovedEvent, MouseRegion, MouseRegionId, MouseState, ParentId, ReadModel, ReadView, + RenderContext, RenderParams, SceneBuilder, UpgradeModelHandle, UpgradeViewHandle, View, + ViewHandle, WeakModelHandle, WeakViewHandle, }; use anyhow::bail; use collections::{HashMap, HashSet}; @@ -56,7 +56,7 @@ impl Presenter { font_cache: Arc, text_layout_cache: TextLayoutCache, asset_cache: Arc, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Self { Self { window_id, @@ -80,7 +80,7 @@ impl Presenter { &mut self, invalidation: &mut WindowInvalidation, appearance: Appearance, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { cx.start_frame(); self.appearance = appearance; @@ -111,7 +111,7 @@ impl Presenter { &mut self, invalidation: &mut WindowInvalidation, appearance: Appearance, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { self.invalidate(invalidation, appearance, cx); for (view_id, view) in &mut self.rendered_views { @@ -138,7 +138,7 @@ impl Presenter { window_size: Vector2F, scale_factor: f32, refreshing: bool, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Scene { let mut scene_builder = SceneBuilder::new(scale_factor); @@ -169,7 +169,7 @@ impl Presenter { } } - fn layout(&mut self, window_size: Vector2F, refreshing: bool, cx: &mut MutableAppContext) { + fn layout(&mut self, window_size: Vector2F, refreshing: bool, cx: &mut AppContext) { if let Some(root_view_id) = cx.root_view_id(self.window_id) { self.build_layout_context(window_size, refreshing, cx) .layout(root_view_id, SizeConstraint::strict(window_size)); @@ -180,7 +180,7 @@ impl Presenter { &'a mut self, window_size: Vector2F, refreshing: bool, - cx: &'a mut MutableAppContext, + cx: &'a mut AppContext, ) -> LayoutContext<'a> { LayoutContext { window_id: self.window_id, @@ -206,7 +206,7 @@ impl Presenter { &'a mut self, scene: &'a mut SceneBuilder, window_size: Vector2F, - cx: &'a mut MutableAppContext, + cx: &'a mut AppContext, ) -> PaintContext { PaintContext { scene, @@ -234,7 +234,7 @@ impl Presenter { &mut self, event: Event, event_reused: bool, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> bool { let mut mouse_events = SmallVec::<[_; 2]>::new(); let mut notified_views: HashSet = Default::default(); @@ -557,7 +557,7 @@ impl Presenter { pub fn build_event_context<'a>( &'a mut self, notified_views: &'a mut HashSet, - cx: &'a mut MutableAppContext, + cx: &'a mut AppContext, ) -> EventContext<'a> { EventContext { font_cache: &self.font_cache, @@ -595,7 +595,7 @@ pub struct LayoutContext<'a> { pub font_system: Arc, pub text_layout_cache: &'a TextLayoutCache, pub asset_cache: &'a AssetCache, - pub app: &'a mut MutableAppContext, + pub app: &'a mut AppContext, pub refreshing: bool, pub window_size: Vector2F, titlebar_height: f32, @@ -692,7 +692,7 @@ impl<'a> LayoutContext<'a> { } impl<'a> Deref for LayoutContext<'a> { - type Target = MutableAppContext; + type Target = AppContext; fn deref(&self) -> &Self::Target { self.app @@ -804,7 +804,7 @@ impl<'a> Deref for PaintContext<'a> { pub struct EventContext<'a> { pub font_cache: &'a FontCache, pub text_layout_cache: &'a TextLayoutCache, - pub app: &'a mut MutableAppContext, + pub app: &'a mut AppContext, pub window_id: usize, pub notify_count: usize, view_stack: Vec, @@ -871,7 +871,7 @@ impl<'a> EventContext<'a> { } impl<'a> Deref for EventContext<'a> { - type Target = MutableAppContext; + type Target = AppContext; fn deref(&self) -> &Self::Target { self.app diff --git a/crates/gpui/src/test.rs b/crates/gpui/src/test.rs index d784d43ece..089fb86f90 100644 --- a/crates/gpui/src/test.rs +++ b/crates/gpui/src/test.rs @@ -18,7 +18,7 @@ use crate::{ executor::{self, ExecutorEvent}, platform, util::CwdBacktrace, - Element, ElementBox, Entity, FontCache, Handle, MutableAppContext, Platform, RenderContext, + AppContext, Element, ElementBox, Entity, FontCache, Handle, Platform, RenderContext, Subscription, TestAppContext, View, }; @@ -40,7 +40,7 @@ pub fn run_test( detect_nondeterminism: bool, test_fn: &mut (dyn RefUnwindSafe + Fn( - &mut MutableAppContext, + &mut AppContext, Rc, Arc, u64, diff --git a/crates/gpui/src/text_layout.rs b/crates/gpui/src/text_layout.rs index 72ea0c8919..aa93f3bfd4 100644 --- a/crates/gpui/src/text_layout.rs +++ b/crates/gpui/src/text_layout.rs @@ -660,7 +660,7 @@ mod tests { use crate::fonts::{Properties, Weight}; #[crate::test(self)] - fn test_wrap_line(cx: &mut crate::MutableAppContext) { + fn test_wrap_line(cx: &mut crate::AppContext) { let font_cache = cx.font_cache().clone(); let font_system = cx.platform().fonts(); let family = font_cache @@ -721,7 +721,7 @@ mod tests { } #[crate::test(self, retries = 5)] - fn test_wrap_shaped_line(cx: &mut crate::MutableAppContext) { + fn test_wrap_shaped_line(cx: &mut crate::AppContext) { // This is failing intermittently on CI and we don't have time to figure it out let font_cache = cx.font_cache().clone(); let font_system = cx.platform().fonts(); diff --git a/crates/gpui/src/views.rs b/crates/gpui/src/views.rs index 73f8a57518..fa9a63b3d3 100644 --- a/crates/gpui/src/views.rs +++ b/crates/gpui/src/views.rs @@ -2,6 +2,6 @@ mod select; pub use select::{ItemType, Select, SelectStyle}; -pub fn init(cx: &mut super::MutableAppContext) { +pub fn init(cx: &mut super::AppContext) { select::init(cx); } diff --git a/crates/gpui/src/views/select.rs b/crates/gpui/src/views/select.rs index 216491db0f..3b8f24fb7f 100644 --- a/crates/gpui/src/views/select.rs +++ b/crates/gpui/src/views/select.rs @@ -1,8 +1,8 @@ use serde::Deserialize; use crate::{ - actions, elements::*, impl_actions, AppContext, Entity, MouseButton, MutableAppContext, - RenderContext, View, ViewContext, WeakViewHandle, + actions, elements::*, impl_actions, AppContext, Entity, MouseButton, RenderContext, View, + ViewContext, WeakViewHandle, }; pub struct Select { @@ -12,7 +12,7 @@ pub struct Select { item_count: usize, is_open: bool, list_state: UniformListState, - build_style: Option SelectStyle>>, + build_style: Option SelectStyle>>, } #[derive(Clone, Default)] @@ -35,7 +35,7 @@ impl_actions!(select, [SelectItem]); pub enum Event {} -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(Select::toggle); cx.add_action(Select::select_item); } @@ -57,10 +57,7 @@ impl Select { } } - pub fn with_style( - mut self, - f: impl 'static + FnMut(&mut MutableAppContext) -> SelectStyle, - ) -> Self { + pub fn with_style(mut self, f: impl 'static + FnMut(&mut AppContext) -> SelectStyle) -> Self { self.build_style = Some(Box::new(f)); self } diff --git a/crates/journal/src/journal.rs b/crates/journal/src/journal.rs index 6db078ee93..bdc6f5893f 100644 --- a/crates/journal/src/journal.rs +++ b/crates/journal/src/journal.rs @@ -1,6 +1,6 @@ use chrono::{Datelike, Local, NaiveTime, Timelike}; use editor::{scroll::autoscroll::Autoscroll, Editor}; -use gpui::{actions, MutableAppContext}; +use gpui::{actions, AppContext}; use settings::{HourFormat, Settings}; use std::{ fs::OpenOptions, @@ -12,11 +12,11 @@ use workspace::AppState; actions!(journal, [NewJournalEntry]); -pub fn init(app_state: Arc, cx: &mut MutableAppContext) { +pub fn init(app_state: Arc, cx: &mut AppContext) { cx.add_global_action(move |_: &NewJournalEntry, cx| new_journal_entry(app_state.clone(), cx)); } -pub fn new_journal_entry(app_state: Arc, cx: &mut MutableAppContext) { +pub fn new_journal_entry(app_state: Arc, cx: &mut AppContext) { let settings = cx.global::(); let journal_dir = match journal_dir(&settings) { Some(journal_dir) => journal_dir, diff --git a/crates/language/src/buffer.rs b/crates/language/src/buffer.rs index de1fc009ad..8c9f34789f 100644 --- a/crates/language/src/buffer.rs +++ b/crates/language/src/buffer.rs @@ -15,7 +15,7 @@ use anyhow::{anyhow, Result}; use clock::ReplicaId; use fs::LineEnding; use futures::FutureExt as _; -use gpui::{fonts::HighlightStyle, AppContext, Entity, ModelContext, MutableAppContext, Task}; +use gpui::{fonts::HighlightStyle, AppContext, Entity, ModelContext, Task}; use parking_lot::Mutex; use settings::Settings; use similar::{ChangeTag, TextDiff}; @@ -232,7 +232,7 @@ pub trait LocalFile: File { fingerprint: RopeFingerprint, line_ending: LineEnding, mtime: SystemTime, - cx: &mut MutableAppContext, + cx: &mut AppContext, ); } diff --git a/crates/language/src/buffer_tests.rs b/crates/language/src/buffer_tests.rs index 765c0f9673..8d99b9bad7 100644 --- a/crates/language/src/buffer_tests.rs +++ b/crates/language/src/buffer_tests.rs @@ -2,7 +2,7 @@ use super::*; use clock::ReplicaId; use collections::BTreeMap; use fs::LineEnding; -use gpui::{ModelHandle, MutableAppContext}; +use gpui::{AppContext, ModelHandle}; use indoc::indoc; use proto::deserialize_operation; use rand::prelude::*; @@ -35,7 +35,7 @@ fn init_logger() { } #[gpui::test] -fn test_line_endings(cx: &mut gpui::MutableAppContext) { +fn test_line_endings(cx: &mut gpui::AppContext) { cx.set_global(Settings::test(cx)); cx.add_model(|cx| { let mut buffer = @@ -128,7 +128,7 @@ fn test_select_language() { } #[gpui::test] -fn test_edit_events(cx: &mut gpui::MutableAppContext) { +fn test_edit_events(cx: &mut gpui::AppContext) { let mut now = Instant::now(); let buffer_1_events = Rc::new(RefCell::new(Vec::new())); let buffer_2_events = Rc::new(RefCell::new(Vec::new())); @@ -675,7 +675,7 @@ async fn test_symbols_containing(cx: &mut gpui::TestAppContext) { } #[gpui::test] -fn test_enclosing_bracket_ranges(cx: &mut MutableAppContext) { +fn test_enclosing_bracket_ranges(cx: &mut AppContext) { let mut assert = |selection_text, range_markers| { assert_bracket_pairs(selection_text, range_markers, rust_lang(), cx) }; @@ -791,9 +791,7 @@ fn test_enclosing_bracket_ranges(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_enclosing_bracket_ranges_where_brackets_are_not_outermost_children( - cx: &mut MutableAppContext, -) { +fn test_enclosing_bracket_ranges_where_brackets_are_not_outermost_children(cx: &mut AppContext) { let mut assert = |selection_text, bracket_pair_texts| { assert_bracket_pairs(selection_text, bracket_pair_texts, javascript_lang(), cx) }; @@ -825,7 +823,7 @@ fn test_enclosing_bracket_ranges_where_brackets_are_not_outermost_children( } #[gpui::test] -fn test_range_for_syntax_ancestor(cx: &mut MutableAppContext) { +fn test_range_for_syntax_ancestor(cx: &mut AppContext) { cx.add_model(|cx| { let text = "fn a() { b(|c| {}) }"; let buffer = Buffer::new(0, text, cx).with_language(Arc::new(rust_lang()), cx); @@ -863,7 +861,7 @@ fn test_range_for_syntax_ancestor(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_autoindent_with_soft_tabs(cx: &mut MutableAppContext) { +fn test_autoindent_with_soft_tabs(cx: &mut AppContext) { let settings = Settings::test(cx); cx.set_global(settings); @@ -904,7 +902,7 @@ fn test_autoindent_with_soft_tabs(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_autoindent_with_hard_tabs(cx: &mut MutableAppContext) { +fn test_autoindent_with_hard_tabs(cx: &mut AppContext) { let mut settings = Settings::test(cx); settings.editor_overrides.hard_tabs = Some(true); cx.set_global(settings); @@ -946,7 +944,7 @@ fn test_autoindent_with_hard_tabs(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_autoindent_does_not_adjust_lines_with_unchanged_suggestion(cx: &mut MutableAppContext) { +fn test_autoindent_does_not_adjust_lines_with_unchanged_suggestion(cx: &mut AppContext) { let settings = Settings::test(cx); cx.set_global(settings); @@ -1083,7 +1081,7 @@ fn test_autoindent_does_not_adjust_lines_with_unchanged_suggestion(cx: &mut Muta } #[gpui::test] -fn test_autoindent_does_not_adjust_lines_within_newly_created_errors(cx: &mut MutableAppContext) { +fn test_autoindent_does_not_adjust_lines_within_newly_created_errors(cx: &mut AppContext) { let settings = Settings::test(cx); cx.set_global(settings); @@ -1146,7 +1144,7 @@ fn test_autoindent_does_not_adjust_lines_within_newly_created_errors(cx: &mut Mu } #[gpui::test] -fn test_autoindent_adjusts_lines_when_only_text_changes(cx: &mut MutableAppContext) { +fn test_autoindent_adjusts_lines_when_only_text_changes(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); cx.add_model(|cx| { let mut buffer = Buffer::new( @@ -1202,7 +1200,7 @@ fn test_autoindent_adjusts_lines_when_only_text_changes(cx: &mut MutableAppConte } #[gpui::test] -fn test_autoindent_with_edit_at_end_of_buffer(cx: &mut MutableAppContext) { +fn test_autoindent_with_edit_at_end_of_buffer(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); cx.add_model(|cx| { let text = "a\nb"; @@ -1218,7 +1216,7 @@ fn test_autoindent_with_edit_at_end_of_buffer(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_autoindent_multi_line_insertion(cx: &mut MutableAppContext) { +fn test_autoindent_multi_line_insertion(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); cx.add_model(|cx| { let text = " @@ -1258,7 +1256,7 @@ fn test_autoindent_multi_line_insertion(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_autoindent_block_mode(cx: &mut MutableAppContext) { +fn test_autoindent_block_mode(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); cx.add_model(|cx| { let text = r#" @@ -1340,7 +1338,7 @@ fn test_autoindent_block_mode(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_autoindent_block_mode_without_original_indent_columns(cx: &mut MutableAppContext) { +fn test_autoindent_block_mode_without_original_indent_columns(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); cx.add_model(|cx| { let text = r#" @@ -1418,7 +1416,7 @@ fn test_autoindent_block_mode_without_original_indent_columns(cx: &mut MutableAp } #[gpui::test] -fn test_autoindent_language_without_indents_query(cx: &mut MutableAppContext) { +fn test_autoindent_language_without_indents_query(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); cx.add_model(|cx| { let text = " @@ -1461,7 +1459,7 @@ fn test_autoindent_language_without_indents_query(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_autoindent_with_injected_languages(cx: &mut MutableAppContext) { +fn test_autoindent_with_injected_languages(cx: &mut AppContext) { cx.set_global({ let mut settings = Settings::test(cx); settings.language_overrides.extend([ @@ -1575,7 +1573,7 @@ fn test_autoindent_with_injected_languages(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_autoindent_query_with_outdent_captures(cx: &mut MutableAppContext) { +fn test_autoindent_query_with_outdent_captures(cx: &mut AppContext) { let mut settings = Settings::test(cx); settings.editor_defaults.tab_size = Some(2.try_into().unwrap()); cx.set_global(settings); @@ -1618,7 +1616,7 @@ fn test_autoindent_query_with_outdent_captures(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_language_config_at(cx: &mut MutableAppContext) { +fn test_language_config_at(cx: &mut AppContext) { cx.set_global(Settings::test(cx)); cx.add_model(|cx| { let language = Language::new( @@ -1705,7 +1703,7 @@ fn test_language_config_at(cx: &mut MutableAppContext) { } #[gpui::test] -fn test_serialization(cx: &mut gpui::MutableAppContext) { +fn test_serialization(cx: &mut gpui::AppContext) { let mut now = Instant::now(); let buffer1 = cx.add_model(|cx| { @@ -1746,7 +1744,7 @@ fn test_serialization(cx: &mut gpui::MutableAppContext) { } #[gpui::test(iterations = 100)] -fn test_random_collaboration(cx: &mut MutableAppContext, mut rng: StdRng) { +fn test_random_collaboration(cx: &mut AppContext, mut rng: StdRng) { let min_peers = env::var("MIN_PEERS") .map(|i| i.parse().expect("invalid `MIN_PEERS` variable")) .unwrap_or(1); @@ -2199,7 +2197,7 @@ fn assert_bracket_pairs( selection_text: &'static str, bracket_pair_texts: Vec<&'static str>, language: Language, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { cx.set_global(Settings::test(cx)); let (expected_text, selection_ranges) = marked_text_ranges(selection_text, false); diff --git a/crates/language/src/language.rs b/crates/language/src/language.rs index beda6aa557..60bb2cfddf 100644 --- a/crates/language/src/language.rs +++ b/crates/language/src/language.rs @@ -16,7 +16,7 @@ use futures::{ future::{BoxFuture, Shared}, FutureExt, TryFutureExt as _, }; -use gpui::{executor::Background, MutableAppContext, Task}; +use gpui::{executor::Background, AppContext, Task}; use highlight_map::HighlightMap; use lazy_static::lazy_static; use lsp::CodeActionKind; @@ -147,7 +147,7 @@ impl CachedLspAdapter { pub fn workspace_configuration( &self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option> { self.adapter.workspace_configuration(cx) } @@ -223,10 +223,7 @@ pub trait LspAdapter: 'static + Send + Sync { None } - fn workspace_configuration( - &self, - _: &mut MutableAppContext, - ) -> Option> { + fn workspace_configuration(&self, _: &mut AppContext) -> Option> { None } @@ -584,7 +581,7 @@ impl LanguageRegistry { result } - pub fn workspace_configuration(&self, cx: &mut MutableAppContext) -> Task { + pub fn workspace_configuration(&self, cx: &mut AppContext) -> Task { let lsp_adapters = { let state = self.state.read(); state @@ -769,7 +766,7 @@ impl LanguageRegistry { language: Arc, root_path: Arc, http_client: Arc, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>> { #[cfg(any(test, feature = "test-support"))] if language.fake_adapter.is_some() { diff --git a/crates/language_selector/src/language_selector.rs b/crates/language_selector/src/language_selector.rs index b395a2b735..b281d621ba 100644 --- a/crates/language_selector/src/language_selector.rs +++ b/crates/language_selector/src/language_selector.rs @@ -5,7 +5,7 @@ use editor::Editor; use fuzzy::{match_strings, StringMatch, StringMatchCandidate}; use gpui::{ actions, elements::*, AnyViewHandle, AppContext, Entity, ModelHandle, MouseState, - MutableAppContext, RenderContext, View, ViewContext, ViewHandle, + RenderContext, View, ViewContext, ViewHandle, }; use language::{Buffer, LanguageRegistry}; use picker::{Picker, PickerDelegate}; @@ -16,7 +16,7 @@ use workspace::{AppState, Workspace}; actions!(language_selector, [Toggle]); -pub fn init(app_state: Arc, cx: &mut MutableAppContext) { +pub fn init(app_state: Arc, cx: &mut AppContext) { Picker::::init(cx); cx.add_action({ let language_registry = app_state.languages.clone(); diff --git a/crates/live_kit_client/examples/test_app.rs b/crates/live_kit_client/examples/test_app.rs index 14332b3e06..96480e92bc 100644 --- a/crates/live_kit_client/examples/test_app.rs +++ b/crates/live_kit_client/examples/test_app.rs @@ -89,6 +89,6 @@ fn main() { }); } -fn quit(_: &Quit, cx: &mut gpui::MutableAppContext) { +fn quit(_: &Quit, cx: &mut gpui::AppContext) { cx.platform().quit(); } diff --git a/crates/outline/src/outline.rs b/crates/outline/src/outline.rs index 52c709d91f..2259570fa0 100644 --- a/crates/outline/src/outline.rs +++ b/crates/outline/src/outline.rs @@ -5,7 +5,7 @@ use editor::{ use fuzzy::StringMatch; use gpui::{ actions, elements::*, geometry::vector::Vector2F, AnyViewHandle, AppContext, Entity, - MouseState, MutableAppContext, RenderContext, Task, View, ViewContext, ViewHandle, + MouseState, RenderContext, Task, View, ViewContext, ViewHandle, }; use language::Outline; use ordered_float::OrderedFloat; @@ -16,7 +16,7 @@ use workspace::Workspace; actions!(outline, [Toggle]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(OutlineView::toggle); Picker::::init(cx); } @@ -38,7 +38,7 @@ pub enum Event { impl Entity for OutlineView { type Event = Event; - fn release(&mut self, cx: &mut MutableAppContext) { + fn release(&mut self, cx: &mut AppContext) { self.restore_active_editor(cx); } } @@ -100,7 +100,7 @@ impl OutlineView { } } - fn restore_active_editor(&mut self, cx: &mut MutableAppContext) { + fn restore_active_editor(&mut self, cx: &mut AppContext) { self.active_editor.update(cx, |editor, cx| { editor.highlight_rows(None); if let Some(scroll_position) = self.prev_scroll_position { diff --git a/crates/picker/src/picker.rs b/crates/picker/src/picker.rs index bb591583e9..8e559053ea 100644 --- a/crates/picker/src/picker.rs +++ b/crates/picker/src/picker.rs @@ -4,8 +4,8 @@ use gpui::{ geometry::vector::{vec2f, Vector2F}, keymap_matcher::KeymapContext, platform::CursorStyle, - AnyViewHandle, AppContext, Axis, Entity, MouseButton, MouseState, MutableAppContext, - RenderContext, Task, View, ViewContext, ViewHandle, WeakViewHandle, + AnyViewHandle, AppContext, Axis, Entity, MouseButton, MouseState, RenderContext, Task, View, + ViewContext, ViewHandle, WeakViewHandle, }; use menu::{Cancel, Confirm, SelectFirst, SelectIndex, SelectLast, SelectNext, SelectPrev}; use parking_lot::Mutex; @@ -141,7 +141,7 @@ impl View for Picker { } impl Picker { - pub fn init(cx: &mut MutableAppContext) { + pub fn init(cx: &mut AppContext) { cx.add_action(Self::select_first); cx.add_action(Self::select_last); cx.add_action(Self::select_next); diff --git a/crates/project/src/project.rs b/crates/project/src/project.rs index b0a9784ba9..0fa7d0a63d 100644 --- a/crates/project/src/project.rs +++ b/crates/project/src/project.rs @@ -18,8 +18,8 @@ use futures::{ AsyncWriteExt, Future, FutureExt, StreamExt, TryFutureExt, }; use gpui::{ - AnyModelHandle, AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, - MutableAppContext, Task, UpgradeModelHandle, WeakModelHandle, + AnyModelHandle, AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, Task, + UpgradeModelHandle, WeakModelHandle, }; use language::{ point_to_lsp, @@ -414,7 +414,7 @@ impl Project { user_store: ModelHandle, languages: Arc, fs: Arc, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> ModelHandle { cx.add_model(|cx: &mut ModelContext| Self { worktrees: Default::default(), @@ -6519,7 +6519,7 @@ impl<'a> Iterator for PathMatchCandidateSetIter<'a> { impl Entity for Project { type Event = Event; - fn release(&mut self, _: &mut gpui::MutableAppContext) { + fn release(&mut self, _: &mut gpui::AppContext) { match &self.client_state { Some(ProjectClientState::Local { remote_id, .. }) => { let _ = self.client.send(proto::UnshareProject { @@ -6537,7 +6537,7 @@ impl Entity for Project { fn app_will_quit( &mut self, - _: &mut MutableAppContext, + _: &mut AppContext, ) -> Option>>> { let shutdown_futures = self .language_servers diff --git a/crates/project/src/project_tests.rs b/crates/project/src/project_tests.rs index 023ce3c5bc..95d49da1df 100644 --- a/crates/project/src/project_tests.rs +++ b/crates/project/src/project_tests.rs @@ -2,6 +2,7 @@ use crate::{worktree::WorktreeHandle, Event, *}; use fs::LineEnding; use fs::{FakeFs, RealFs}; use futures::{future, StreamExt}; +use gpui::AppContext; use gpui::{executor::Deterministic, test::subscribe}; use language::{ tree_sitter_rust, tree_sitter_typescript, Diagnostic, FakeLspAdapter, LanguageConfig, diff --git a/crates/project/src/worktree.rs b/crates/project/src/worktree.rs index 2357052d2c..3a9838c62d 100644 --- a/crates/project/src/worktree.rs +++ b/crates/project/src/worktree.rs @@ -16,10 +16,7 @@ use futures::{ }; use fuzzy::CharBag; use git::{DOT_GIT, GITIGNORE}; -use gpui::{ - executor, AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, MutableAppContext, - Task, -}; +use gpui::{executor, AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, Task}; use language::{ proto::{ deserialize_fingerprint, deserialize_version, serialize_fingerprint, serialize_line_ending, @@ -287,7 +284,7 @@ impl Worktree { replica_id: ReplicaId, worktree: proto::WorktreeMetadata, client: Arc, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> ModelHandle { cx.add_model(|cx: &mut ModelContext| { let snapshot = Snapshot { @@ -1896,7 +1893,7 @@ impl language::LocalFile for File { fingerprint: RopeFingerprint, line_ending: LineEnding, mtime: SystemTime, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { let worktree = self.worktree.read(cx).as_local().unwrap(); if let Some(project_id) = worktree.share.as_ref().map(|share| share.project_id) { diff --git a/crates/project_panel/src/project_panel.rs b/crates/project_panel/src/project_panel.rs index b6e6799062..dd08d68f71 100644 --- a/crates/project_panel/src/project_panel.rs +++ b/crates/project_panel/src/project_panel.rs @@ -13,8 +13,8 @@ use gpui::{ impl_internal_actions, keymap_matcher::KeymapContext, platform::CursorStyle, - AppContext, ClipboardItem, Element, ElementBox, Entity, ModelHandle, MouseButton, - MutableAppContext, PromptLevel, RenderContext, Task, View, ViewContext, ViewHandle, + AppContext, ClipboardItem, Element, ElementBox, Entity, ModelHandle, MouseButton, PromptLevel, + RenderContext, Task, View, ViewContext, ViewHandle, }; use menu::{Confirm, SelectNext, SelectPrev}; use project::{Entry, EntryKind, Project, ProjectEntryId, ProjectPath, Worktree, WorktreeId}; @@ -132,7 +132,7 @@ impl_internal_actions!( [Open, ToggleExpanded, DeployContextMenu, MoveProjectEntry] ); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ProjectPanel::deploy_context_menu); cx.add_action(ProjectPanel::expand_selected_entry); cx.add_action(ProjectPanel::collapse_selected_entry); diff --git a/crates/project_symbols/src/project_symbols.rs b/crates/project_symbols/src/project_symbols.rs index 3039f83b4a..a2e8012b89 100644 --- a/crates/project_symbols/src/project_symbols.rs +++ b/crates/project_symbols/src/project_symbols.rs @@ -5,7 +5,7 @@ use editor::{ use fuzzy::{StringMatch, StringMatchCandidate}; use gpui::{ actions, elements::*, AnyViewHandle, AppContext, Entity, ModelHandle, MouseState, - MutableAppContext, RenderContext, Task, View, ViewContext, ViewHandle, + RenderContext, Task, View, ViewContext, ViewHandle, }; use ordered_float::OrderedFloat; use picker::{Picker, PickerDelegate}; @@ -17,7 +17,7 @@ use workspace::Workspace; actions!(project_symbols, [Toggle]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ProjectSymbolsView::toggle); Picker::::init(cx); } diff --git a/crates/recent_projects/src/recent_projects.rs b/crates/recent_projects/src/recent_projects.rs index c9a501a2a6..3cb94ecee7 100644 --- a/crates/recent_projects/src/recent_projects.rs +++ b/crates/recent_projects/src/recent_projects.rs @@ -4,7 +4,7 @@ use fuzzy::{StringMatch, StringMatchCandidate}; use gpui::{ actions, elements::{ChildView, Flex, ParentElement}, - AnyViewHandle, Element, ElementBox, Entity, MutableAppContext, RenderContext, Task, View, + AnyViewHandle, Element, ElementBox, Entity, AppContext, RenderContext, Task, View, ViewContext, ViewHandle, }; use highlighted_workspace_location::HighlightedWorkspaceLocation; @@ -18,7 +18,7 @@ use workspace::{ actions!(projects, [OpenRecent]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(RecentProjectsView::toggle); Picker::::init(cx); } diff --git a/crates/search/src/buffer_search.rs b/crates/search/src/buffer_search.rs index 5fae189c68..488b47d2cb 100644 --- a/crates/search/src/buffer_search.rs +++ b/crates/search/src/buffer_search.rs @@ -6,8 +6,7 @@ use collections::HashMap; use editor::Editor; use gpui::{ actions, elements::*, impl_actions, platform::CursorStyle, Action, AnyViewHandle, AppContext, - Entity, MouseButton, MutableAppContext, RenderContext, Subscription, Task, View, ViewContext, - ViewHandle, + Entity, MouseButton, RenderContext, Subscription, Task, View, ViewContext, ViewHandle, }; use project::search::SearchQuery; use serde::Deserialize; @@ -31,7 +30,7 @@ pub enum Event { UpdateLocation, } -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(BufferSearchBar::deploy); cx.add_action(BufferSearchBar::dismiss); cx.add_action(BufferSearchBar::focus_editor); @@ -45,7 +44,7 @@ pub fn init(cx: &mut MutableAppContext) { add_toggle_option_action::(SearchOption::Regex, cx); } -fn add_toggle_option_action(option: SearchOption, cx: &mut MutableAppContext) { +fn add_toggle_option_action(option: SearchOption, cx: &mut AppContext) { cx.add_action(move |pane: &mut Pane, _: &A, cx: &mut ViewContext| { if let Some(search_bar) = pane.toolbar().read(cx).item_of_type::() { if search_bar.update(cx, |search_bar, cx| search_bar.show(false, false, cx)) { diff --git a/crates/search/src/project_search.rs b/crates/search/src/project_search.rs index 6735d99c95..1c8052174f 100644 --- a/crates/search/src/project_search.rs +++ b/crates/search/src/project_search.rs @@ -10,8 +10,8 @@ use editor::{ use futures::StreamExt; use gpui::{ actions, elements::*, platform::CursorStyle, Action, AnyViewHandle, AppContext, ElementBox, - Entity, ModelContext, ModelHandle, MouseButton, MutableAppContext, RenderContext, Subscription, - Task, View, ViewContext, ViewHandle, WeakModelHandle, WeakViewHandle, + Entity, ModelContext, ModelHandle, MouseButton, RenderContext, Subscription, Task, View, + ViewContext, ViewHandle, WeakModelHandle, WeakViewHandle, }; use menu::Confirm; use project::{search::SearchQuery, Project}; @@ -36,7 +36,7 @@ actions!(project_search, [SearchInNew, ToggleFocus]); #[derive(Default)] struct ActiveSearches(HashMap, WeakViewHandle>); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.set_global(ActiveSearches::default()); cx.add_action(ProjectSearchView::deploy); cx.add_action(ProjectSearchBar::search); @@ -50,7 +50,7 @@ pub fn init(cx: &mut MutableAppContext) { add_toggle_option_action::(SearchOption::Regex, cx); } -fn add_toggle_option_action(option: SearchOption, cx: &mut MutableAppContext) { +fn add_toggle_option_action(option: SearchOption, cx: &mut AppContext) { cx.add_action(move |pane: &mut Pane, _: &A, cx: &mut ViewContext| { if let Some(search_bar) = pane.toolbar().read(cx).item_of_type::() { if search_bar.update(cx, |search_bar, cx| { diff --git a/crates/search/src/search.rs b/crates/search/src/search.rs index 954aa0b500..90ea508cc6 100644 --- a/crates/search/src/search.rs +++ b/crates/search/src/search.rs @@ -1,11 +1,11 @@ pub use buffer_search::BufferSearchBar; -use gpui::{actions, Action, MutableAppContext}; +use gpui::{actions, Action, AppContext}; pub use project_search::{ProjectSearchBar, ProjectSearchView}; pub mod buffer_search; pub mod project_search; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { buffer_search::init(cx); project_search::init(cx); } diff --git a/crates/settings/src/keymap_file.rs b/crates/settings/src/keymap_file.rs index 2235bc351d..a45a53145e 100644 --- a/crates/settings/src/keymap_file.rs +++ b/crates/settings/src/keymap_file.rs @@ -2,7 +2,7 @@ use crate::{parse_json_with_comments, Settings}; use anyhow::{Context, Result}; use assets::Assets; use collections::BTreeMap; -use gpui::{keymap_matcher::Binding, MutableAppContext}; +use gpui::{keymap_matcher::Binding, AppContext}; use schemars::{ gen::{SchemaGenerator, SchemaSettings}, schema::{InstanceType, Schema, SchemaObject, SingleOrVec, SubschemaValidation}, @@ -41,7 +41,7 @@ impl JsonSchema for KeymapAction { struct ActionWithData(Box, Box); impl KeymapFileContent { - pub fn load_defaults(cx: &mut MutableAppContext) { + pub fn load_defaults(cx: &mut AppContext) { for path in ["keymaps/default.json", "keymaps/vim.json"] { Self::load(path, cx).unwrap(); } @@ -51,13 +51,13 @@ impl KeymapFileContent { } } - pub fn load(asset_path: &str, cx: &mut MutableAppContext) -> Result<()> { + pub fn load(asset_path: &str, cx: &mut AppContext) -> Result<()> { let content = Assets::get(asset_path).unwrap().data; let content_str = std::str::from_utf8(content.as_ref()).unwrap(); parse_json_with_comments::(content_str)?.add_to_cx(cx) } - pub fn add_to_cx(self, cx: &mut MutableAppContext) -> Result<()> { + pub fn add_to_cx(self, cx: &mut AppContext) -> Result<()> { for KeymapBlock { context, bindings } in self.0 { let bindings = bindings .into_iter() diff --git a/crates/settings/src/settings_file.rs b/crates/settings/src/settings_file.rs index 2ab5c10598..99c73ce81a 100644 --- a/crates/settings/src/settings_file.rs +++ b/crates/settings/src/settings_file.rs @@ -2,7 +2,7 @@ use crate::{update_settings_file, watched_json::WatchedJsonFile, SettingsFileCon use anyhow::Result; use assets::Assets; use fs::Fs; -use gpui::{AssetSource, MutableAppContext}; +use gpui::{AppContext, AssetSource}; use std::{io::ErrorKind, path::Path, sync::Arc}; // TODO: Switch SettingsFile to open a worktree and buffer for synchronization @@ -49,7 +49,7 @@ impl SettingsFile { } pub fn update( - cx: &mut MutableAppContext, + cx: &mut AppContext, update: impl 'static + Send + FnOnce(&mut SettingsFileContent), ) { let this = cx.global::(); @@ -211,7 +211,7 @@ mod tests { } fn assert_key_bindings_for<'a>( - cx: &mut MutableAppContext, + cx: &mut AppContext, actions: Vec<(&'static str, &'a dyn Action)>, line: u32, ) { diff --git a/crates/settings/src/watched_json.rs b/crates/settings/src/watched_json.rs index c4cc64cd62..16be82fa35 100644 --- a/crates/settings/src/watched_json.rs +++ b/crates/settings/src/watched_json.rs @@ -1,6 +1,6 @@ use fs::Fs; use futures::StreamExt; -use gpui::{executor, MutableAppContext}; +use gpui::{executor, AppContext}; use postage::sink::Sink as _; use postage::{prelude::Stream, watch}; use serde::Deserialize; @@ -67,7 +67,7 @@ pub fn watch_files( settings_file: WatchedJsonFile, theme_registry: Arc, keymap_file: WatchedJsonFile, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { watch_settings_file(defaults, settings_file, theme_registry, cx); watch_keymap_file(keymap_file, cx); @@ -77,7 +77,7 @@ pub(crate) fn watch_settings_file( defaults: Settings, mut file: WatchedJsonFile, theme_registry: Arc, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { settings_updated(&defaults, file.0.borrow().clone(), &theme_registry, cx); cx.spawn(|mut cx| async move { @@ -88,7 +88,7 @@ pub(crate) fn watch_settings_file( .detach(); } -fn keymap_updated(content: KeymapFileContent, cx: &mut MutableAppContext) { +fn keymap_updated(content: KeymapFileContent, cx: &mut AppContext) { cx.clear_bindings(); KeymapFileContent::load_defaults(cx); content.add_to_cx(cx).log_err(); @@ -98,7 +98,7 @@ fn settings_updated( defaults: &Settings, content: SettingsFileContent, theme_registry: &Arc, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { let mut settings = defaults.clone(); settings.set_user_settings(content, theme_registry, cx.font_cache()); @@ -106,7 +106,7 @@ fn settings_updated( cx.refresh_windows(); } -fn watch_keymap_file(mut file: WatchedJsonFile, cx: &mut MutableAppContext) { +fn watch_keymap_file(mut file: WatchedJsonFile, cx: &mut AppContext) { cx.spawn(|mut cx| async move { let mut settings_subscription = None; while let Some(content) = file.0.recv().await { diff --git a/crates/staff_mode/src/staff_mode.rs b/crates/staff_mode/src/staff_mode.rs index 3dbd6c2500..a222aafb02 100644 --- a/crates/staff_mode/src/staff_mode.rs +++ b/crates/staff_mode/src/staff_mode.rs @@ -1,4 +1,4 @@ -use gpui::MutableAppContext; +use gpui::AppContext; #[derive(Debug, Default)] pub struct StaffMode(pub bool); @@ -13,8 +13,8 @@ impl std::ops::Deref for StaffMode { /// Despite what the type system requires me to tell you, the init function will only be called a once /// as soon as we know that the staff mode is enabled. -pub fn staff_mode( - cx: &mut MutableAppContext, +pub fn staff_mode( + cx: &mut AppContext, mut init: F, ) { if **cx.default_global::() { @@ -32,8 +32,8 @@ pub fn staff_mode( /// Immediately checks and runs the init function if the staff mode is not enabled. /// This is only included for symettry with staff_mode() above -pub fn not_staff_mode( - cx: &mut MutableAppContext, +pub fn not_staff_mode( + cx: &mut AppContext, init: F, ) { if !**cx.default_global::() { diff --git a/crates/terminal_view/src/terminal_button.rs b/crates/terminal_view/src/terminal_button.rs index 87f0a2deba..934ba4f107 100644 --- a/crates/terminal_view/src/terminal_button.rs +++ b/crates/terminal_view/src/terminal_button.rs @@ -1,7 +1,7 @@ use context_menu::{ContextMenu, ContextMenuItem}; use gpui::{ elements::*, impl_internal_actions, CursorStyle, Element, ElementBox, Entity, MouseButton, - MutableAppContext, RenderContext, View, ViewContext, ViewHandle, WeakModelHandle, + AppContext, RenderContext, View, ViewContext, ViewHandle, WeakModelHandle, WeakViewHandle, }; use settings::Settings; @@ -21,7 +21,7 @@ pub struct FocusTerminal { impl_internal_actions!(terminal, [FocusTerminal, DeployTerminalMenu]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(TerminalButton::deploy_terminal_menu); cx.add_action(TerminalButton::focus_terminal); } diff --git a/crates/terminal_view/src/terminal_view.rs b/crates/terminal_view/src/terminal_view.rs index e325cddc45..df2cdcc830 100644 --- a/crates/terminal_view/src/terminal_view.rs +++ b/crates/terminal_view/src/terminal_view.rs @@ -16,8 +16,8 @@ use gpui::{ geometry::vector::Vector2F, impl_actions, impl_internal_actions, keymap_matcher::{KeymapContext, Keystroke}, - AnyViewHandle, AppContext, Element, ElementBox, Entity, ModelHandle, MutableAppContext, Task, - View, ViewContext, ViewHandle, WeakViewHandle, + AnyViewHandle, AppContext, Element, ElementBox, Entity, ModelHandle, Task, View, ViewContext, + ViewHandle, WeakViewHandle, }; use project::{LocalWorktree, Project}; use serde::Deserialize; @@ -68,7 +68,7 @@ impl_actions!(terminal, [SendText, SendKeystroke]); impl_internal_actions!(project_panel, [DeployContextMenu]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(TerminalView::deploy); register_deserializable_item::(cx); diff --git a/crates/theme_selector/src/theme_selector.rs b/crates/theme_selector/src/theme_selector.rs index ca7fc41cbe..f2bdc130bf 100644 --- a/crates/theme_selector/src/theme_selector.rs +++ b/crates/theme_selector/src/theme_selector.rs @@ -1,7 +1,7 @@ use fuzzy::{match_strings, StringMatch, StringMatchCandidate}; use gpui::{ actions, elements::*, AnyViewHandle, AppContext, Element, ElementBox, Entity, MouseState, - MutableAppContext, RenderContext, View, ViewContext, ViewHandle, + RenderContext, View, ViewContext, ViewHandle, }; use picker::{Picker, PickerDelegate}; use settings::{settings_file::SettingsFile, Settings}; @@ -22,7 +22,7 @@ pub struct ThemeSelector { actions!(theme_selector, [Toggle, Reload]); -pub fn init(app_state: Arc, cx: &mut MutableAppContext) { +pub fn init(app_state: Arc, cx: &mut AppContext) { Picker::::init(cx); cx.add_action({ let theme_registry = app_state.themes.clone(); @@ -83,7 +83,7 @@ impl ThemeSelector { } #[cfg(debug_assertions)] - pub fn reload(themes: Arc, cx: &mut MutableAppContext) { + pub fn reload(themes: Arc, cx: &mut AppContext) { let current_theme_name = cx.global::().theme.meta.name.clone(); themes.clear(); match themes.get(¤t_theme_name) { @@ -131,7 +131,7 @@ impl ThemeSelector { } } - fn set_theme(theme: Arc, cx: &mut MutableAppContext) { + fn set_theme(theme: Arc, cx: &mut AppContext) { cx.update_global::(|settings, cx| { settings.theme = theme; cx.refresh_windows(); @@ -243,7 +243,7 @@ impl PickerDelegate for ThemeSelector { impl Entity for ThemeSelector { type Event = Event; - fn release(&mut self, cx: &mut MutableAppContext) { + fn release(&mut self, cx: &mut AppContext) { if !self.selection_completed { Self::set_theme(self.original_theme.clone(), cx); } diff --git a/crates/theme_testbench/src/theme_testbench.rs b/crates/theme_testbench/src/theme_testbench.rs index ad01f51a15..963b06083c 100644 --- a/crates/theme_testbench/src/theme_testbench.rs +++ b/crates/theme_testbench/src/theme_testbench.rs @@ -6,8 +6,8 @@ use gpui::{ Padding, ParentElement, }, fonts::TextStyle, - AppContext, Border, Element, Entity, ModelHandle, MutableAppContext, Quad, RenderContext, Task, - View, ViewContext, ViewHandle, WeakViewHandle, + AppContext, Border, Element, Entity, ModelHandle, Quad, RenderContext, Task, View, ViewContext, + ViewHandle, WeakViewHandle, }; use project::Project; use settings::Settings; @@ -16,7 +16,7 @@ use workspace::{item::Item, register_deserializable_item, Pane, Workspace}; actions!(theme, [DeployThemeTestbench]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(ThemeTestbench::deploy); register_deserializable_item::(cx) diff --git a/crates/vim/src/editor_events.rs b/crates/vim/src/editor_events.rs index c58f66478f..dc253254cc 100644 --- a/crates/vim/src/editor_events.rs +++ b/crates/vim/src/editor_events.rs @@ -1,15 +1,15 @@ use editor::{EditorBlurred, EditorFocused, EditorMode, EditorReleased, Event}; -use gpui::MutableAppContext; +use gpui::AppContext; use crate::{state::Mode, Vim}; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.subscribe_global(focused).detach(); cx.subscribe_global(blurred).detach(); cx.subscribe_global(released).detach(); } -fn focused(EditorFocused(editor): &EditorFocused, cx: &mut MutableAppContext) { +fn focused(EditorFocused(editor): &EditorFocused, cx: &mut AppContext) { Vim::update(cx, |vim, cx| { if let Some(previously_active_editor) = vim .active_editor @@ -48,7 +48,7 @@ fn focused(EditorFocused(editor): &EditorFocused, cx: &mut MutableAppContext) { }); } -fn blurred(EditorBlurred(editor): &EditorBlurred, cx: &mut MutableAppContext) { +fn blurred(EditorBlurred(editor): &EditorBlurred, cx: &mut AppContext) { Vim::update(cx, |vim, cx| { if let Some(previous_editor) = vim.active_editor.clone() { if previous_editor == editor.clone() { @@ -59,7 +59,7 @@ fn blurred(EditorBlurred(editor): &EditorBlurred, cx: &mut MutableAppContext) { }) } -fn released(EditorReleased(editor): &EditorReleased, cx: &mut MutableAppContext) { +fn released(EditorReleased(editor): &EditorReleased, cx: &mut AppContext) { cx.update_default_global(|vim: &mut Vim, _| { if let Some(previous_editor) = vim.active_editor.clone() { if previous_editor == editor.clone() { @@ -69,7 +69,7 @@ fn released(EditorReleased(editor): &EditorReleased, cx: &mut MutableAppContext) }); } -fn local_selections_changed(newest_empty: bool, cx: &mut MutableAppContext) { +fn local_selections_changed(newest_empty: bool, cx: &mut AppContext) { Vim::update(cx, |vim, cx| { if vim.enabled && vim.state.mode == Mode::Normal && !newest_empty { vim.switch_mode(Mode::Visual { line: false }, false, cx) diff --git a/crates/vim/src/insert.rs b/crates/vim/src/insert.rs index d8aea4aa33..537f6a15f1 100644 --- a/crates/vim/src/insert.rs +++ b/crates/vim/src/insert.rs @@ -1,12 +1,12 @@ use crate::{state::Mode, Vim}; use editor::{scroll::autoscroll::Autoscroll, Bias}; -use gpui::{actions, MutableAppContext, ViewContext}; +use gpui::{actions, AppContext, ViewContext}; use language::SelectionGoal; use workspace::Workspace; actions!(vim, [NormalBefore]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(normal_before); } diff --git a/crates/vim/src/motion.rs b/crates/vim/src/motion.rs index 1794e999a2..e919cf8096 100644 --- a/crates/vim/src/motion.rs +++ b/crates/vim/src/motion.rs @@ -5,7 +5,7 @@ use editor::{ display_map::{DisplaySnapshot, ToDisplayPoint}, movement, Bias, CharKind, DisplayPoint, ToOffset, }; -use gpui::{actions, impl_actions, MutableAppContext}; +use gpui::{actions, impl_actions, AppContext}; use language::{Point, Selection, SelectionGoal}; use serde::Deserialize; use workspace::Workspace; @@ -80,7 +80,7 @@ actions!( ); impl_actions!(vim, [NextWordStart, NextWordEnd, PreviousWordStart]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(|_: &mut Workspace, _: &Left, cx: _| motion(Motion::Left, cx)); cx.add_action(|_: &mut Workspace, _: &Backspace, cx: _| motion(Motion::Backspace, cx)); cx.add_action(|_: &mut Workspace, _: &Down, cx: _| motion(Motion::Down, cx)); @@ -116,7 +116,7 @@ pub fn init(cx: &mut MutableAppContext) { cx.add_action(|_: &mut Workspace, &NextLineStart, cx: _| motion(Motion::NextLineStart, cx)) } -pub(crate) fn motion(motion: Motion, cx: &mut MutableAppContext) { +pub(crate) fn motion(motion: Motion, cx: &mut AppContext) { if let Some(Operator::Namespace(_)) | Some(Operator::FindForward { .. }) | Some(Operator::FindBackward { .. }) = Vim::read(cx).active_operator() diff --git a/crates/vim/src/normal.rs b/crates/vim/src/normal.rs index 55134e2149..ef73f2e9b4 100644 --- a/crates/vim/src/normal.rs +++ b/crates/vim/src/normal.rs @@ -16,7 +16,7 @@ use editor::{ scroll::{autoscroll::Autoscroll, scroll_amount::ScrollAmount}, Anchor, Bias, ClipboardSelection, DisplayPoint, Editor, }; -use gpui::{actions, impl_actions, MutableAppContext, ViewContext}; +use gpui::{actions, impl_actions, AppContext, ViewContext}; use language::{AutoindentMode, Point, SelectionGoal}; use log::error; use serde::Deserialize; @@ -50,7 +50,7 @@ actions!( impl_actions!(vim, [Scroll]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(insert_after); cx.add_action(insert_first_non_whitespace); cx.add_action(insert_end_of_line); @@ -94,7 +94,7 @@ pub fn normal_motion( motion: Motion, operator: Option, times: usize, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { Vim::update(cx, |vim, cx| { match operator { @@ -110,7 +110,7 @@ pub fn normal_motion( }); } -pub fn normal_object(object: Object, cx: &mut MutableAppContext) { +pub fn normal_object(object: Object, cx: &mut AppContext) { Vim::update(cx, |vim, cx| { match vim.state.operator_stack.pop() { Some(Operator::Object { around }) => match vim.state.operator_stack.pop() { @@ -129,7 +129,7 @@ pub fn normal_object(object: Object, cx: &mut MutableAppContext) { }) } -fn move_cursor(vim: &mut Vim, motion: Motion, times: usize, cx: &mut MutableAppContext) { +fn move_cursor(vim: &mut Vim, motion: Motion, times: usize, cx: &mut AppContext) { vim.update_active_editor(cx, |editor, cx| { editor.change_selections(Some(Autoscroll::fit()), cx, |s| { s.move_cursors_with(|map, cursor, goal| { @@ -424,7 +424,7 @@ fn scroll(editor: &mut Editor, amount: &ScrollAmount, cx: &mut ViewContext, cx: &mut MutableAppContext) { +pub(crate) fn normal_replace(text: Arc, cx: &mut AppContext) { Vim::update(cx, |vim, cx| { vim.update_active_editor(cx, |editor, cx| { editor.transact(cx, |editor, cx| { diff --git a/crates/vim/src/normal/change.rs b/crates/vim/src/normal/change.rs index ca372801c7..899fea8b73 100644 --- a/crates/vim/src/normal/change.rs +++ b/crates/vim/src/normal/change.rs @@ -3,10 +3,10 @@ use editor::{ char_kind, display_map::DisplaySnapshot, movement, scroll::autoscroll::Autoscroll, CharKind, DisplayPoint, }; -use gpui::MutableAppContext; +use gpui::AppContext; use language::Selection; -pub fn change_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut MutableAppContext) { +pub fn change_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut AppContext) { // Some motions ignore failure when switching to normal mode let mut motion_succeeded = matches!( motion, @@ -38,7 +38,7 @@ pub fn change_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut Mutab } } -pub fn change_object(vim: &mut Vim, object: Object, around: bool, cx: &mut MutableAppContext) { +pub fn change_object(vim: &mut Vim, object: Object, around: bool, cx: &mut AppContext) { let mut objects_found = false; vim.update_active_editor(cx, |editor, cx| { // We are swapping to insert mode anyway. Just set the line end clipping behavior now diff --git a/crates/vim/src/normal/delete.rs b/crates/vim/src/normal/delete.rs index b22579438f..1be264b7fd 100644 --- a/crates/vim/src/normal/delete.rs +++ b/crates/vim/src/normal/delete.rs @@ -1,9 +1,9 @@ use crate::{motion::Motion, object::Object, utils::copy_selections_content, Vim}; use collections::{HashMap, HashSet}; use editor::{display_map::ToDisplayPoint, scroll::autoscroll::Autoscroll, Bias}; -use gpui::MutableAppContext; +use gpui::AppContext; -pub fn delete_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut MutableAppContext) { +pub fn delete_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut AppContext) { vim.update_active_editor(cx, |editor, cx| { editor.transact(cx, |editor, cx| { editor.set_clip_at_line_ends(false, cx); @@ -36,7 +36,7 @@ pub fn delete_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut Mutab }); } -pub fn delete_object(vim: &mut Vim, object: Object, around: bool, cx: &mut MutableAppContext) { +pub fn delete_object(vim: &mut Vim, object: Object, around: bool, cx: &mut AppContext) { vim.update_active_editor(cx, |editor, cx| { editor.transact(cx, |editor, cx| { editor.set_clip_at_line_ends(false, cx); diff --git a/crates/vim/src/normal/yank.rs b/crates/vim/src/normal/yank.rs index e7d6b3076b..610b137d49 100644 --- a/crates/vim/src/normal/yank.rs +++ b/crates/vim/src/normal/yank.rs @@ -1,8 +1,8 @@ use crate::{motion::Motion, object::Object, utils::copy_selections_content, Vim}; use collections::HashMap; -use gpui::MutableAppContext; +use gpui::AppContext; -pub fn yank_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut MutableAppContext) { +pub fn yank_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut AppContext) { vim.update_active_editor(cx, |editor, cx| { editor.transact(cx, |editor, cx| { editor.set_clip_at_line_ends(false, cx); @@ -25,7 +25,7 @@ pub fn yank_motion(vim: &mut Vim, motion: Motion, times: usize, cx: &mut Mutable }); } -pub fn yank_object(vim: &mut Vim, object: Object, around: bool, cx: &mut MutableAppContext) { +pub fn yank_object(vim: &mut Vim, object: Object, around: bool, cx: &mut AppContext) { vim.update_active_editor(cx, |editor, cx| { editor.transact(cx, |editor, cx| { editor.set_clip_at_line_ends(false, cx); diff --git a/crates/vim/src/object.rs b/crates/vim/src/object.rs index 81aee220a9..c7ad577d55 100644 --- a/crates/vim/src/object.rs +++ b/crates/vim/src/object.rs @@ -1,7 +1,7 @@ use std::ops::Range; use editor::{char_kind, display_map::DisplaySnapshot, movement, Bias, CharKind, DisplayPoint}; -use gpui::{actions, impl_actions, MutableAppContext}; +use gpui::{actions, impl_actions, AppContext}; use language::Selection; use serde::Deserialize; use workspace::Workspace; @@ -43,7 +43,7 @@ actions!( ); impl_actions!(vim, [Word]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action( |_: &mut Workspace, &Word { ignore_punctuation }: &Word, cx: _| { object(Object::Word { ignore_punctuation }, cx) @@ -61,7 +61,7 @@ pub fn init(cx: &mut MutableAppContext) { cx.add_action(|_: &mut Workspace, _: &AngleBrackets, cx: _| object(Object::AngleBrackets, cx)); } -fn object(object: Object, cx: &mut MutableAppContext) { +fn object(object: Object, cx: &mut AppContext) { match Vim::read(cx).state.mode { Mode::Normal => normal_object(object, cx), Mode::Visual { .. } => visual_object(object, cx), diff --git a/crates/vim/src/utils.rs b/crates/vim/src/utils.rs index 6f682f6146..7f9a549367 100644 --- a/crates/vim/src/utils.rs +++ b/crates/vim/src/utils.rs @@ -1,7 +1,7 @@ use editor::{ClipboardSelection, Editor}; -use gpui::{ClipboardItem, MutableAppContext}; +use gpui::{AppContext, ClipboardItem}; -pub fn copy_selections_content(editor: &mut Editor, linewise: bool, cx: &mut MutableAppContext) { +pub fn copy_selections_content(editor: &mut Editor, linewise: bool, cx: &mut AppContext) { let selections = editor.selections.all_adjusted(cx); let buffer = editor.buffer().read(cx).snapshot(cx); let mut text = String::new(); diff --git a/crates/vim/src/vim.rs b/crates/vim/src/vim.rs index 34afcb5f84..b553596421 100644 --- a/crates/vim/src/vim.rs +++ b/crates/vim/src/vim.rs @@ -15,7 +15,7 @@ use std::sync::Arc; use collections::CommandPaletteFilter; use editor::{Bias, Cancel, Editor, EditorMode}; use gpui::{ - actions, impl_actions, MutableAppContext, Subscription, ViewContext, ViewHandle, WeakViewHandle, + actions, impl_actions, AppContext, Subscription, ViewContext, ViewHandle, WeakViewHandle, }; use language::CursorShape; use motion::Motion; @@ -38,7 +38,7 @@ struct Number(u8); actions!(vim, [Tab, Enter]); impl_actions!(vim, [Number, SwitchMode, PushOperator]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { editor_events::init(cx); normal::init(cx); visual::init(cx); @@ -65,7 +65,7 @@ pub fn init(cx: &mut MutableAppContext) { // Otherwise forward cancel on to the editor let vim = Vim::read(cx); if vim.state.mode != Mode::Normal || vim.active_operator().is_some() { - MutableAppContext::defer(cx, |cx| { + AppContext::defer(cx, |cx| { Vim::update(cx, |state, cx| { state.switch_mode(Mode::Normal, false, cx); }); @@ -95,7 +95,7 @@ pub fn init(cx: &mut MutableAppContext) { .detach(); } -pub fn observe_keystrokes(window_id: usize, cx: &mut MutableAppContext) { +pub fn observe_keystrokes(window_id: usize, cx: &mut AppContext) { cx.observe_keystrokes(window_id, |_keystroke, _result, handled_by, cx| { if let Some(handled_by) = handled_by { // Keystroke is handled by the vim system, so continue forward @@ -131,20 +131,20 @@ pub struct Vim { } impl Vim { - fn read(cx: &mut MutableAppContext) -> &Self { + fn read(cx: &mut AppContext) -> &Self { cx.default_global() } - fn update(cx: &mut MutableAppContext, update: F) -> S + fn update(cx: &mut AppContext, update: F) -> S where - F: FnOnce(&mut Self, &mut MutableAppContext) -> S, + F: FnOnce(&mut Self, &mut AppContext) -> S, { cx.update_default_global(update) } fn update_active_editor( &self, - cx: &mut MutableAppContext, + cx: &mut AppContext, update: impl FnOnce(&mut Editor, &mut ViewContext) -> S, ) -> Option { self.active_editor @@ -153,7 +153,7 @@ impl Vim { .map(|ae| ae.update(cx, update)) } - fn switch_mode(&mut self, mode: Mode, leave_selections: bool, cx: &mut MutableAppContext) { + fn switch_mode(&mut self, mode: Mode, leave_selections: bool, cx: &mut AppContext) { self.state.mode = mode; self.state.operator_stack.clear(); @@ -188,12 +188,12 @@ impl Vim { } } - fn push_operator(&mut self, operator: Operator, cx: &mut MutableAppContext) { + fn push_operator(&mut self, operator: Operator, cx: &mut AppContext) { self.state.operator_stack.push(operator); self.sync_vim_settings(cx); } - fn push_number(&mut self, Number(number): &Number, cx: &mut MutableAppContext) { + fn push_number(&mut self, Number(number): &Number, cx: &mut AppContext) { if let Some(Operator::Number(current_number)) = self.active_operator() { self.pop_operator(cx); self.push_operator(Operator::Number(current_number * 10 + *number as usize), cx); @@ -202,14 +202,14 @@ impl Vim { } } - fn pop_operator(&mut self, cx: &mut MutableAppContext) -> Operator { + fn pop_operator(&mut self, cx: &mut AppContext) -> Operator { let popped_operator = self.state.operator_stack.pop() .expect("Operator popped when no operator was on the stack. This likely means there is an invalid keymap config"); self.sync_vim_settings(cx); popped_operator } - fn pop_number_operator(&mut self, cx: &mut MutableAppContext) -> usize { + fn pop_number_operator(&mut self, cx: &mut AppContext) -> usize { let mut times = 1; if let Some(Operator::Number(number)) = self.active_operator() { times = number; @@ -218,7 +218,7 @@ impl Vim { times } - fn clear_operator(&mut self, cx: &mut MutableAppContext) { + fn clear_operator(&mut self, cx: &mut AppContext) { self.state.operator_stack.clear(); self.sync_vim_settings(cx); } @@ -227,7 +227,7 @@ impl Vim { self.state.operator_stack.last().copied() } - fn active_editor_input_ignored(text: Arc, cx: &mut MutableAppContext) { + fn active_editor_input_ignored(text: Arc, cx: &mut AppContext) { if text.is_empty() { return; } @@ -248,7 +248,7 @@ impl Vim { } } - fn set_enabled(&mut self, enabled: bool, cx: &mut MutableAppContext) { + fn set_enabled(&mut self, enabled: bool, cx: &mut AppContext) { if self.enabled != enabled { self.enabled = enabled; self.state = Default::default(); @@ -259,7 +259,7 @@ impl Vim { } } - fn sync_vim_settings(&self, cx: &mut MutableAppContext) { + fn sync_vim_settings(&self, cx: &mut AppContext) { let state = &self.state; let cursor_shape = state.cursor_shape(); @@ -291,7 +291,7 @@ impl Vim { } } - fn unhook_vim_settings(&self, editor: ViewHandle, cx: &mut MutableAppContext) { + fn unhook_vim_settings(&self, editor: ViewHandle, cx: &mut AppContext) { editor.update(cx, |editor, cx| { editor.set_cursor_shape(CursorShape::Bar, cx); editor.set_clip_at_line_ends(false, cx); diff --git a/crates/vim/src/visual.rs b/crates/vim/src/visual.rs index 2180fbdabb..80ddae69be 100644 --- a/crates/vim/src/visual.rs +++ b/crates/vim/src/visual.rs @@ -4,7 +4,7 @@ use collections::HashMap; use editor::{ display_map::ToDisplayPoint, movement, scroll::autoscroll::Autoscroll, Bias, ClipboardSelection, }; -use gpui::{actions, MutableAppContext, ViewContext}; +use gpui::{actions, AppContext, ViewContext}; use language::{AutoindentMode, SelectionGoal}; use workspace::Workspace; @@ -18,14 +18,14 @@ use crate::{ actions!(vim, [VisualDelete, VisualChange, VisualYank, VisualPaste]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(change); cx.add_action(delete); cx.add_action(yank); cx.add_action(paste); } -pub fn visual_motion(motion: Motion, times: usize, cx: &mut MutableAppContext) { +pub fn visual_motion(motion: Motion, times: usize, cx: &mut AppContext) { Vim::update(cx, |vim, cx| { vim.update_active_editor(cx, |editor, cx| { editor.change_selections(Some(Autoscroll::fit()), cx, |s| { @@ -56,7 +56,7 @@ pub fn visual_motion(motion: Motion, times: usize, cx: &mut MutableAppContext) { }); } -pub fn visual_object(object: Object, cx: &mut MutableAppContext) { +pub fn visual_object(object: Object, cx: &mut AppContext) { Vim::update(cx, |vim, cx| { if let Operator::Object { around } = vim.pop_operator(cx) { vim.update_active_editor(cx, |editor, cx| { @@ -313,7 +313,7 @@ pub fn paste(_: &mut Workspace, _: &VisualPaste, cx: &mut ViewContext }); } -pub(crate) fn visual_replace(text: Arc, line: bool, cx: &mut MutableAppContext) { +pub(crate) fn visual_replace(text: Arc, line: bool, cx: &mut AppContext) { Vim::update(cx, |vim, cx| { vim.update_active_editor(cx, |editor, cx| { editor.transact(cx, |editor, cx| { @@ -648,7 +648,7 @@ mod test { cx.assert_state( indoc! {" The quick brown - the + the ˇfox jumps over dog"}, Mode::Normal, diff --git a/crates/welcome/src/base_keymap_picker.rs b/crates/welcome/src/base_keymap_picker.rs index dd7ca341b3..01d6d41c64 100644 --- a/crates/welcome/src/base_keymap_picker.rs +++ b/crates/welcome/src/base_keymap_picker.rs @@ -2,7 +2,7 @@ use fuzzy::{match_strings, StringMatch, StringMatchCandidate}; use gpui::{ actions, elements::{ChildView, Element as _, Label}, - AnyViewHandle, Entity, MutableAppContext, View, ViewContext, ViewHandle, + AnyViewHandle, Entity, AppContext, View, ViewContext, ViewHandle, }; use picker::{Picker, PickerDelegate}; use settings::{settings_file::SettingsFile, BaseKeymap, Settings}; @@ -16,7 +16,7 @@ pub struct BaseKeymapSelector { actions!(welcome, [ToggleBaseKeymapSelector]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { Picker::::init(cx); cx.add_action({ move |workspace, _: &ToggleBaseKeymapSelector, cx| BaseKeymapSelector::toggle(workspace, cx) diff --git a/crates/welcome/src/welcome.rs b/crates/welcome/src/welcome.rs index 33d93dcd87..b5941903f4 100644 --- a/crates/welcome/src/welcome.rs +++ b/crates/welcome/src/welcome.rs @@ -5,7 +5,7 @@ use std::sync::Arc; use db::kvp::KEY_VALUE_STORE; use gpui::{ elements::{Flex, Label, ParentElement}, - Element, ElementBox, Entity, MutableAppContext, Subscription, View, ViewContext, + Element, ElementBox, Entity, AppContext, Subscription, View, ViewContext, }; use settings::{settings_file::SettingsFile, Settings}; @@ -18,7 +18,7 @@ use crate::base_keymap_picker::ToggleBaseKeymapSelector; pub const FIRST_OPEN: &str = "first_open"; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(|workspace: &mut Workspace, _: &Welcome, cx| { let welcome_page = cx.add_view(WelcomePage::new); workspace.add_item(Box::new(welcome_page), cx) @@ -27,7 +27,7 @@ pub fn init(cx: &mut MutableAppContext) { base_keymap_picker::init(cx); } -pub fn show_welcome_experience(app_state: &Arc, cx: &mut MutableAppContext) { +pub fn show_welcome_experience(app_state: &Arc, cx: &mut AppContext) { open_new(&app_state, cx, |workspace, cx| { workspace.toggle_sidebar(SidebarSide::Left, cx); let welcome_page = cx.add_view(|cx| WelcomePage::new(cx)); diff --git a/crates/workspace/src/dock.rs b/crates/workspace/src/dock.rs index 03629ffff0..773c521ddb 100644 --- a/crates/workspace/src/dock.rs +++ b/crates/workspace/src/dock.rs @@ -7,8 +7,8 @@ use gpui::{ actions, elements::{ChildView, Container, Empty, MouseEventHandler, ParentElement, Side, Stack}, geometry::vector::Vector2F, - impl_internal_actions, Border, CursorStyle, Element, ElementBox, MouseButton, - MutableAppContext, RenderContext, SizeConstraint, ViewContext, ViewHandle, + impl_internal_actions, AppContext, Border, CursorStyle, Element, ElementBox, MouseButton, + RenderContext, SizeConstraint, ViewContext, ViewHandle, }; use settings::{DockAnchor, Settings}; use theme::Theme; @@ -36,7 +36,7 @@ actions!( ); impl_internal_actions!(dock, [MoveDock, AddDefaultItemToDock]); -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(Dock::focus_dock); cx.add_action(Dock::hide_dock); cx.add_action( diff --git a/crates/workspace/src/item.rs b/crates/workspace/src/item.rs index 4a669c078b..f9b3550003 100644 --- a/crates/workspace/src/item.rs +++ b/crates/workspace/src/item.rs @@ -15,8 +15,8 @@ use std::{ use anyhow::Result; use client::{proto, Client}; use gpui::{ - AnyViewHandle, AppContext, ElementBox, ModelHandle, MutableAppContext, Task, View, ViewContext, - ViewHandle, WeakViewHandle, + AnyViewHandle, AppContext, ElementBox, ModelHandle, Task, View, ViewContext, ViewHandle, + WeakViewHandle, }; use project::{Project, ProjectEntryId, ProjectPath}; use settings::{Autosave, Settings}; @@ -159,8 +159,8 @@ pub trait Item: View { pub trait ItemHandle: 'static + fmt::Debug { fn subscribe_to_item_events( &self, - cx: &mut MutableAppContext, - handler: Box, + cx: &mut AppContext, + handler: Box, ) -> gpui::Subscription; fn tab_description<'a>(&self, detail: usize, cx: &'a AppContext) -> Option>; fn tab_content(&self, detail: Option, style: &theme::Tab, cx: &AppContext) @@ -174,7 +174,7 @@ pub trait ItemHandle: 'static + fmt::Debug { fn clone_on_split( &self, workspace_id: WorkspaceId, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>; fn added_to_pane( &self, @@ -182,35 +182,34 @@ pub trait ItemHandle: 'static + fmt::Debug { pane: ViewHandle, cx: &mut ViewContext, ); - fn deactivated(&self, cx: &mut MutableAppContext); - fn workspace_deactivated(&self, cx: &mut MutableAppContext); - fn navigate(&self, data: Box, cx: &mut MutableAppContext) -> bool; + fn deactivated(&self, cx: &mut AppContext); + fn workspace_deactivated(&self, cx: &mut AppContext); + fn navigate(&self, data: Box, cx: &mut AppContext) -> bool; fn id(&self) -> usize; fn window_id(&self) -> usize; fn as_any(&self) -> &AnyViewHandle; fn is_dirty(&self, cx: &AppContext) -> bool; fn has_conflict(&self, cx: &AppContext) -> bool; fn can_save(&self, cx: &AppContext) -> bool; - fn save(&self, project: ModelHandle, cx: &mut MutableAppContext) -> Task>; + fn save(&self, project: ModelHandle, cx: &mut AppContext) -> Task>; fn save_as( &self, project: ModelHandle, abs_path: PathBuf, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task>; - fn reload(&self, project: ModelHandle, cx: &mut MutableAppContext) - -> Task>; + fn reload(&self, project: ModelHandle, cx: &mut AppContext) -> Task>; fn git_diff_recalc( &self, project: ModelHandle, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task>; fn act_as_type<'a>(&'a self, type_id: TypeId, cx: &'a AppContext) -> Option<&'a AnyViewHandle>; fn to_followable_item_handle(&self, cx: &AppContext) -> Option>; fn on_release( &self, - cx: &mut MutableAppContext, - callback: Box, + cx: &mut AppContext, + callback: Box, ) -> gpui::Subscription; fn to_searchable_item_handle(&self, cx: &AppContext) -> Option>; fn breadcrumb_location(&self, cx: &AppContext) -> ToolbarItemLocation; @@ -239,8 +238,8 @@ impl dyn ItemHandle { impl ItemHandle for ViewHandle { fn subscribe_to_item_events( &self, - cx: &mut MutableAppContext, - handler: Box, + cx: &mut AppContext, + handler: Box, ) -> gpui::Subscription { cx.subscribe(self, move |_, event, cx| { for item_event in T::to_item_events(event) { @@ -306,7 +305,7 @@ impl ItemHandle for ViewHandle { fn clone_on_split( &self, workspace_id: WorkspaceId, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option> { self.update(cx, |item, cx| { cx.add_option_view(|cx| item.clone_on_split(workspace_id, cx)) @@ -493,15 +492,15 @@ impl ItemHandle for ViewHandle { }); } - fn deactivated(&self, cx: &mut MutableAppContext) { + fn deactivated(&self, cx: &mut AppContext) { self.update(cx, |this, cx| this.deactivated(cx)); } - fn workspace_deactivated(&self, cx: &mut MutableAppContext) { + fn workspace_deactivated(&self, cx: &mut AppContext) { self.update(cx, |this, cx| this.workspace_deactivated(cx)); } - fn navigate(&self, data: Box, cx: &mut MutableAppContext) -> bool { + fn navigate(&self, data: Box, cx: &mut AppContext) -> bool { self.update(cx, |this, cx| this.navigate(data, cx)) } @@ -529,7 +528,7 @@ impl ItemHandle for ViewHandle { self.read(cx).can_save(cx) } - fn save(&self, project: ModelHandle, cx: &mut MutableAppContext) -> Task> { + fn save(&self, project: ModelHandle, cx: &mut AppContext) -> Task> { self.update(cx, |item, cx| item.save(project, cx)) } @@ -537,23 +536,19 @@ impl ItemHandle for ViewHandle { &self, project: ModelHandle, abs_path: PathBuf, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task> { self.update(cx, |item, cx| item.save_as(project, abs_path, cx)) } - fn reload( - &self, - project: ModelHandle, - cx: &mut MutableAppContext, - ) -> Task> { + fn reload(&self, project: ModelHandle, cx: &mut AppContext) -> Task> { self.update(cx, |item, cx| item.reload(project, cx)) } fn git_diff_recalc( &self, project: ModelHandle, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task> { self.update(cx, |item, cx| item.git_diff_recalc(project, cx)) } @@ -574,8 +569,8 @@ impl ItemHandle for ViewHandle { fn on_release( &self, - cx: &mut MutableAppContext, - callback: Box, + cx: &mut AppContext, + callback: Box, ) -> gpui::Subscription { cx.observe_release(self, move |_, cx| callback(cx)) } @@ -651,7 +646,7 @@ pub trait FollowableItem: Item { project: ModelHandle, id: ViewId, state: &mut Option, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option>>>; fn add_event_to_update_proto( &self, @@ -672,7 +667,7 @@ pub trait FollowableItem: Item { pub trait FollowableItemHandle: ItemHandle { fn remote_id(&self, client: &Arc, cx: &AppContext) -> Option; - fn set_leader_replica_id(&self, leader_replica_id: Option, cx: &mut MutableAppContext); + fn set_leader_replica_id(&self, leader_replica_id: Option, cx: &mut AppContext); fn to_state_proto(&self, cx: &AppContext) -> Option; fn add_event_to_update_proto( &self, @@ -684,7 +679,7 @@ pub trait FollowableItemHandle: ItemHandle { &self, project: &ModelHandle, message: proto::update_view::Variant, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task>; fn should_unfollow_on_event(&self, event: &dyn Any, cx: &AppContext) -> bool; } @@ -699,7 +694,7 @@ impl FollowableItemHandle for ViewHandle { }) } - fn set_leader_replica_id(&self, leader_replica_id: Option, cx: &mut MutableAppContext) { + fn set_leader_replica_id(&self, leader_replica_id: Option, cx: &mut AppContext) { self.update(cx, |this, cx| { this.set_leader_replica_id(leader_replica_id, cx) }) @@ -726,7 +721,7 @@ impl FollowableItemHandle for ViewHandle { &self, project: &ModelHandle, message: proto::update_view::Variant, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task> { self.update(cx, |this, cx| this.apply_update_proto(project, message, cx)) } @@ -745,8 +740,8 @@ pub(crate) mod test { use super::{Item, ItemEvent}; use crate::{sidebar::SidebarItem, ItemId, ItemNavHistory, Pane, Workspace, WorkspaceId}; use gpui::{ - elements::Empty, AppContext, Element, ElementBox, Entity, ModelHandle, MutableAppContext, - RenderContext, Task, View, ViewContext, ViewHandle, WeakViewHandle, + elements::Empty, AppContext, Element, ElementBox, Entity, ModelHandle, RenderContext, Task, + View, ViewContext, ViewHandle, WeakViewHandle, }; use project::{Project, ProjectEntryId, ProjectPath, WorktreeId}; use smallvec::SmallVec; @@ -812,7 +807,7 @@ pub(crate) mod test { } impl TestProjectItem { - pub fn new(id: u64, path: &str, cx: &mut MutableAppContext) -> ModelHandle { + pub fn new(id: u64, path: &str, cx: &mut AppContext) -> ModelHandle { let entry_id = Some(ProjectEntryId::from_proto(id)); let project_path = Some(ProjectPath { worktree_id: WorktreeId::from_usize(0), @@ -824,7 +819,7 @@ pub(crate) mod test { }) } - pub fn new_untitled(cx: &mut MutableAppContext) -> ModelHandle { + pub fn new_untitled(cx: &mut AppContext) -> ModelHandle { cx.add_model(|_| Self { project_path: None, entry_id: None, diff --git a/crates/workspace/src/notifications.rs b/crates/workspace/src/notifications.rs index 16faab87e8..337d1145c0 100644 --- a/crates/workspace/src/notifications.rs +++ b/crates/workspace/src/notifications.rs @@ -1,11 +1,11 @@ use std::{any::TypeId, ops::DerefMut}; use collections::HashSet; -use gpui::{AnyViewHandle, Entity, MutableAppContext, View, ViewContext, ViewHandle}; +use gpui::{AnyViewHandle, Entity, AppContext, View, ViewContext, ViewHandle}; use crate::Workspace; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.set_global(NotificationTracker::new()); simple_message_notification::init(cx); } @@ -138,7 +138,7 @@ pub mod simple_message_notification { use gpui::{ actions, elements::{Flex, MouseEventHandler, Padding, ParentElement, Svg, Text}, - impl_actions, Action, CursorStyle, Element, Entity, MouseButton, MutableAppContext, View, + impl_actions, Action, CursorStyle, Element, Entity, MouseButton, AppContext, View, ViewContext, }; use menu::Cancel; @@ -162,7 +162,7 @@ pub mod simple_message_notification { impl_actions!(message_notifications, [OsOpen]); - pub fn init(cx: &mut MutableAppContext) { + pub fn init(cx: &mut AppContext) { cx.add_action(MessageNotification::dismiss); cx.add_action( |_workspace: &mut Workspace, open_action: &OsOpen, cx: &mut ViewContext| { diff --git a/crates/workspace/src/pane.rs b/crates/workspace/src/pane.rs index 7d4b4673ed..edd48268f9 100644 --- a/crates/workspace/src/pane.rs +++ b/crates/workspace/src/pane.rs @@ -24,8 +24,8 @@ use gpui::{ keymap_matcher::KeymapContext, platform::{CursorStyle, NavigationDirection}, Action, AnyViewHandle, AnyWeakViewHandle, AppContext, AsyncAppContext, Entity, EventContext, - ModelHandle, MouseButton, MouseRegion, MutableAppContext, PromptLevel, Quad, RenderContext, - Task, View, ViewContext, ViewHandle, WeakViewHandle, + ModelHandle, MouseButton, MouseRegion, PromptLevel, Quad, RenderContext, Task, View, + ViewContext, ViewHandle, WeakViewHandle, }; use project::{Project, ProjectEntryId, ProjectPath}; use serde::Deserialize; @@ -108,7 +108,7 @@ const MAX_NAVIGATION_HISTORY_LEN: usize = 1024; pub type BackgroundActions = fn() -> &'static [(&'static str, &'static dyn Action)]; -pub fn init(cx: &mut MutableAppContext) { +pub fn init(cx: &mut AppContext) { cx.add_action(|pane: &mut Pane, action: &ActivateItem, cx| { pane.activate_item(action.0, true, true, cx); }); @@ -1094,7 +1094,7 @@ impl Pane { pub fn autosave_item( item: &dyn ItemHandle, project: ModelHandle, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task> { if Self::can_autosave_item(item, cx) { item.save(project, cx) @@ -1740,15 +1740,15 @@ fn render_tab_bar_button( } impl ItemNavHistory { - pub fn push(&self, data: Option, cx: &mut MutableAppContext) { + pub fn push(&self, data: Option, cx: &mut AppContext) { self.history.borrow_mut().push(data, self.item.clone(), cx); } - pub fn pop_backward(&self, cx: &mut MutableAppContext) -> Option { + pub fn pop_backward(&self, cx: &mut AppContext) -> Option { self.history.borrow_mut().pop(NavigationMode::GoingBack, cx) } - pub fn pop_forward(&self, cx: &mut MutableAppContext) -> Option { + pub fn pop_forward(&self, cx: &mut AppContext) -> Option { self.history .borrow_mut() .pop(NavigationMode::GoingForward, cx) @@ -1768,7 +1768,7 @@ impl NavHistory { self.mode = NavigationMode::Normal; } - fn pop(&mut self, mode: NavigationMode, cx: &mut MutableAppContext) -> Option { + fn pop(&mut self, mode: NavigationMode, cx: &mut AppContext) -> Option { let entry = match mode { NavigationMode::Normal | NavigationMode::Disabled | NavigationMode::ClosingItem => { return None @@ -1788,7 +1788,7 @@ impl NavHistory { &mut self, data: Option, item: Rc, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { match self.mode { NavigationMode::Disabled => {} @@ -1833,7 +1833,7 @@ impl NavHistory { self.did_update(cx); } - fn did_update(&self, cx: &mut MutableAppContext) { + fn did_update(&self, cx: &mut AppContext) { if let Some(pane) = self.pane.upgrade(cx) { cx.defer(move |cx| pane.update(cx, |pane, cx| pane.history_updated(cx))); } diff --git a/crates/workspace/src/searchable.rs b/crates/workspace/src/searchable.rs index ae78ca0eb8..c72023b4d4 100644 --- a/crates/workspace/src/searchable.rs +++ b/crates/workspace/src/searchable.rs @@ -1,8 +1,8 @@ use std::any::Any; use gpui::{ - AnyViewHandle, AnyWeakViewHandle, AppContext, MutableAppContext, Subscription, Task, - ViewContext, ViewHandle, WeakViewHandle, + AnyViewHandle, AnyWeakViewHandle, AppContext, Subscription, Task, ViewContext, ViewHandle, + WeakViewHandle, }; use project::search::SearchQuery; @@ -90,34 +90,29 @@ pub trait SearchableItemHandle: ItemHandle { fn supported_options(&self) -> SearchOptions; fn subscribe_to_search_events( &self, - cx: &mut MutableAppContext, - handler: Box, + cx: &mut AppContext, + handler: Box, ) -> Subscription; - fn clear_matches(&self, cx: &mut MutableAppContext); - fn update_matches(&self, matches: &Vec>, cx: &mut MutableAppContext); - fn query_suggestion(&self, cx: &mut MutableAppContext) -> String; - fn activate_match( - &self, - index: usize, - matches: &Vec>, - cx: &mut MutableAppContext, - ); + fn clear_matches(&self, cx: &mut AppContext); + fn update_matches(&self, matches: &Vec>, cx: &mut AppContext); + fn query_suggestion(&self, cx: &mut AppContext) -> String; + fn activate_match(&self, index: usize, matches: &Vec>, cx: &mut AppContext); fn match_index_for_direction( &self, matches: &Vec>, current_index: usize, direction: Direction, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> usize; fn find_matches( &self, query: SearchQuery, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task>>; fn active_match_index( &self, matches: &Vec>, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option; } @@ -136,8 +131,8 @@ impl SearchableItemHandle for ViewHandle { fn subscribe_to_search_events( &self, - cx: &mut MutableAppContext, - handler: Box, + cx: &mut AppContext, + handler: Box, ) -> Subscription { cx.subscribe(self, move |_, event, cx| { if let Some(search_event) = T::to_search_event(event) { @@ -146,21 +141,21 @@ impl SearchableItemHandle for ViewHandle { }) } - fn clear_matches(&self, cx: &mut MutableAppContext) { + fn clear_matches(&self, cx: &mut AppContext) { self.update(cx, |this, cx| this.clear_matches(cx)); } - fn update_matches(&self, matches: &Vec>, cx: &mut MutableAppContext) { + fn update_matches(&self, matches: &Vec>, cx: &mut AppContext) { let matches = downcast_matches(matches); self.update(cx, |this, cx| this.update_matches(matches, cx)); } - fn query_suggestion(&self, cx: &mut MutableAppContext) -> String { + fn query_suggestion(&self, cx: &mut AppContext) -> String { self.update(cx, |this, cx| this.query_suggestion(cx)) } fn activate_match( &self, index: usize, matches: &Vec>, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { let matches = downcast_matches(matches); self.update(cx, |this, cx| this.activate_match(index, matches, cx)); @@ -170,7 +165,7 @@ impl SearchableItemHandle for ViewHandle { matches: &Vec>, current_index: usize, direction: Direction, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> usize { let matches = downcast_matches(matches); self.update(cx, |this, cx| { @@ -180,7 +175,7 @@ impl SearchableItemHandle for ViewHandle { fn find_matches( &self, query: SearchQuery, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task>> { let matches = self.update(cx, |this, cx| this.find_matches(query, cx)); cx.foreground().spawn(async { @@ -194,7 +189,7 @@ impl SearchableItemHandle for ViewHandle { fn active_match_index( &self, matches: &Vec>, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option { let matches = downcast_matches(matches); self.update(cx, |this, cx| this.active_match_index(matches, cx)) diff --git a/crates/workspace/src/status_bar.rs b/crates/workspace/src/status_bar.rs index a9c8547fd5..355c327dfd 100644 --- a/crates/workspace/src/status_bar.rs +++ b/crates/workspace/src/status_bar.rs @@ -8,9 +8,8 @@ use gpui::{ vector::{vec2f, Vector2F}, }, json::{json, ToJson}, - AnyViewHandle, DebugContext, ElementBox, Entity, LayoutContext, MeasurementContext, - MutableAppContext, PaintContext, RenderContext, SizeConstraint, Subscription, View, - ViewContext, ViewHandle, + AnyViewHandle, AppContext, DebugContext, ElementBox, Entity, LayoutContext, MeasurementContext, + PaintContext, RenderContext, SizeConstraint, Subscription, View, ViewContext, ViewHandle, }; use settings::Settings; @@ -24,11 +23,7 @@ pub trait StatusItemView: View { trait StatusItemViewHandle { fn as_any(&self) -> &AnyViewHandle; - fn set_active_pane_item( - &self, - active_pane_item: Option<&dyn ItemHandle>, - cx: &mut MutableAppContext, - ); + fn set_active_pane_item(&self, active_pane_item: Option<&dyn ItemHandle>, cx: &mut AppContext); } pub struct StatusBar { @@ -130,11 +125,7 @@ impl StatusItemViewHandle for ViewHandle { self } - fn set_active_pane_item( - &self, - active_pane_item: Option<&dyn ItemHandle>, - cx: &mut MutableAppContext, - ) { + fn set_active_pane_item(&self, active_pane_item: Option<&dyn ItemHandle>, cx: &mut AppContext) { self.update(cx, |this, cx| { this.set_active_pane_item(active_pane_item, cx) }); diff --git a/crates/workspace/src/toolbar.rs b/crates/workspace/src/toolbar.rs index ac431c5590..d76135ad15 100644 --- a/crates/workspace/src/toolbar.rs +++ b/crates/workspace/src/toolbar.rs @@ -1,7 +1,7 @@ use crate::{ItemHandle, Pane}; use gpui::{ elements::*, platform::CursorStyle, Action, AnyViewHandle, AppContext, ElementBox, Entity, - MouseButton, MutableAppContext, RenderContext, View, ViewContext, ViewHandle, WeakViewHandle, + MouseButton, RenderContext, View, ViewContext, ViewHandle, WeakViewHandle, }; use settings::Settings; @@ -21,7 +21,7 @@ pub trait ToolbarItemView: View { current_location } - fn pane_focus_update(&mut self, _pane_focused: bool, _cx: &mut MutableAppContext) {} + fn pane_focus_update(&mut self, _pane_focused: bool, _cx: &mut AppContext) {} } trait ToolbarItemViewHandle { @@ -30,9 +30,9 @@ trait ToolbarItemViewHandle { fn set_active_pane_item( &self, active_pane_item: Option<&dyn ItemHandle>, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> ToolbarItemLocation; - fn pane_focus_update(&mut self, pane_focused: bool, cx: &mut MutableAppContext); + fn pane_focus_update(&mut self, pane_focused: bool, cx: &mut AppContext); } #[derive(Copy, Clone, Debug, PartialEq)] @@ -263,7 +263,7 @@ impl Toolbar { } } - pub fn pane_focus_update(&mut self, pane_focused: bool, cx: &mut MutableAppContext) { + pub fn pane_focus_update(&mut self, pane_focused: bool, cx: &mut AppContext) { for (toolbar_item, _) in self.items.iter_mut() { toolbar_item.pane_focus_update(pane_focused, cx); } @@ -292,14 +292,14 @@ impl ToolbarItemViewHandle for ViewHandle { fn set_active_pane_item( &self, active_pane_item: Option<&dyn ItemHandle>, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> ToolbarItemLocation { self.update(cx, |this, cx| { this.set_active_pane_item(active_pane_item, cx) }) } - fn pane_focus_update(&mut self, pane_focused: bool, cx: &mut MutableAppContext) { + fn pane_focus_update(&mut self, pane_focused: bool, cx: &mut AppContext) { self.update(cx, |this, cx| this.pane_focus_update(pane_focused, cx)); } } diff --git a/crates/workspace/src/workspace.rs b/crates/workspace/src/workspace.rs index 1e11578ac1..de010e518c 100644 --- a/crates/workspace/src/workspace.rs +++ b/crates/workspace/src/workspace.rs @@ -42,9 +42,9 @@ use gpui::{ keymap_matcher::KeymapContext, platform::{CursorStyle, WindowOptions}, Action, AnyModelHandle, AnyViewHandle, AppContext, AsyncAppContext, Entity, ModelContext, - ModelHandle, MouseButton, MutableAppContext, PathPromptOptions, Platform, PromptLevel, - RenderContext, SizeConstraint, Subscription, Task, View, ViewContext, ViewHandle, - WeakViewHandle, WindowBounds, + ModelHandle, MouseButton, PathPromptOptions, Platform, PromptLevel, RenderContext, + SizeConstraint, Subscription, Task, View, ViewContext, ViewHandle, WeakViewHandle, + WindowBounds, }; use item::{FollowableItem, FollowableItemHandle, Item, ItemHandle, ProjectItem}; use language::LanguageRegistry; @@ -245,12 +245,12 @@ impl_internal_actions!( ); impl_actions!(workspace, [ActivatePane]); -pub fn init(app_state: Arc, cx: &mut MutableAppContext) { +pub fn init(app_state: Arc, cx: &mut AppContext) { pane::init(cx); dock::init(cx); notifications::init(cx); - cx.add_global_action(|_: &Open, cx: &mut MutableAppContext| { + cx.add_global_action(|_: &Open, cx: &mut AppContext| { let mut paths = cx.prompt_for_paths(PathPromptOptions { files: true, directories: true, @@ -283,7 +283,7 @@ pub fn init(app_state: Arc, cx: &mut MutableAppContext) { }); cx.add_global_action({ let app_state = Arc::downgrade(&app_state); - move |action: &OpenPaths, cx: &mut MutableAppContext| { + move |action: &OpenPaths, cx: &mut AppContext| { if let Some(app_state) = app_state.upgrade() { open_paths(&action.paths, &app_state, None, cx).detach(); } @@ -315,7 +315,7 @@ pub fn init(app_state: Arc, cx: &mut MutableAppContext) { cx.add_global_action({ let app_state = Arc::downgrade(&app_state); - move |_: &NewWindow, cx: &mut MutableAppContext| { + move |_: &NewWindow, cx: &mut AppContext| { if let Some(app_state) = app_state.upgrade() { open_new(&app_state, cx, |_, cx| cx.dispatch_action(NewFile)).detach(); } @@ -323,7 +323,7 @@ pub fn init(app_state: Arc, cx: &mut MutableAppContext) { }); cx.add_global_action({ let app_state = Arc::downgrade(&app_state); - move |_: &NewFile, cx: &mut MutableAppContext| { + move |_: &NewFile, cx: &mut AppContext| { if let Some(app_state) = app_state.upgrade() { open_new(&app_state, cx, |_, cx| cx.dispatch_action(NewFile)).detach(); } @@ -444,7 +444,7 @@ type ProjectItemBuilders = HashMap< TypeId, fn(ModelHandle, AnyModelHandle, &mut ViewContext) -> Box, >; -pub fn register_project_item(cx: &mut MutableAppContext) { +pub fn register_project_item(cx: &mut AppContext) { cx.update_default_global(|builders: &mut ProjectItemBuilders, _| { builders.insert(TypeId::of::(), |project, model, cx| { let item = model.downcast::().unwrap(); @@ -458,7 +458,7 @@ type FollowableItemBuilder = fn( ModelHandle, ViewId, &mut Option, - &mut MutableAppContext, + &mut AppContext, ) -> Option>>>; type FollowableItemBuilders = HashMap< TypeId, @@ -467,7 +467,7 @@ type FollowableItemBuilders = HashMap< fn(&AnyViewHandle) -> Box, ), >; -pub fn register_followable_item(cx: &mut MutableAppContext) { +pub fn register_followable_item(cx: &mut AppContext) { cx.update_default_global(|builders: &mut FollowableItemBuilders, _| { builders.insert( TypeId::of::(), @@ -494,7 +494,7 @@ type ItemDeserializers = HashMap< &mut ViewContext, ) -> Task>>, >; -pub fn register_deserializable_item(cx: &mut MutableAppContext) { +pub fn register_deserializable_item(cx: &mut AppContext) { cx.update_default_global(|deserializers: &mut ItemDeserializers, _cx| { if let Some(serialized_item_kind) = I::serialized_item_kind() { deserializers.insert( @@ -524,7 +524,7 @@ pub struct AppState { impl AppState { #[cfg(any(test, feature = "test-support"))] - pub fn test(cx: &mut MutableAppContext) -> Arc { + pub fn test(cx: &mut AppContext) -> Arc { let settings = Settings::test(cx); cx.set_global(settings); @@ -857,7 +857,7 @@ impl Workspace { abs_paths: Vec, app_state: Arc, requesting_window_id: Option, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task<( ViewHandle, Vec, anyhow::Error>>>, @@ -1097,7 +1097,7 @@ impl Workspace { } } - pub fn close_global(_: &CloseWindow, cx: &mut MutableAppContext) { + pub fn close_global(_: &CloseWindow, cx: &mut AppContext) { let id = cx.window_ids().find(|&id| cx.window_is_active(id)); if let Some(id) = id { //This can only get called when the window's project connection has been lost @@ -1132,7 +1132,7 @@ impl Workspace { let window_id = cx.window_id(); let workspace_count = cx .window_ids() - .flat_map(|window_id| cx.root_view::(window_id)) + .filter_map(|window_id| cx.root_view(window_id)?.clone().downcast::()) .count(); cx.spawn(|this, mut cx| async move { @@ -1313,7 +1313,7 @@ impl Workspace { project: ModelHandle, abs_path: &Path, visible: bool, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task, ProjectPath)>> { let entry = project.update(cx, |project, cx| { project.find_or_create_local_worktree(abs_path, visible, cx) @@ -1405,7 +1405,7 @@ impl Workspace { let project = self.project.clone(); if let Some(item) = self.active_item(cx) { if !force_name_change && item.can_save(cx) { - if item.has_conflict(cx.as_ref()) { + if item.has_conflict(cx) { const CONFLICT_MESSAGE: &str = "This file has changed on disk since you started editing it. Do you want to overwrite it?"; let mut answer = cx.prompt( @@ -2669,7 +2669,7 @@ impl Workspace { fn load_from_serialized_workspace( workspace: WeakViewHandle, serialized_workspace: SerializedWorkspace, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) { cx.spawn(|mut cx| async move { if let Some(workspace) = workspace.upgrade(&cx) { @@ -2990,15 +2990,15 @@ impl std::fmt::Debug for OpenPaths { pub struct WorkspaceCreated(WeakViewHandle); pub fn activate_workspace_for_project( - cx: &mut MutableAppContext, + cx: &mut AppContext, predicate: impl Fn(&mut Project, &mut ModelContext) -> bool, ) -> Option> { for window_id in cx.window_ids().collect::>() { - if let Some(workspace_handle) = cx.root_view::(window_id) { + if let Some(workspace_handle) = cx.root_view(window_id)?.downcast_ref::() { let project = workspace_handle.read(cx).project.clone(); if project.update(cx, &predicate) { cx.activate_window(window_id); - return Some(workspace_handle); + return Some(workspace_handle.clone()); } } } @@ -3014,7 +3014,7 @@ pub fn open_paths( abs_paths: &[PathBuf], app_state: &Arc, requesting_window_id: Option, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Task<( ViewHandle, Vec, anyhow::Error>>>, @@ -3066,7 +3066,7 @@ pub fn open_paths( pub fn open_new( app_state: &Arc, - cx: &mut MutableAppContext, + cx: &mut AppContext, init: impl FnOnce(&mut Workspace, &mut ViewContext) + 'static, ) -> Task<()> { let task = Workspace::new_local(Vec::new(), app_state.clone(), None, cx); diff --git a/crates/zed/src/languages/json.rs b/crates/zed/src/languages/json.rs index 566b042392..5c3edfba25 100644 --- a/crates/zed/src/languages/json.rs +++ b/crates/zed/src/languages/json.rs @@ -2,7 +2,7 @@ use anyhow::{anyhow, Result}; use async_trait::async_trait; use collections::HashMap; use futures::{future::BoxFuture, FutureExt, StreamExt}; -use gpui::MutableAppContext; +use gpui::AppContext; use language::{LanguageRegistry, LanguageServerBinary, LanguageServerName, LspAdapter}; use node_runtime::NodeRuntime; use serde_json::json; @@ -125,7 +125,7 @@ impl LspAdapter for JsonLspAdapter { fn workspace_configuration( &self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option> { let action_names = cx.all_action_names().collect::>(); let theme_names = self diff --git a/crates/zed/src/languages/yaml.rs b/crates/zed/src/languages/yaml.rs index a9a87fd585..337c2c0653 100644 --- a/crates/zed/src/languages/yaml.rs +++ b/crates/zed/src/languages/yaml.rs @@ -1,7 +1,7 @@ use anyhow::{anyhow, Result}; use async_trait::async_trait; use futures::{future::BoxFuture, FutureExt, StreamExt}; -use gpui::MutableAppContext; +use gpui::AppContext; use language::{LanguageServerBinary, LanguageServerName, LspAdapter}; use node_runtime::NodeRuntime; use serde_json::Value; @@ -101,7 +101,7 @@ impl LspAdapter for YamlLspAdapter { fn workspace_configuration( &self, - cx: &mut MutableAppContext, + cx: &mut AppContext, ) -> Option> { let settings = cx.global::(); Some( diff --git a/crates/zed/src/main.rs b/crates/zed/src/main.rs index fd7e0f22c7..7b4b415620 100644 --- a/crates/zed/src/main.rs +++ b/crates/zed/src/main.rs @@ -14,7 +14,7 @@ use futures::{ channel::{mpsc, oneshot}, FutureExt, SinkExt, StreamExt, }; -use gpui::{Action, App, AssetSource, AsyncAppContext, MutableAppContext, Task, ViewContext}; +use gpui::{Action, App, AppContext, AssetSource, AsyncAppContext, Task, ViewContext}; use isahc::{config::Configurable, Request}; use language::LanguageRegistry; use log::LevelFilter; @@ -370,7 +370,7 @@ fn init_panic_hook(app_version: String) { })); } -fn upload_previous_panics(http: Arc, cx: &mut MutableAppContext) { +fn upload_previous_panics(http: Arc, cx: &mut AppContext) { let diagnostics_telemetry = cx.global::().telemetry_diagnostics(); cx.background() diff --git a/crates/zed/src/zed.rs b/crates/zed/src/zed.rs index 11546565a4..6e7cf0dd10 100644 --- a/crates/zed/src/zed.rs +++ b/crates/zed/src/zed.rs @@ -72,16 +72,16 @@ actions!( const MIN_FONT_SIZE: f32 = 6.0; -pub fn init(app_state: &Arc, cx: &mut gpui::MutableAppContext) { +pub fn init(app_state: &Arc, cx: &mut gpui::AppContext) { terminal_button::init(cx); cx.add_action(about); - cx.add_global_action(|_: &Hide, cx: &mut gpui::MutableAppContext| { + cx.add_global_action(|_: &Hide, cx: &mut gpui::AppContext| { cx.platform().hide(); }); - cx.add_global_action(|_: &HideOthers, cx: &mut gpui::MutableAppContext| { + cx.add_global_action(|_: &HideOthers, cx: &mut gpui::AppContext| { cx.platform().hide_other_apps(); }); - cx.add_global_action(|_: &ShowAll, cx: &mut gpui::MutableAppContext| { + cx.add_global_action(|_: &ShowAll, cx: &mut gpui::AppContext| { cx.platform().unhide_other_apps(); }); cx.add_action( @@ -367,10 +367,10 @@ pub fn build_window_options( } } -fn restart(_: &Restart, cx: &mut gpui::MutableAppContext) { +fn restart(_: &Restart, cx: &mut gpui::AppContext) { let mut workspaces = cx .window_ids() - .filter_map(|window_id| cx.root_view::(window_id)) + .filter_map(|window_id| cx.root_view(window_id)?.clone().downcast::()) .collect::>(); // If multiple windows have unsaved changes, and need a save prompt, @@ -411,10 +411,10 @@ fn restart(_: &Restart, cx: &mut gpui::MutableAppContext) { .detach_and_log_err(cx); } -fn quit(_: &Quit, cx: &mut gpui::MutableAppContext) { +fn quit(_: &Quit, cx: &mut gpui::AppContext) { let mut workspaces = cx .window_ids() - .filter_map(|window_id| cx.root_view::(window_id)) + .filter_map(|window_id| cx.root_view(window_id)?.clone().downcast::()) .collect::>(); // If multiple windows have unsaved changes, and need a save prompt, @@ -656,9 +656,7 @@ mod tests { use super::*; use assets::Assets; use editor::{scroll::autoscroll::Autoscroll, DisplayPoint, Editor}; - use gpui::{ - executor::Deterministic, AssetSource, MutableAppContext, TestAppContext, ViewHandle, - }; + use gpui::{executor::Deterministic, AppContext, AssetSource, TestAppContext, ViewHandle}; use language::LanguageRegistry; use node_runtime::NodeRuntime; use project::{Project, ProjectPath}; @@ -717,7 +715,11 @@ mod tests { cx.update(|cx| open_paths(&[PathBuf::from("/root/a")], &app_state, None, cx)) .await; assert_eq!(cx.window_ids().len(), 1); - let workspace_1 = cx.root_view::(cx.window_ids()[0]).unwrap(); + let workspace_1 = cx + .root_view(cx.window_ids()[0]) + .unwrap() + .downcast::() + .unwrap(); workspace_1.update(cx, |workspace, cx| { assert_eq!(workspace.worktrees(cx).count(), 2); assert!(workspace.left_sidebar().read(cx).is_open()); @@ -747,7 +749,12 @@ mod tests { }) .await; assert_eq!(cx.window_ids().len(), 2); - let workspace_1 = cx.root_view::(window_id).unwrap(); + let workspace_1 = cx + .root_view(window_id) + .unwrap() + .clone() + .downcast::() + .unwrap(); workspace_1.read_with(cx, |workspace, cx| { assert_eq!( workspace @@ -775,7 +782,12 @@ mod tests { assert_eq!(cx.window_ids().len(), 1); // When opening the workspace, the window is not in a edited state. - let workspace = cx.root_view::(cx.window_ids()[0]).unwrap(); + let workspace = cx + .root_view(cx.window_ids()[0]) + .unwrap() + .clone() + .downcast::() + .unwrap(); let editor = workspace.read_with(cx, |workspace, cx| { workspace .active_item(cx) @@ -842,7 +854,12 @@ mod tests { .await; let window_id = *cx.window_ids().first().unwrap(); - let workspace = cx.root_view::(window_id).unwrap(); + let workspace = cx + .root_view(window_id) + .unwrap() + .clone() + .downcast::() + .unwrap(); let editor = workspace.update(cx, |workspace, cx| { workspace .active_item(cx) @@ -950,7 +967,7 @@ mod tests { .read(cx) .active_item() .unwrap() - .project_path(cx.as_ref()), + .project_path(cx), Some(file2.clone()) ); }); @@ -1251,7 +1268,7 @@ mod tests { // Edit the file and save it again. This time, there is no filename prompt. editor.update(cx, |editor, cx| { editor.handle_input(" there", cx); - assert!(editor.is_dirty(cx.as_ref())); + assert!(editor.is_dirty(cx)); }); let save_task = workspace.update(cx, |workspace, cx| workspace.save_active_item(false, cx)); save_task.await.unwrap(); @@ -1311,7 +1328,7 @@ mod tests { &languages::PLAIN_TEXT )); editor.handle_input("hi", cx); - assert!(editor.is_dirty(cx.as_ref())); + assert!(editor.is_dirty(cx)); }); // Save the buffer. This prompts for a filename. @@ -1374,7 +1391,7 @@ mod tests { assert_ne!(pane_1, pane_2); let pane2_item = pane_2.read(cx).active_item().unwrap(); - assert_eq!(pane2_item.project_path(cx.as_ref()), Some(file1.clone())); + assert_eq!(pane2_item.project_path(cx), Some(file1.clone())); pane2_item.downcast::().unwrap().downgrade() }); @@ -1819,7 +1836,7 @@ mod tests { } #[gpui::test] - fn test_bundled_settings_and_themes(cx: &mut MutableAppContext) { + fn test_bundled_settings_and_themes(cx: &mut AppContext) { cx.platform() .fonts() .add_fonts(&[ @@ -1850,7 +1867,7 @@ mod tests { } #[gpui::test] - fn test_bundled_languages(cx: &mut MutableAppContext) { + fn test_bundled_languages(cx: &mut AppContext) { let mut languages = LanguageRegistry::test(); languages.set_executor(cx.background().clone()); let languages = Arc::new(languages);