Re: [PATCH] rust: kernel: str: Implement Debug for CString

From: Miguel Ojeda
Date: Fri Jul 14 2023 - 11:02:29 EST


On Fri, Jul 14, 2023 at 4:02 PM Alice Ryhl <aliceryhl@xxxxxxxxxx> wrote:
>
> Adding some sort of motivation usually works quite well, e.g.:
>
> Make it possible to use a CString with the `pr_*` macros directly, that
> is, instead of
>
> pr_debug!("trying to open {:?}\n", &*filename);
>
> we can now write
>
> pr_debug!("trying to open {:?}\n", filename);

Indeed, this would be the most important bit, i.e. answering the "why?".

The "what?" and the "how?" are pretty much explained by the title, but
it is also fine giving more details (but if the implementation
requires an explanation, then it is usually best to write an actual
source code comment instead).

Cheers,
Miguel