Re: [PATCH v2 2/2] Makefile: improve comment documentation for the rust-analyzer target
From: Miguel Ojeda
Date: Wed Jun 26 2024 - 04:43:03 EST
On Wed, Jun 26, 2024 at 10:08 AM Alice Ryhl <aliceryhl@xxxxxxxxxx> wrote:
>
> Is "symbol browsing database" the right word here? It's not actually a
> list of symbols, but instructions for how to compile the code.
Yeah, I think the sentence mixes a bit what the file is with what
(some of) the users of the file do with it.
What about something like (getting inspiration from the official documentation):
# Generate `rust-project.json` (a file that describes the
structure of non-Cargo Rust projects) for `rust-analyzer` (an
implementation of the Language Server Protocol).
I would avoid mentioning `compile_commands.json`, since they are
slightly different the Rust one does not really contain the compile
commands.
As for "IDE", I am happy either way (i.e. removing it or not). Another
alternative that may clarify by giving context could be "Editor / IDE"
(since one may use LSP with "simple editors" and not "full IDEs"
anyway).
With that changed, if Masahiro wants to pick these two up:
Acked-by: Miguel Ojeda <ojeda@xxxxxxxxxx>
Otherwise I am happy to take them too.
Cheers,
Miguel