[PATCH 35/58] drm/i915: introduce struct intel_set_config

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

 



On Wed, Sep 5, 2012 at 6:34 PM, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
>         if (!config)
>                 return;
> ?
>
> My eyes always hurt a little when I see if blocks that take up the
> whole function. :)  Just a nit pick though.

Yeah, looks better. Bikeshed applied locally since it resulted in some
conflicts later on.
-Daniel
-- 
Daniel Vetter
daniel.vetter at ffwll.ch - +41 (0) 79 365 57 48 - http://blog.ffwll.ch


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux