[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAH5fLgjDt0aDw6RwCVixsJZGJRi2-_ygOs+CPHGE-pvJVVHhcA@mail.gmail.com>
Date: Tue, 4 Mar 2025 13:52:37 +0100
From: Alice Ryhl <aliceryhl@...gle.com>
To: Guilherme Giacomo Simoes <trintaeoitogc@...il.com>
Cc: a.hindborg@...nel.org, alex.gaynor@...il.com, apw@...onical.com,
arnd@...db.de, aswinunni01@...il.com, axboe@...nel.dk, benno.lossin@...ton.me,
bhelgaas@...gle.com, bjorn3_gh@...tonmail.com, boqun.feng@...il.com,
dakr@...nel.org, dwaipayanray1@...il.com, ethan.twardy@...il.com,
fujita.tomonori@...il.com, gary@...yguo.net, gregkh@...uxfoundation.org,
joe@...ches.com, lukas.bulwahn@...il.com, ojeda@...nel.org,
pbonzini@...hat.com, tmgross@...ch.edu, walmeida@...rosoft.com,
rust-for-linux@...r.kernel.org, linux-kernel@...r.kernel.org,
Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
Subject: Re: [PATCH V6 1/2] rust: module: change author to an array
On Sun, Feb 23, 2025 at 6:42 PM Guilherme Giacomo Simoes
<trintaeoitogc@...il.com> wrote:
>
> In the module! macro, the author field is currently of type String.
> Since modules can have multiple authors, this limitation prevents
> specifying more than one.
> Change the author field to Option<Vec<String>> to allow creating modules
> with multiple authors. Additionally, rename the field from author to
> authors to make it explicit that it can refer to multiple authors. In
> modules that use the author field, update its value to an array of
> strings, and also rename it from author to authors.
>
> Suggested-by: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
> Link: https://github.com/Rust-for-Linux/linux/issues/244
> Signed-off-by: Guilherme Giacomo Simoes <trintaeoitogc@...il.com>
Reviewed-by: Alice Ryhl <aliceryhl@...gle.com>
Powered by blists - more mailing lists