[PATCH 5/9] compositor: Output unplug in clone mode

Zhang, Xiong Y xiong.y.zhang at intel.com
Sun Feb 16 19:11:20 PST 2014


On Fri, 2014-02-14 at 09:56 +0200, Pekka Paalanen wrote:
> On Fri, 14 Feb 2014 15:17:40 +0800
> Xiong Zhang <xiong.y.zhang at intel.com> wrote:

> Hi,
> 
> just a general note: it seems the DRM backend code is getting into
> pretty deep indentation levels. I would recommend splitting chunks out
> into functions where you have a logical does-one-thing block, even
> if the function would be only used in one place. It makes the code flow
> easier to read, when the blocks have been chosen properly and the
> resulting functions have descriptive names. Only if possible, of course.
> 
> 
Thanks for your recommendation. I will remember it and modify the code.

thanks 
> Thanks,
> pq



More information about the wayland-devel mailing list