On Fri, 2014-02-14 at 09:56 +0200, Pekka Paalanen wrote: > On Fri, 14 Feb 2014 15:17:40 +0800 > Xiong Zhang <[email protected]> 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 _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
