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