Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unneeded 'static lifetimes on &strs in constants #8698

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion crates/ai/src/providers/open_ai.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,4 @@ pub use completion::*;
pub use embedding::*;
pub use model::OpenAiLanguageModel;

pub const OPEN_AI_API_URL: &'static str = "https://api.openai.com/v1";
pub const OPEN_AI_API_URL: &str = "https://api.openai.com/v1";
2 changes: 1 addition & 1 deletion crates/collab/src/db.rs
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,7 @@ impl Database {
}

fn is_serialization_error(error: &Error) -> bool {
const SERIALIZATION_FAILURE_CODE: &'static str = "40001";
const SERIALIZATION_FAILURE_CODE: &str = "40001";
match error {
Error::Database(
DbErr::Exec(sea_orm::RuntimeErr::SqlxError(error))
Expand Down
4 changes: 2 additions & 2 deletions crates/collab/src/db/tests/feature_flag_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,8 @@ async fn test_get_user_flags(db: &Arc<Database>) {
.unwrap()
.user_id;

const CHANNELS_ALPHA: &'static str = "channels-alpha";
const NEW_SEARCH: &'static str = "new-search";
const CHANNELS_ALPHA: &str = "channels-alpha";
const NEW_SEARCH: &str = "new-search";

let channels_flag = db.create_user_flag(CHANNELS_ALPHA).await.unwrap();
let search_flag = db.create_user_flag(NEW_SEARCH).await.unwrap();
Expand Down
2 changes: 1 addition & 1 deletion crates/collab/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ use tracing_log::LogTracer;
use tracing_subscriber::{filter::EnvFilter, fmt::format::JsonFields, Layer};
use util::ResultExt;

const VERSION: &'static str = env!("CARGO_PKG_VERSION");
const VERSION: &str = env!("CARGO_PKG_VERSION");
const REVISION: Option<&'static str> = option_env!("GITHUB_SHA");

#[tokio::main]
Expand Down
4 changes: 2 additions & 2 deletions crates/collab/src/tests/integration_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2821,8 +2821,8 @@ async fn test_git_status_sync(
)
.await;

const A_TXT: &'static str = "a.txt";
const B_TXT: &'static str = "b.txt";
const A_TXT: &str = "a.txt";
const B_TXT: &str = "b.txt";

client_a.fs().set_status_for_repo_via_git_operation(
Path::new("/dir/.git"),
Expand Down
2 changes: 1 addition & 1 deletion crates/collab_ui/src/chat_panel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ use workspace::{
mod message_editor;

const MESSAGE_LOADING_THRESHOLD: usize = 50;
const CHAT_PANEL_KEY: &'static str = "ChatPanel";
const CHAT_PANEL_KEY: &str = "ChatPanel";

pub fn init(cx: &mut AppContext) {
cx.observe_new_views(|workspace: &mut Workspace, _| {
Expand Down
2 changes: 1 addition & 1 deletion crates/collab_ui/src/collab_panel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ struct ChannelMoveClipboard {
channel_id: ChannelId,
}

const COLLABORATION_PANEL_KEY: &'static str = "CollaborationPanel";
const COLLABORATION_PANEL_KEY: &str = "CollaborationPanel";

pub fn init(cx: &mut AppContext) {
cx.observe_new_views(|workspace: &mut Workspace, _| {
Expand Down
2 changes: 1 addition & 1 deletion crates/collab_ui/src/notification_panel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ use workspace::{
const LOADING_THRESHOLD: usize = 30;
const MARK_AS_READ_DELAY: Duration = Duration::from_secs(1);
const TOAST_DURATION: Duration = Duration::from_secs(5);
const NOTIFICATION_PANEL_KEY: &'static str = "NotificationPanel";
const NOTIFICATION_PANEL_KEY: &str = "NotificationPanel";

pub struct NotificationPanel {
client: Arc<Client>,
Expand Down
2 changes: 1 addition & 1 deletion crates/copilot/src/copilot.rs
Original file line number Diff line number Diff line change
Expand Up @@ -971,7 +971,7 @@ async fn clear_copilot_dir() {
}

async fn get_copilot_lsp(http: Arc<dyn HttpClient>) -> anyhow::Result<PathBuf> {
const SERVER_PATH: &'static str = "dist/agent.js";
const SERVER_PATH: &str = "dist/agent.js";

///Check for the latest copilot language server and download it if we haven't already
async fn fetch_latest(http: Arc<dyn HttpClient>) -> anyhow::Result<PathBuf> {
Expand Down
2 changes: 1 addition & 1 deletion crates/copilot_ui/src/sign_in.rs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ use gpui::{
use ui::{prelude::*, Button, IconName, Label};
use workspace::ModalView;

const COPILOT_SIGN_UP_URL: &'static str = "https://github.com/features/copilot";
const COPILOT_SIGN_UP_URL: &str = "https://github.com/features/copilot";

pub struct CopilotCodeVerification {
status: Status,
Expand Down
8 changes: 4 additions & 4 deletions crates/db/src/db.rs
Original file line number Diff line number Diff line change
Expand Up @@ -22,20 +22,20 @@ use std::path::{Path, PathBuf};
use std::sync::atomic::{AtomicBool, Ordering};
use util::{async_maybe, ResultExt};

const CONNECTION_INITIALIZE_QUERY: &'static str = sql!(
const CONNECTION_INITIALIZE_QUERY: &str = sql!(
PRAGMA foreign_keys=TRUE;
);

const DB_INITIALIZE_QUERY: &'static str = sql!(
const DB_INITIALIZE_QUERY: &str = sql!(
PRAGMA journal_mode=WAL;
PRAGMA busy_timeout=1;
PRAGMA case_sensitive_like=TRUE;
PRAGMA synchronous=NORMAL;
);

const FALLBACK_DB_NAME: &'static str = "FALLBACK_MEMORY_DB";
const FALLBACK_DB_NAME: &str = "FALLBACK_MEMORY_DB";

const DB_FILE_NAME: &'static str = "db.sqlite";
const DB_FILE_NAME: &str = "db.sqlite";

lazy_static::lazy_static! {
pub static ref ZED_STATELESS: bool = std::env::var("ZED_STATELESS").map_or(false, |v| !v.is_empty());
Expand Down
2 changes: 1 addition & 1 deletion crates/editor/src/editor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -8644,7 +8644,7 @@ impl Editor {
let mut cwd = worktree.read(cx).abs_path().to_path_buf();
cwd.push(".git");

const REMOTE_NAME: &'static str = "origin";
const REMOTE_NAME: &str = "origin";
let repo = project
.fs()
.open_repo(&cwd)
Expand Down
3 changes: 1 addition & 2 deletions crates/gpui/src/platform/mac/metal_renderer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,7 @@ pub(crate) type PointF = crate::Point<f32>;
#[cfg(not(feature = "runtime_shaders"))]
const SHADERS_METALLIB: &[u8] = include_bytes!(concat!(env!("OUT_DIR"), "/shaders.metallib"));
#[cfg(feature = "runtime_shaders")]
const SHADERS_SOURCE_FILE: &'static str =
include_str!(concat!(env!("OUT_DIR"), "/stitched_shaders.metal"));
const SHADERS_SOURCE_FILE: &str = include_str!(concat!(env!("OUT_DIR"), "/stitched_shaders.metal"));
const INSTANCE_BUFFER_SIZE: usize = 2 * 1024 * 1024; // This is an arbitrary decision. There's probably a more optimal value (maybe even we could adjust dynamically...)

pub type Context = Arc<Mutex<Vec<metal::Buffer>>>;
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/astro.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ use std::{
};
use util::ResultExt;

const SERVER_PATH: &'static str = "node_modules/@astrojs/language-server/bin/nodeServer.js";
const SERVER_PATH: &str = "node_modules/@astrojs/language-server/bin/nodeServer.js";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/css.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ use std::{
};
use util::{async_maybe, ResultExt};

const SERVER_PATH: &'static str =
const SERVER_PATH: &str =
"node_modules/vscode-langservers-extracted/bin/vscode-css-language-server";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
Expand Down
3 changes: 1 addition & 2 deletions crates/languages/src/dockerfile.rs
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@ use std::{
};
use util::{async_maybe, ResultExt};

const SERVER_PATH: &'static str =
"node_modules/dockerfile-language-server-nodejs/bin/docker-langserver";
const SERVER_PATH: &str = "node_modules/dockerfile-language-server-nodejs/bin/docker-langserver";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
4 changes: 2 additions & 2 deletions crates/languages/src/elm.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@ use std::{
};
use util::ResultExt;

const SERVER_NAME: &'static str = "elm-language-server";
const SERVER_PATH: &'static str = "node_modules/@elm-tooling/elm-language-server/out/node/index.js";
const SERVER_NAME: &str = "elm-language-server";
const SERVER_PATH: &str = "node_modules/@elm-tooling/elm-language-server/out/node/index.js";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/html.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ use std::{
};
use util::{async_maybe, ResultExt};

const SERVER_PATH: &'static str =
const SERVER_PATH: &str =
"node_modules/vscode-langservers-extracted/bin/vscode-html-language-server";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
Expand Down
3 changes: 1 addition & 2 deletions crates/languages/src/json.rs
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ use std::{
};
use util::{async_maybe, paths, ResultExt};

const SERVER_PATH: &'static str =
"node_modules/vscode-json-languageserver/bin/vscode-json-languageserver";
const SERVER_PATH: &str = "node_modules/vscode-json-languageserver/bin/vscode-json-languageserver";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/prisma.rs
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ use std::{
};
use util::{async_maybe, ResultExt};

const SERVER_PATH: &'static str = "node_modules/.bin/prisma-language-server";
const SERVER_PATH: &str = "node_modules/.bin/prisma-language-server";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/purescript.rs
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ use std::{
};
use util::{async_maybe, ResultExt};

const SERVER_PATH: &'static str = "node_modules/.bin/purescript-language-server";
const SERVER_PATH: &str = "node_modules/.bin/purescript-language-server";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/python.rs
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ use std::{
};
use util::ResultExt;

const SERVER_PATH: &'static str = "node_modules/pyright/langserver.index.js";
const SERVER_PATH: &str = "node_modules/pyright/langserver.index.js";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/svelte.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ use std::{
};
use util::{async_maybe, ResultExt};

const SERVER_PATH: &'static str = "node_modules/svelte-language-server/bin/server.js";
const SERVER_PATH: &str = "node_modules/svelte-language-server/bin/server.js";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/tailwind.rs
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ use std::{
};
use util::{async_maybe, ResultExt};

const SERVER_PATH: &'static str = "node_modules/.bin/tailwindcss-language-server";
const SERVER_PATH: &str = "node_modules/.bin/tailwindcss-language-server";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
2 changes: 1 addition & 1 deletion crates/languages/src/yaml.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ use std::{
};
use util::{async_maybe, ResultExt};

const SERVER_PATH: &'static str = "node_modules/yaml-language-server/bin/yaml-language-server";
const SERVER_PATH: &str = "node_modules/yaml-language-server/bin/yaml-language-server";

fn server_binary_arguments(server_path: &Path) -> Vec<OsString> {
vec![server_path.into(), "--stdio".into()]
Expand Down
16 changes: 8 additions & 8 deletions crates/project_core/src/worktree_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2102,7 +2102,7 @@ async fn test_git_repository_for_path(cx: &mut TestAppContext) {
async fn test_git_status(cx: &mut TestAppContext) {
init_test(cx);
cx.executor().allow_parking();
const IGNORE_RULE: &'static str = "**/target";
const IGNORE_RULE: &str = "**/target";

let root = temp_tree(json!({
"project": {
Expand All @@ -2122,12 +2122,12 @@ async fn test_git_status(cx: &mut TestAppContext) {

}));

const A_TXT: &'static str = "a.txt";
const B_TXT: &'static str = "b.txt";
const E_TXT: &'static str = "c/d/e.txt";
const F_TXT: &'static str = "f.txt";
const DOTGITIGNORE: &'static str = ".gitignore";
const BUILD_FILE: &'static str = "target/build_file";
const A_TXT: &str = "a.txt";
const B_TXT: &str = "b.txt";
const E_TXT: &str = "c/d/e.txt";
const F_TXT: &str = "f.txt";
const DOTGITIGNORE: &str = ".gitignore";
const BUILD_FILE: &str = "target/build_file";
let project_path = Path::new("project");

// Set up git repository before creating the worktree.
Expand Down Expand Up @@ -2243,7 +2243,7 @@ async fn test_git_status(cx: &mut TestAppContext) {
cx.executor().run_until_parked();

let mut renamed_dir_name = "first_directory/second_directory";
const RENAMED_FILE: &'static str = "rf.txt";
const RENAMED_FILE: &str = "rf.txt";

std::fs::create_dir_all(work_dir.join(renamed_dir_name)).unwrap();
std::fs::write(
Expand Down
10 changes: 5 additions & 5 deletions crates/project_panel/src/file_associations.rs
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,11 @@ pub struct FileAssociations {

impl Global for FileAssociations {}

const COLLAPSED_DIRECTORY_TYPE: &'static str = "collapsed_folder";
const EXPANDED_DIRECTORY_TYPE: &'static str = "expanded_folder";
const COLLAPSED_CHEVRON_TYPE: &'static str = "collapsed_chevron";
const EXPANDED_CHEVRON_TYPE: &'static str = "expanded_chevron";
pub const FILE_TYPES_ASSET: &'static str = "icons/file_icons/file_types.json";
const COLLAPSED_DIRECTORY_TYPE: &str = "collapsed_folder";
const EXPANDED_DIRECTORY_TYPE: &str = "expanded_folder";
const COLLAPSED_CHEVRON_TYPE: &str = "collapsed_chevron";
const EXPANDED_CHEVRON_TYPE: &str = "expanded_chevron";
pub const FILE_TYPES_ASSET: &str = "icons/file_icons/file_types.json";

pub fn init(assets: impl AssetSource, cx: &mut AppContext) {
cx.set_global(FileAssociations::new(assets))
Expand Down
2 changes: 1 addition & 1 deletion crates/project_panel/src/project_panel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ use workspace::{
Workspace,
};

const PROJECT_PANEL_KEY: &'static str = "ProjectPanel";
const PROJECT_PANEL_KEY: &str = "ProjectPanel";
const NEW_ENTRY_ID: ProjectEntryId = ProjectEntryId::MAX;

pub struct ProjectPanel {
Expand Down
4 changes: 2 additions & 2 deletions crates/rpc/src/notification.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ use serde::{Deserialize, Serialize};
use serde_json::{map, Value};
use strum::{EnumVariantNames, VariantNames as _};

const KIND: &'static str = "kind";
const ENTITY_ID: &'static str = "entity_id";
const KIND: &str = "kind";
const ENTITY_ID: &str = "entity_id";

/// A notification that can be stored, associated with a given recipient.
///
Expand Down
2 changes: 1 addition & 1 deletion crates/terminal_view/src/terminal_panel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ use workspace::{

use anyhow::Result;

const TERMINAL_PANEL_KEY: &'static str = "TerminalPanel";
const TERMINAL_PANEL_KEY: &str = "TerminalPanel";

actions!(terminal_panel, [ToggleFocus]);

Expand Down
2 changes: 1 addition & 1 deletion crates/ui/src/components/stories/list_item.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ use story::Story;
use crate::{prelude::*, Avatar};
use crate::{IconName, ListItem};

const OVERFLOWING_TEXT: &'static str = "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Aenean mauris ligula, luctus vel dignissim eu, vestibulum sed libero. Sed at convallis velit.";
const OVERFLOWING_TEXT: &str = "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Aenean mauris ligula, luctus vel dignissim eu, vestibulum sed libero. Sed at convallis velit.";

pub struct ListItemStory;

Expand Down
2 changes: 1 addition & 1 deletion crates/vim/src/object.rs
Original file line number Diff line number Diff line change
Expand Up @@ -835,7 +835,7 @@ mod test {
test::{ExemptionFeatures, NeovimBackedTestContext, VimTestContext},
};

const WORD_LOCATIONS: &'static str = indoc! {"
const WORD_LOCATIONS: &str = indoc! {"
The quick ˇbrowˇnˇ•••
fox ˇjuˇmpsˇ over
the lazy dogˇ••
Expand Down
Loading