Re: [PULL] topic/core-stuff

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 24, 2014 at 6:24 PM, Ilia Mirkin <imirkin@xxxxxxxxxxxx> wrote:
> On Wed, Sep 24, 2014 at 6:24 AM, Daniel Vetter <daniel.vetter@xxxxxxxx> wrote:
>> Hi Dave,
>>
>> Just noticed that you've picked up the header rework stuff already, so
>> I've rebased that out again. Otherwise just two stragglers from the vblank
>> rework and the universal cursor planes locking fix. Plus sprinkling
>> container_of all over fbdev emulation from Fabian.
>
> I thought it was questionable whether these were desired... the
> pattern of casting between structs that contain others as bases is a
> pretty common one, and container_of just makes it harder to read. Did
> the discussion end in a way that concluded the opposite? I might have
> missed it.

I like them since you can employ gcc to check mundane stuff for you.
And container_of is so massively established in the kernel (with
list_head alone) that I don't think any serious kernel hacker has
problem's reading it. Alan Cox complained a bit a bit on imo elitist
grounds so until someone goes on record with an official Nack I think
they're worth it.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux