Re: [PATCH v3] rust: clist: Add support to interface with C linked lists

From: John Hubbard
Date: Mon Dec 01 2025 - 14:36:06 EST


On 12/1/25 8:51 AM, Daniel Almeida wrote:
>> On 29 Nov 2025, at 18:30, Joel Fernandes <joelagnelf@xxxxxxxxxx> wrote:
...
>> +#[repr(transparent)]
>> +pub struct ClistHead(Opaque<bindings::list_head>);
>
> I still think we should call this CList. IMHO, it does not make sense to have a

I am guessing you meant to place this comment after Clist, rather than here
(after ClistHead)? Otherwise I don't know what you are actually suggesting?

> Clist, and a ClistHead (notice the capitalization). CList and CListHead are
> easier to read and reason about.
>
> Did anyone push back on this?

If you are simply recommending renaming:
Clist --> CList
ClistHead --> CListHead

...then I'd say "+1" for that suggestion.

thanks,
--
John Hubbard