[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CANiq72m=wBvQOYo6-Uwbc7icou7bBCZNJgpK0DiaCwMKmY4OaQ@mail.gmail.com>
Date: Fri, 7 Apr 2023 00:33:26 +0200
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: Gary Guo <gary@...yguo.net>,
Wedson Almeida Filho <wedsonaf@...il.com>
Cc: Asahi Lina <lina@...hilina.net>,
Andreas Hindborg <nmi@...aspace.dk>,
rust-for-linux@...r.kernel.org,
Andreas Hindborg <a.hindborg@...sung.com>,
Miguel Ojeda <ojeda@...nel.org>,
Alex Gaynor <alex.gaynor@...il.com>,
Boqun Feng <boqun.feng@...il.com>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>,
open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] scripts: rust-analyzer: Skip crate module directories
On Tue, Mar 7, 2023 at 6:14 PM Miguel Ojeda
<miguel.ojeda.sandonis@...il.com> wrote:
>
> I don't have a strong opinion either way -- this was originally done
> to improve fuzzy searching, see commit 829c2df153d7 ("rust: move `net`
> and `sync` modules to uniquely-named files") upstream:
>
> This is so that each file in the module has a unique name instead of the
> generic `mod.rs` name. It makes it easier to open files when using fuzzy
> finders like `fzf` once names are unique.
Apparently the "encouraged" way is using `name.rs`:
https://doc.rust-lang.org/stable/reference/items/modules.html#module-source-filenames
https://doc.rust-lang.org/edition-guide/rust-2018/path-changes.html#no-more-modrs
Another argument I saw for `name.rs` is that one can easily the name
of the file in editor's tabs/titles, and some of the editors can add
part of the path to disambiguate, which may take more space in the UI.
Cheers,
Miguel
Powered by blists - more mailing lists