diff options
| author | Dylan DPC <99973273+Dylan-DPC@users.noreply.github.com> | 2022-11-01 22:01:37 +0530 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-01 22:01:37 +0530 |
| commit | 2b10891b969dea3875cd42a5b63d198abd47fcbc (patch) | |
| tree | a1fa959a0328fc522144dbf0c4be5e37732c72c5 /src/tools/rust-analyzer/docs/dev | |
| parent | 2b0ead31261e36185b1b41847f717be7641348fc (diff) | |
| parent | 0b3e75e1cdc8d56fc5540e17450d96228b26e268 (diff) | |
| download | rust-2b10891b969dea3875cd42a5b63d198abd47fcbc.tar.gz rust-2b10891b969dea3875cd42a5b63d198abd47fcbc.zip | |
Rollup merge of #103833 - lnicola:rust-analyzer-2022-11-01, r=lnicola
:arrow_up: rust-analyzer r? `@ghost`
Diffstat (limited to 'src/tools/rust-analyzer/docs/dev')
| -rw-r--r-- | src/tools/rust-analyzer/docs/dev/architecture.md | 2 | ||||
| -rw-r--r-- | src/tools/rust-analyzer/docs/dev/lsp-extensions.md | 8 |
2 files changed, 3 insertions, 7 deletions
diff --git a/src/tools/rust-analyzer/docs/dev/architecture.md b/src/tools/rust-analyzer/docs/dev/architecture.md index c173a239fea..e3a4fdfda90 100644 --- a/src/tools/rust-analyzer/docs/dev/architecture.md +++ b/src/tools/rust-analyzer/docs/dev/architecture.md @@ -479,7 +479,9 @@ It is not cheap enough to enable in prod, and this is a bug which should be fixe ### Configurability rust-analyzer strives to be as configurable as possible while offering reasonable defaults where no configuration exists yet. +The rule of thumb is to enable most features by default unless they are buggy or degrade performance too much. There will always be features that some people find more annoying than helpful, so giving the users the ability to tweak or disable these is a big part of offering a good user experience. +Enabling them by default is a matter of discoverability, as many users end up don't know about some features even though they are presented in the manual. Mind the code--architecture gap: at the moment, we are using fewer feature flags than we really should. ### Serialization diff --git a/src/tools/rust-analyzer/docs/dev/lsp-extensions.md b/src/tools/rust-analyzer/docs/dev/lsp-extensions.md index 6d2c7d7b063..fe316fcae9b 100644 --- a/src/tools/rust-analyzer/docs/dev/lsp-extensions.md +++ b/src/tools/rust-analyzer/docs/dev/lsp-extensions.md @@ -1,5 +1,5 @@ <!--- -lsp_ext.rs hash: 7b710095d773b978 +lsp_ext.rs hash: 62068e53ac202dc8 If you need to change the above hash to make the test pass, please check if you need to adjust this doc as well and ping this issue: @@ -19,12 +19,6 @@ Requests, which are likely to always remain specific to `rust-analyzer` are unde If you want to be notified about the changes to this document, subscribe to [#4604](https://github.com/rust-lang/rust-analyzer/issues/4604). -## UTF-8 offsets - -rust-analyzer supports clangd's extension for opting into UTF-8 as the coordinate space for offsets (by default, LSP uses UTF-16 offsets). - -https://clangd.llvm.org/extensions.html#utf-8-offsets - ## Configuration in `initializationOptions` **Upstream Issue:** https://github.com/microsoft/language-server-protocol/issues/567 |